Conversation
tests/test_auth.py
Outdated
|
|
||
|
|
||
| def test_configure_flaat_logs_modes(logger): | ||
| def test_configure_flaat_logs_modes(logger, mandatory_settings_fields): |
There was a problem hiding this comment.
Non ho la fissa di commentare tutti gli arg delle funzioni, però qui forse non è molto chiaro quali siano i "mandatory settings fields" quindi metterei o un nome più parlante o la doc string per chiarire cosa effettivamente ci va
There was a problem hiding this comment.
I test di pytest sono definiti tramite funzioni che iniziano con test. Queste funzioni però non vengono "chiamate/usate" realmente da qualcuno. I parametri che vedi nella funzione sono al 99% delle fixture (ovvero delle altre funzioni).
In ogni caso aggiornerò la doc della funzione per spiegare meglio cosa contengono i parametri passati al test.
giosava94
force-pushed
the
33-update-cicd
branch
from
7ec73f8 to
f740ef5
Compare
|
2 participants
Add Jenkinsfile to top directory. This pipeline executes tests on any branch. Moreover executes linting and formatting and builds and pushes docker images on PR.