-
Notifications
You must be signed in to change notification settings - Fork 97
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
Run tests parallely in Circle CI #762
Conversation
Makefile
Outdated
@@ -19,3 +19,4 @@ test: check-syntax-errors check-no-prints | |||
@# Launch tests from openfisca_france/tests directory (and not .) because TaxBenefitSystem must be initialized | |||
@# before parsing source files containing formulas. | |||
nosetests tests --exe --with-doctest | |||
openfisca-run-test -c openfisca_france --nose tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use long option for
-c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
@fpagnoux, cette configuration CircleCI utilise 4 conteneurs pour un build. Une conséquence de cela, c'est qu'un seul build peut être lancé à la fois. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💃
circle.yml
Outdated
parallel: true | ||
files: | ||
- tests/**/*.yaml | ||
- tests/**/*.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use a single glob:
tests/**/*.ya?ml
Makefile
Outdated
@@ -19,3 +19,4 @@ test: check-syntax-errors check-no-prints | |||
@# Launch tests from openfisca_france/tests directory (and not .) because TaxBenefitSystem must be initialized | |||
@# before parsing source files containing formulas. | |||
nosetests tests --exe --with-doctest | |||
openfisca-run-test -c openfisca_france --nose tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Depends on openfisca/openfisca-core#516
Connected to #734
Pas trop d'enthousiasme : une fois qu'on aura ré-introduit les tests, le build sera long de nouveau ;)
Ces changements :
Preuve que ça fonctionne :