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

Expandir tests y Actions #2

Open
agucova opened this issue Jun 2, 2020 · 2 comments
Open

Expandir tests y Actions #2

agucova opened this issue Jun 2, 2020 · 2 comments

Comments

@agucova
Copy link
Contributor

agucova commented Jun 2, 2020

Los tests tienen una implementación bastante obscura y Travis CI podría eliminarse a favor de una Github Action, lo que sería más transparente y mantenible.

Aprovechando Github Actions, un linter como flake8 o prospector podría ayudar. Estuve pasándome por el código y hay muchos import innecesarios, violaciones de PEP 8 y PEP 257, y harta lógica innecesaria.

Véase aquí.

@agucova
Copy link
Contributor Author

agucova commented Jun 2, 2020

Intentando replicar localmente, noté que uno de los tests fallaba. Con un diff semántico del JSON llegué a esto:
image

No sé si esto es comportamiento esperado, pero el test parece estar pescando detalles como las vacantes que por razones lógicas cambian a cada rato. Actualicé la vacante y los cupos y pasé los tests.

@agucova
Copy link
Contributor Author

agucova commented Jun 2, 2020

Con relación a #1, se podría agregar un test con el parámetro de campus, falta coverage.

agucova added a commit to agucova/BC_API that referenced this issue Jun 2, 2020
As commented in igbasly#2, this adds flake8 and pytest.py to a Github Actions pipeline.

Only errors on failed tests, syntax errors or undefined names. Pending LL update.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant