diff options
| author | Ned Batchelder <ned@nedbatchelder.com> | 2021-05-31 19:37:53 -0400 |
|---|---|---|
| committer | Ned Batchelder <ned@nedbatchelder.com> | 2021-05-31 19:37:53 -0400 |
| commit | 1157999dc5c6c3aa3129a7cd4bf249fe73598501 (patch) | |
| tree | cba407bacb180fb8fcb46a1b5eaf7da2e0fd88fb /coverage/cmdline.py | |
| parent | 65d9e0edbca0dc893ca70fbf64969e2ddc5a3680 (diff) | |
| download | python-coveragepy-git-1157999dc5c6c3aa3129a7cd4bf249fe73598501.tar.gz | |
fix: --fail-under=100 could report 100 is less than 100.
Use the same rounding rules for the fail-under message that are used for totals
everywhere else, so that it won't say:
total of 100 is less than fail-under=100
Diffstat (limited to 'coverage/cmdline.py')
| -rw-r--r-- | coverage/cmdline.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/coverage/cmdline.py b/coverage/cmdline.py index d1e8f283..697d2960 100644 --- a/coverage/cmdline.py +++ b/coverage/cmdline.py @@ -20,7 +20,7 @@ from coverage.data import line_counts from coverage.debug import info_formatter, info_header, short_stack from coverage.exceptions import BaseCoverageException, ExceptionDuringRun, NoSource from coverage.execfile import PyRunner -from coverage.results import should_fail_under +from coverage.results import Numbers, should_fail_under class Opts: @@ -655,8 +655,8 @@ class CoverageScript: fail_under = self.coverage.get_option("report:fail_under") precision = self.coverage.get_option("report:precision") if should_fail_under(total, fail_under, precision): - msg = "total of {total:.{p}f} is less than fail-under={fail_under:.{p}f}".format( - total=total, fail_under=fail_under, p=precision, + msg = "total of {total} is less than fail-under={fail_under:.{p}f}".format( + total=Numbers.display_covered(total), fail_under=fail_under, p=precision, ) print("Coverage failure:", msg) return FAIL_UNDER |