diff options
| -rw-r--r-- | coverage/cmdline.py | 5 | ||||
| -rw-r--r-- | tests/test_process.py | 10 |
2 files changed, 12 insertions, 3 deletions
diff --git a/coverage/cmdline.py b/coverage/cmdline.py index d557ddf1..62edc207 100644 --- a/coverage/cmdline.py +++ b/coverage/cmdline.py @@ -629,7 +629,10 @@ class CoverageScript(object): fail_under = self.coverage.get_option("report:fail_under") precision = self.coverage.get_option("report:precision") if should_fail_under(total, fail_under, precision): - print("fail-under has failed") + msg = "total of {total:.{p}f} is less than fail-under={fail_under:.{p}f}".format( + total=total, fail_under=fail_under, p=precision, + ) + print("Coverage failure:", msg) return FAIL_UNDER return OK diff --git a/tests/test_process.py b/tests/test_process.py index 0428db75..8f8b009c 100644 --- a/tests/test_process.py +++ b/tests/test_process.py @@ -1236,13 +1236,19 @@ class FailUnderTest(CoverageTest): def test_report_43_is_not_ok(self): st, out = self.run_command_status("coverage report --fail-under=44") self.assertEqual(st, 2) - self.assertEqual(self.last_line_squeezed(out), "fail-under has failed") + self.assertEqual( + self.last_line_squeezed(out), + "Coverage failure: total of 43 is less than fail-under=44" + ) def test_report_42p86_is_not_ok(self): self.make_file(".coveragerc", "[report]\nprecision = 2") st, out = self.run_command_status("coverage report --fail-under=42.88") self.assertEqual(st, 2) - self.assertEqual(self.last_line_squeezed(out), "fail-under has failed") + self.assertEqual( + self.last_line_squeezed(out), + "Coverage failure: total of 42.86 is less than fail-under=42.88" + ) class FailUnderNoFilesTest(CoverageTest): |