grouped coverage testing steps together

This commit is contained in:
dvoraen 2022-12-01 06:43:20 -08:00
parent 37ac4d55a4
commit 868b3bdaeb

View file

@ -62,6 +62,18 @@ jobs:
evennia migrate
evennia collectstatic --noinput
# For non-coverage tests, run them in parallel.
- name: Run test suite
if: ! ${{ matrix.coverage-test }}
working-directory: testing_mygame
run: |
evennia test \
--settings=settings \
--keepdb \
--parallel 4 \
--timing \
evennia
# OBS - it's important to not run the coverage tests with --parallel, it messes up the coverage
# calculation!
- name: Run test suite with coverage
@ -79,18 +91,6 @@ jobs:
coverage --version
coverage report | grep TOTAL
# For other runs, run tests in parallel
- name: Run test suite
if: ! ${{ matrix.coverage-test }}
working-directory: testing_mygame
run: |
evennia test \
--settings=settings \
--keepdb \
--parallel 4 \
--timing \
evennia
# we only want to run coverall once, so we only do it for the designated matrix combination(s)
# it's also not critical if pushing to either service fails (happens for PRs since env is not
# available outside of the evennia org)