Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fleet management tests #1829

Merged
merged 5 commits into from
Jun 12, 2024
Merged

Conversation

unjust
Copy link
Member

@unjust unjust commented Jun 4, 2024

Addresses #1809 (continues #1817)

Aggrega

  • Tests en el directorio /postman una coleccion sin auth y otro con auth
  • Una seccion de readme sobre testing y como descargar newman
  • Agrega un video que hizo Christian de como correrlos

Aqui es ejemplo output para ver clara el tipo de tests que hay

@unjust unjust changed the base branch from main to next June 4, 2024 02:33
@unjust unjust changed the title Fleet management tests 1809 Fleet management tests Jun 4, 2024
@unjust unjust requested a review from ssinuco June 4, 2024 02:33
@unjust unjust added this to the Fleet Management milestone Jun 4, 2024
@unjust unjust linked an issue Jun 4, 2024 that may be closed by this pull request
3 tasks
@unjust
Copy link
Member Author

unjust commented Jun 4, 2024

@ssinuco tengo un poco de comentarios sobre el API spec que me ocurrio mientras escribia estes tests:

  1. en /trajectories podemos nombrar la propiedad timestamp en vez de date
  2. no hay paginacion para /trajectories
  3. debemos tener un 404 para taxi not found con /trajectories/taxiId= ?
  4. debemos tener un 400 si no hay params de taxiId y date con trajectories?
  5. PATCH de /users de verdad solo podemos actualizar de name y no email ni password ?
  6. debemos retornar 201 para post /users con exito
  7. /taxis debe tener 400 para page y limit invalid errores? creo tenemos eso para /users

Copy link
Collaborator

@ssinuco ssinuco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Algunas mejoras pequeñas en la redaccion. No pude ejecutar las pruebas contra una API funcional.

projects/05-fleet-management-api/README.md Outdated Show resolved Hide resolved
projects/05-fleet-management-api/README.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@ssinuco ssinuco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ivy:

✅ 1. en /trajectories podemos nombrar la propiedad timestamp en vez de date -->

  1. no hay paginacion para /trajectories

Este endoint esta pensando para ser usado en una app web que muestre las trayectorias en un mapa, tal como se muestra en la animacion que agregue en la historia de usuario. Entonces por eso no hay paginacion.

✅ 5. debemos tener un 404 para taxi not found con /trajectories/taxiId= ?

✅ 6. debemos tener un 400 si no hay params de taxiId y date con trajectories?

  1. PATCH de /users de verdad solo podemos actualizar de name y no email ni password ?

Normalmente el email no es posible cambiarlo. Para el password las API Rest proveen otro endpoint. Aqui en esta API no lo implementamos.

✅ 4. debemos retornar 201 para post /users con exito

✅ 8. /taxis debe tener 400 para page y limit invalid errores? creo tenemos eso para /users

Co-authored-by: Sergio Sinuco <ssinuco@gmail.com>
@ssinuco ssinuco self-requested a review June 11, 2024 20:16
@unjust unjust merged commit d02fc9a into Laboratoria:next Jun 12, 2024
4 checks passed
@unjust unjust linked an issue Jun 12, 2024 that may be closed by this pull request
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Opciones para escribir tests para Fleet Management Prep Fleet Management to replace BQ API in Bootcamp
3 participants