Allow running test script with or without coverage

Author: duckontheweb

.github/workflows/continuous-integration.yml

@@ -97,11 +97,14 @@ jobs:
           pip install -e ".[validation]"
       
       - name: Execute test suite
-        # --fail-under=0 ensures we publish the coverage regardless of whether it meets
-        # the minimum so we can use Codecov to evaluate gaps
-        run: |
-          coverage run --source=pystac/ -m unittest discover tests/
-          coverage xml --fail-under=0
+        run: ./scripts/test
+        env:
+          CHECK_COVERAGE: true
+
+      - name: Prepare ./coverage.xml
+        # Ignore the configured fail-under to ensure we upload the coverage report. We
+        # will trigger a failure for coverage drops in a later job
+        run: coverage xml --fail-under 0
 
       - name: Upload All coverage to Codecov
         uses: codecov/codecov-action@v1
@@ -111,6 +114,11 @@ jobs:
           file: ./coverage.xml
           fail_ci_if_error: false 
 
+      - name: Check for coverage drop
+        # This will use the configured fail-under, causing this job to fail if the
+        # coverage drops.
+        run: coverage report
+
   lint:
     runs-on: ubuntu-latest
     strategy:

scripts/test

@@ -5,11 +5,15 @@ set -e
 if [[ -z ${CI} ]]; then
     pre-commit run --all-files
 fi
-
-echo
-echo " -- RUNNING UNIT TESTS --"
 echo
 
-# Test suite with coverage enabled
-coverage run -m unittest discover tests
-coverage xml
+if [[ -z ${CI} || -n ${CHECK_COVERAGE} ]]; then
+    echo " -- RUNNING UNIT TESTS (WITH COVERAGE) --"
+    # Test suite with coverage enabled
+    coverage run -m unittest discover tests
+else
+    echo " -- RUNNING UNIT TESTS (WITHOUT COVERAGE) --"
+    python -m unittest discover tests
+fi
+
+echo