-
Notifications
You must be signed in to change notification settings - Fork 460
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
Fleet management tests #1829
Conversation
@ssinuco tengo un poco de comentarios sobre el API spec que me ocurrio mientras escribia estes tests:
|
There was a problem hiding this 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.
There was a problem hiding this 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
-->
- 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?
- PATCH de
/users
de verdad solo podemos actualizar dename
y noemail
nipassword
?
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>
Addresses #1809 (continues #1817)
Aggrega
/postman
una coleccion sin auth y otro con authAqui es ejemplo output para ver clara el tipo de tests que hay