Merge pull request #383 from biglocalnews/coverage-action

Add coverage to testing
biglocalnews · Jan 24, 2022 · 1ee79f2 · 1ee79f2
2 parents e605d7f + c8616c4
commit 1ee79f2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/.github/workflows/continuous-deployment.yml b/.github/workflows/continuous-deployment.yml
@@ -85,9 +85,13 @@ jobs:
           python-version: ${{ matrix.python }}
 
       - id: run
-        name: Run
+        name: Run tests
         run: make test
 
+      - id: coverage
+        name: Coverage report
+        run: make coverage
+
   test-build:
     name: Build Python package
     runs-on: ubuntu-latest

diff --git a/Makefile b/Makefile
@@ -92,11 +92,10 @@ mypy: ## run mypy type checks
 
 test: ## run all tests
 	$(call banner,       🤖 Running tests 🤖)
-	@$(PYTHON) setup.py -q test
+	@$(PIPENV) coverage run setup.py -q test
 
 
 coverage: ## check code coverage
-	@$(PIPENV) coverage run --source warn -m pytest
 	@$(PIPENV) coverage report -m
 
 #