Actividad 4 - Grupo 9

[tespinosa9]

https://github.com/EmilianoColavita/vetsoft

[ivanduva]

Buenas! Les puedo pedir el link a donde lo deployaron?

[tespinosa9]

Hola Iván! Te dejo el enlace a Render

[RodrigoJacznik]

Buenas. Les dejo el feedback de la actividad 3 y 4.

Actividad 3:

Me gusto que hayan elegido un formato para los mensajes de los commits y se hayan comprometido en seguirlo.
Tambien me gusto que hayan armado los fixtures para cargar datos en la db
Los nombres de los branch, el titulo de los PRs y su descripción podrían estar mejor. Son muy pocos descriptivos. Para saber de que se trata un PR tengo que ir a ver el tp.

La actividad esta entregada fuera de termino y algunos PRs necesitan mas trabajo.
EmilianoColavita/vetsoft#6: Hace varias funcionalidades en el mismo PR. No tiene test e2e
EmilianoColavita/vetsoft#10: No permite guardar una mascota con fecha de nacimiento en el futuro pero no muestra ningun error. Seria interesante que lo muestre. Los test e2e fallan.
EmilianoColavita/vetsoft#12: No tiene test de integración ni e2e

Al final de la actividad (commit fdf03d1), varios test e2e fallan.

Actividad 4.

Encontre varios errores en esta actividad

• No hicieron CI/CD antes de empezar la actividad.
• El CI no corre los tests ni el coverage.
• Tomas y Nicolas no hicieron tareas del punto 3.
• Se repite el problema con los PRs (malos nombres y descripciones)

Les pido por favor que reelean la actividad y corrigan lo siguiente.
Antes de empezar agreguen al CI las tareas que corren los test unitarios, integracion y e2e. Ademas que controle el coverage.

@tespinosa9 y Nicolas, tienen que hacer una de las tareas del punto 3 (no se olviden los test)
Nicolas, agrega los test e2e faltantes del PR EmilianoColavita/vetsoft#6
Ian. Hace que muestre el error al querer guardar una mascota con fecha de nacimiento en el futuro. Hace los test
Tomas. Agrega los test de integración y de e2e del PR EmilianoColavita/vetsoft#12
Emiliano. Hace una de las tarea del punto 3 (que ya no este realizada). No te olvides de escribir los test

La actividad tiene que cumplir la premisa del coverage

Intenten mejorar el nombre de los PRs y su descripcion.

Tienen tiempo hasta el Lunes 8/7.