Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ci-config): run simplecov after unit and before system tests #416

Merged
merged 2 commits into from
Sep 23, 2022

Conversation

difernandez
Copy link
Contributor

Contexto

Los proyectos generados recientemente con potassium tienen configurado para que monkeyci comente en los prs cuando alguno de los cambios introducidos en el pr no está testeado. Está pasando que el mono habitualmente se equivoca y comenta sobre líneas que si están testeadas, incluso en el mismo PR.

Qué se esta haciendo

Se arregla este bug. El problema era que, el reporte de coverage que se generaba debido a los tests de sistema estaba sobreescribiendo al reporte de los unitarios. Es decir, reportaba cualquier cambio del pr que no estuviera cubierto por un test de sistema.
Para solucionarlo, simplemente se mueve el reporte de coverage de reviewdog para que ocurra después de los unitarios pero antes de los de sistema. Otra alternativa hubiera sido mergear ambos reportes, que está soportado por simplecov, pero en general vamos a querer asegurar coverage con test unitarios en primer lugar, y con eso en mente también están pensados los archivos que están considerados para el coverage.

Copy link
Collaborator

@ldlsegovia ldlsegovia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tremendo como mover una línea dejaba un desastreee!

@difernandez difernandez merged commit 925b026 into master Sep 23, 2022
@difernandez difernandez deleted the fix-ci-coverage-comments branch September 23, 2022 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants