Actividad 4 - Grupo 2

[BenjaminPeres596]

https://github.com/JuaniLezcano/vetsoft

[ivanduva]

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

[ValentinMilocco]

A su vez se me genero otra consulta de como subir la solucion, debo hacer un push con merge directo? o debo hacer un pr?

[ivanduva]

Son dos cosas, por un lado el error no aparece en el form y por otro los tests de unidad e integración no prueban que aparezca el error que ustedes definieron en la función validate_client.
Tienen que trabajar como lo vienen haciendo, con PRs y revisión de los demás integrantes.
Saludos

[ValentinMilocco]

Hola Ivan, ya sacamos la nueva Release.

[ivanduva]

Buenas,
Hay algunos problemas. Para cerrar el trabajo le vamos a pedir a Valentín lo siguiente:

• En el test test_validation_invalid_email prueban que salte el error con un texto que no tiene formato de email. La idea es que prueben con un formato de mail pero que no termine en "vetsoft.com", que es la validación nueva que ustedes agregaron. Agregá otro test que pruebe eso.
• El test e2e que agregaron no hace ningún expect más allá de este: expect(self.page.get_by_role("form")).to_be_visible() que prueba que el form esté visible. Por lo tanto no prueba nada. Debería probar que salte el error en el form.
• Luego de arreglar esto, sacá una release nueva.

Los demás integrantes deberán hacer review de lo que está haciendo Valentín.

Saludos

[PeresValentin]

Hola Iván! Ahí acomode los test como me pediste, ya esta la nueva Release.
Saludos!

[ivanduva]

Perfecto, el trabajo integrador queda aprobado.

Saludos!