Skip to content

Commit

Permalink
Fix py37 coverage (#970)
Browse files Browse the repository at this point in the history
* Tidy up coverage configuration

* Skip coverage report for py37

* Revert "Skip coverage report for py37"

This reverts commit cf040f8.

* Lower coverage threshold

* Restore HTML report to aid troubleshooting
  • Loading branch information
bhrutledge committed Jan 4, 2023
1 parent 6659ce9 commit 0fda8da
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
1 change: 1 addition & 0 deletions .coveragerc
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
[run]
branch = True
dynamic_context = test_function

[report]
# Regexes for lines to exclude from consideration
Expand Down
6 changes: 4 additions & 2 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,10 @@ passenv =
PYTEST_ADDOPTS
commands =
python -m coverage run -m pytest {posargs}
python -m coverage report -m --fail-under 97
python -m coverage html --show-contexts
python -m coverage html
# HACK: The threshold could be 97, but py37 reports less
# See https://github.com/nedbat/coveragepy/issues/1524
python -m coverage report --skip-covered --show-missing --fail-under 96

[testenv:integration]
deps =
Expand Down

0 comments on commit 0fda8da

Please sign in to comment.