Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resumen, ver #182
diagram.test.ts
)dynamicDiagram.test.ts
que sí prueba toda la interacción del repl con el socket del diagrama dinámico, agregamos un finally para que cierre el server al final, ya que si tiraba error elserver.close()
no se ejecutaba y los otros tests que dependían de tener el puerto 3000 libre fallabanrun.test.ts
, ya que el test del game confiaba en chai, a partir de recibir una funcióndone
como parámetro. No obstante, necesitamos cerrar elioGame
. Al principio fui por la propuesta de agregar un parámetrodryRun
en las opciones del run, pero después se me ocurrió devolver elioGame
en la promesa. Con eso el test pudo poner unawait
, cerrar elioGame
a mano y luego testear que el juego anduvo correctamente. En caso de error logueamos el mensaje para tener una buena experiencia de desarrolladore. Para testear game creé un ejemplo más, y de paso adelanté la validación de la carpeta assets que estaba metida dentro del métodoeventFor
.La duda que me queda es acá:
¿está bien tener un doble listen?