[IV-21-22] Objetivo 4 - Githubissues

edusegrich commented 2 years ago

PR para el Objetivo 4. Se ha realizado la elección de un test runner y una biblioteca de aserciones, así como la implementación de los tests que supervisan la lógica de negocio del PMV del Milestone 2.5.

Lista de comprobación del objetivo:

[x] ¿Hay un camino claro que permita ir desde el código hasta la historia de usuario que desarrolla, pasando por el mensaje de commit y el issue correspondiente?
[x] ¿Se ha intentado cubrir con tests una parte de la lógica de negocio que se esté desarrollando?
[x] ¿Hay un producto mínimamente viable que describa lo que se va a entregar en este objetivo?
[x] ¿En este producto mínimamente viable se han priorizado unas clases, módulos o paquetes más fundamentales frente a otros, movidos a otro PMV/Hito?
[x] ¿Se han documentado las elecciones de biblioteca de aserciones y del de test runner?
[x] ¿Te has asegurado que lo que mencionas como biblioteca de aserciones y test runners lo son realmente y tienen la misma funcionalidad? ¿O estás poniendo unittest como test runner y pytest como biblioteca de aserciones?
[x] ¿Esa documentación incluye criterios de aceptación y también criterios de búsqueda para las diferentes opciones?
[x] ¿Se han seguido los principios F.I.R.S.T. en la creación del test y se ha documentado cómo se ha hecho?
[x] ¿Se describe claramente en el PMV los tests que hay que pasar para que se considere viable?

edusegrich commented 2 years ago

¿Ahora deberían ser asignados unos revisores antes de realizar la entrega del objetivo?

JJ commented 2 years ago

⛹ Revisores → @Olasergiolas @Pablont98 @danifm1321

edusegrich commented 2 years ago

Muchas gracias por vuestras correcciones, @danifm1321 @noise-kngdm. Voy a ponerme a trabajar en todo ello y os aviso en cuanto lo tenga.

edusegrich commented 2 years ago

Listo para revisión @danifm1321 @noise-kngdm. He intentado corregir todo lo que me habéis dicho. En cuanto al test runner he decidido continuar con pytest ya que me siento bastante cómodo con él. No sé si tenéis alguna objeción o comentario al respecto.

edusegrich commented 2 years ago

He corregido todo lo que me has comentado, @noise-kngdm. Listo para revisión.

edusegrich commented 2 years ago

Añadido el tiempo de ejecución de los tests en las notas del desarrollo. Eliminada aserción que no tenía sentido de test_preguntas.py. Establecida la biblioteca de aserciones assertpy solo como dependencia de desarrollo.

@noise-kngdm

edusegrich commented 2 years ago

Hecho @noise-kngdm. De verdad, te agradezco mucho tu dedicación y tus claras correcciones :smiley:.

edusegrich commented 2 years ago

@JJ listo para revisión.

edusegrich commented 2 years ago

@JJ podría echarle un vistazo a los tests para ver si se ha testeado la lógica de negocio? Ya que no he superado el objetivo me gustaría saber todo lo que he fallado en él para poder corregirlo de cara a la convocatoria extraordinaria...

JJ commented 2 years ago

⛹ Revisores → @modejota @Parka015 @Balrrach

edusegrich / OpoTests

[IV-21-22] Objetivo 4 #31