[IV-21-22] Objetivo 4 #13

aleveji · 2021-12-01T11:27:24Z

¿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?
¿Se ha intentado cubrir una parte de la lógica de negocio que se
esté desarrollando?
¿Hay un producto mínimamente viable que describa lo que se va a entregar en
este objetivo?
¿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?
¿Se han documentado las elecciones de biblioteca de aserciones y de
framework de test?
¿Esa documentación incluye criterios de aceptación y también criterios de
búsqueda para las diferentes opciones?
¿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?
¿Se describe claramente en el PMV los tests que hay que pasar para que se
considere viable?

Añade el método toString a la clase Ingrediente para poder llevar a cabo la comprobación de la correcta creación de un Ingrediente en los tests.

Añade la clave test que punta al fichero de los test del repositorio

Añade el export para que pueda ser importado posteriormente

Añade el directorio en el que se encuentran los test del proyecto

Añade un test que comprueba el correcto funcionamiento de la clase Ingrediente. Para que se pase el test debe funcionar correctamente la creación de un Ingrediente y tanto el getter de su nombre como el de calorías deben funcionar correctamente.

Añade imágenes de ejecuciones de test con distintos frameworks como Mocha y Jest.

Añade el enlace del documento que justifica la elección del framework de tests

aleveji · 2021-12-01T15:44:29Z

Listo para revisión @JJ

JJ

Problema principal: ni hay lógica de negocio, ni la testeas. Problemas adicionales sobre las herramientas elegidas y el hecho de que el milestone lo has asignado a voleo en vez de molestarte en incluir lo que surja de este objetivo en el producto mínimamente viable.

JJ · 2021-12-07T17:28:02Z

app/ingrediente.js

        return this.calorias;
    }
+
+    toString() {
+        return "Ingrediente -> " + this.nombre + " [Calorias: " + this.calorias + ", Grasas: " + this.grasas + ", Hidratos: " + this.hidratos + ", Proteinas: " + this.proteinas + "]";


Esto sería mucho más fácil si usaras un diccionario.

JJ · 2021-12-07T17:28:44Z

app/test/test.js

+        });
+    });
+
+    describe('Getters', function() {


No estás testeando ninguna lógica de negocio. Está bien que testees, para dejar el interfaz fijado, pero lo que se pide en este objetivo es que se testee la lógica de negocio.

JJ · 2021-12-07T17:29:35Z

docs/eleccion_framework_test.md

+# Elección framework test
+## Mocha
+Para ejecutar los test de mi proyecto estaba indeciso entre Mocha y Jest, pero finalmente me decidí
+por Mocha ya que es uno de los frameworks más antiguos y cuenta con una gran comunidad. Además, trás


Dices antiguo, leo viejo. No solamente tienes que elegir un test runner, sino también una biblioteca de aserciones. Mira los errores frecuentes de la semana 13 para ver cómo hacer una elección técnica.

JJ · 2021-12-15T19:11:34Z

Ping?

Para ejecutar los test ahora se utilizará aqa en lugar de mocha

Añade el getter de proteinas a la clase Ingrediente y añade una función que maximiza las proteinas minimizando las calorias y devuelve un conjunto de Ingredientes para obtener una receta ideal en función de un objetivo de proteinas y calorias.

Actualiza el test para aqa. Ahora se crean 5 ingredientes y se comprueba su correcta creación y el correcto funcionacmiento de sus getters. Además, se comprueba el correcto funcionamiento de la función maxProteinas.

Añade imagen de varias ejecuciones de los tests con aqa.

Justifica la elección de aqa como nuevo testing framework.

aleveji · 2021-12-17T17:27:43Z

Listo para revisión @JJ

JJ

Te perdonaría lo de que no has puesto los criterios de elección ni las diferentes opciones por la originalidad de usar aqa, pero es que también tienes que tratar de testear la lógica de negocio, no solo creación de objetos, así que del tirón haz los dos, porfa.

JJ · 2021-12-17T19:20:07Z

package.json

@@ -22,6 +22,7 @@
  },
  "homepage": "https://github.com/aleveji/BuscaRecetas#readme",
  "dependencies": {
-    "mocha": "^9.1.3"
+    "aqa": "^1.2.14"


Es una dependencia de desarrollo.

aleveji · 2021-12-17T20:08:54Z

Te perdonaría lo de que no has puesto los criterios de elección ni las diferentes opciones por la originalidad de usar aqa, pero es que también tienes que tratar de testear la lógica de negocio, no solo creación de objetos, así que del tirón haz los dos, porfa.

Para testear la lógica de negocio lo que hecho ha sido una función que maximiza las proteínas en función de las calorías como me comentaste el otro día en clase. ¿No sería suficiente con esto? 🤔

JJ

OK, aprobado, pero como te he dicho si cambias las cosas comentadas antes del siguiente, mejor.

JJ · 2021-12-19T07:58:56Z

app/test/test.js

+    t.deepEqual(newIngrediente5.getCalorias(), 130, "Getter de calorias de Atún no funciona correctamente");
+    t.deepEqual(newIngrediente5.getProteinas(), 29, "Getter de proteinas de Atún no funciona correctamente");
+
+    t.deepEqual(ingrediente.maxProteinas(ingredientes,100).toString(),"Lechuga,Tomate,Piña","La funcion maxProteinas no funciona correctamente");


OK, ya lo veo. El mensaje debe ser en positivo

aleveji and others added 8 commits November 26, 2021 13:25

Añade método toString a la clase Ingrediente #6

c601313

Añade el método toString a la clase Ingrediente para poder llevar a cabo la comprobación de la correcta creación de un Ingrediente en los tests.

Añade clave test #12

66e2d3c

Añade la clave test que punta al fichero de los test del repositorio

Añade export a Ingrediente #6

7f0ab84

Añade el export para que pueda ser importado posteriormente

Arregla el script test #12

409e562

Añade el directorio en el que se encuentran los test del proyecto

Crea test de Ingrediente #6

acd67ee

Añade un test que comprueba el correcto funcionamiento de la clase Ingrediente. Para que se pase el test debe funcionar correctamente la creación de un Ingrediente y tanto el getter de su nombre como el de calorías deben funcionar correctamente.

Añade imágenes de ejecuciones de test #12

2b5183d

Añade imágenes de ejecuciones de test con distintos frameworks como Mocha y Jest.

Justifica la elección del framework de test #12

a0760cd

Arregla enlaces imágenes de los tests #12

abea251

aleveji added this to the [M3] Automatización de tareas milestone Dec 1, 2021

Añade enlace documentación adicional #12

decfe91

Añade el enlace del documento que justifica la elección del framework de tests

JJ suggested changes Dec 7, 2021

View reviewed changes

aleveji added 5 commits December 17, 2021 13:49

Cambia el testing framework de mocha a aqa #12

8ee8683

Para ejecutar los test ahora se utilizará aqa en lugar de mocha

Actualiza test de Ingrediente para aqa #6

c22cf5c

Actualiza el test para aqa. Ahora se crean 5 ingredientes y se comprueba su correcta creación y el correcto funcionacmiento de sus getters. Además, se comprueba el correcto funcionamiento de la función maxProteinas.

Añade imagen de test con aqa #12

dad2be3

Añade imagen de varias ejecuciones de los tests con aqa.

Actualiza la elección del framework de test #12

28f1853

Justifica la elección de aqa como nuevo testing framework.

JJ suggested changes Dec 17, 2021

View reviewed changes

JJ approved these changes Dec 19, 2021

View reviewed changes

aleveji merged commit ee131dd into main Jan 10, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[IV-21-22] Objetivo 4 #13

[IV-21-22] Objetivo 4 #13

aleveji commented Dec 1, 2021 •

edited

aleveji commented Dec 1, 2021

JJ left a comment

JJ Dec 7, 2021

JJ Dec 7, 2021

JJ Dec 7, 2021

JJ commented Dec 15, 2021

aleveji commented Dec 17, 2021

JJ left a comment

JJ Dec 17, 2021

aleveji commented Dec 17, 2021

JJ left a comment

JJ Dec 19, 2021

[IV-21-22] Objetivo 4 #13

[IV-21-22] Objetivo 4 #13

Conversation

aleveji commented Dec 1, 2021 • edited

aleveji commented Dec 1, 2021

JJ left a comment

Choose a reason for hiding this comment

JJ Dec 7, 2021

Choose a reason for hiding this comment

JJ Dec 7, 2021

Choose a reason for hiding this comment

JJ Dec 7, 2021

Choose a reason for hiding this comment

JJ commented Dec 15, 2021

aleveji commented Dec 17, 2021

JJ left a comment

Choose a reason for hiding this comment

JJ Dec 17, 2021

Choose a reason for hiding this comment

aleveji commented Dec 17, 2021

JJ left a comment

Choose a reason for hiding this comment

JJ Dec 19, 2021

Choose a reason for hiding this comment

aleveji commented Dec 1, 2021 •

edited