Skip to content

Commit

Permalink
Require 100% test coverage in certain modules
Browse files Browse the repository at this point in the history
Right now, I have enabled the check for 100% coverage in a few
modules that already have it.  Going forward, we plan to also use
this check for new modules.  When we introduce new modules that
should have 100% coverage, we will have to add their paths to
`tox.ini` using globs (e.g.,
`circuit_knitting_toolbox/circuit_cutting/utils/**/*`), following
https://coverage.readthedocs.io/en/7.2.3/source.html#file-patterns.

The old coverage check, of all modules, follows in the coverage
workflow, so that we can report the correct number to coveralls.

Closes #107.
  • Loading branch information
garrison committed Apr 12, 2023
1 parent cdaf073 commit 16e519c
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ commands =
coverage3 run --source circuit_knitting_toolbox --parallel-mode -m pytest test/ {posargs}
coverage3 combine
coverage3 html
coverage3 report --fail-under=100 --show-missing --include="circuit_knitting_toolbox/circuit_cutting/wire_cutting/wire_cutting_verification.py,circuit_knitting_toolbox/utils/integral_driver.py"
coverage3 report --fail-under=80

[testenv:docs]
Expand Down

0 comments on commit 16e519c

Please sign in to comment.