Skip to content

Commit

Permalink
Disable passing extra arguments option for pytest invocation for midd…
Browse files Browse the repository at this point in the history
…leman_protocol and signing_service

Due to a BUG in pytest: (pytest-dev/pytest#1618) `-p no:django` doesn't work for parallel runs, causing pytest to expect Django setup in non-Django apps and thus failing.
  • Loading branch information
Piotr Dybowski committed Sep 7, 2018
1 parent 616a9a4 commit 352e580
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions run_tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -49,16 +49,18 @@ cd middleman_protocol/
pytest -p no:django \
--cov-report term-missing \
--cov-config ../coverage-config \
--cov=$MODULE $PATTERN \
${TEST_RUNNER_EXTRA_ARGUMENTS}
--cov=$MODULE $PATTERN
# temporarily disabled due to a BUG in pytest-django / pytest-xdist
# ${TEST_RUNNER_EXTRA_ARGUMENTS}
rm .coverage
cd ..

cd signing_service/
pytest -p no:django \
--cov-report term-missing \
--cov-config ../coverage-config \
--cov=$MODULE $PATTERN \
${TEST_RUNNER_EXTRA_ARGUMENTS}
--cov=$MODULE $PATTERN
# temporarily disabled due to a BUG in pytest-django / pytest-xdist
# ${TEST_RUNNER_EXTRA_ARGUMENTS}
rm .coverage
cd ..

0 comments on commit 352e580

Please sign in to comment.