aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar MarkKoz <[email protected]>2021-02-04 18:13:01 -0800
committerGravatar MarkKoz <[email protected]>2021-02-04 18:13:32 -0800
commite12e403ad215efedce92ddcf1da00ebdc9c87116 (patch)
treee73911e7d256b8f2ad6898fede515fb630bd0cb5
parentMerge PR #92 - replace shell scripts with Python scripts (diff)
CI: fix test step not failing when tests fail
Generating the report in the same step resulted in the report exit code overriding the exit code of the test runner.
Diffstat (limited to '')
-rw-r--r--.github/workflows/lint-test-build-push.yaml9
1 files changed, 6 insertions, 3 deletions
diff --git a/.github/workflows/lint-test-build-push.yaml b/.github/workflows/lint-test-build-push.yaml
index 338e301..a76354b 100644
--- a/.github/workflows/lint-test-build-push.yaml
+++ b/.github/workflows/lint-test-build-push.yaml
@@ -120,12 +120,15 @@ jobs:
run: sudo swapoff -a
# Run unittests and generate coverage report in the container
- - name: Run unit tests and generate coverage report
+ - name: Run unit tests
id: run_tests
run: |
echo '::set-output name=started::true'
- cmd='coverage run -m unittest; coverage report -m'
- docker exec snekbox_dev /bin/bash -c "${cmd}"
+ docker exec snekbox_dev /bin/bash -c 'coverage run -m unittest'
+
+ - name: Generate coverage report
+ if: always() && steps.run_tests.outputs.started == 'true'
+ run: docker exec snekbox_dev /bin/bash -c 'coverage report -m'
# Set-up a Python version to process the coverage reports
# Note: This step runs even if the test step failed to make