diff options
author | loic@dachary.org <loic@dachary.org> | 2016-12-12 17:03:20 +0100 |
---|---|---|
committer | loic@dachary.org <loic@dachary.org> | 2016-12-12 17:03:20 +0100 |
commit | ec3eaf7c005de905a511451904a06a570816b91d (patch) | |
tree | d6afa4e3f291f7006142fe90cd9478c3219a1d42 /tests/test_summary.py | |
parent | 0ab9d30c8a1f26bd9a7481a9ee14e8a02284a0cd (diff) | |
download | python-coveragepy-git-ec3eaf7c005de905a511451904a06a570816b91d.tar.gz |
report must ignore skipped files #524
When files are 100% covered and --skipped-covered is specified, they
must not be taken into account to format the text report. It is
specially useful when a long filename is skipped.
--HG--
branch : issue-524-2
Diffstat (limited to 'tests/test_summary.py')
-rw-r--r-- | tests/test_summary.py | 29 |
1 files changed, 25 insertions, 4 deletions
diff --git a/tests/test_summary.py b/tests/test_summary.py index bda65681..f18d8d61 100644 --- a/tests/test_summary.py +++ b/tests/test_summary.py @@ -361,6 +361,27 @@ class SummaryTest(CoverageTest): squeezed = self.squeezed_lines(report) self.assertEqual(squeezed[3], "1 file skipped due to complete coverage.") + def test_report_skip_covered_longfilename(self): + self.make_file("long_______________filename.py", """ + def foo(): + pass + foo() + """) + out = self.run_command("coverage run --branch long_______________filename.py") + self.assertEqual(out, "") + report = self.report_from_command("coverage report --skip-covered") + + # Name Stmts Miss Branch BrPart Cover + # ----------------------------------------- + # + # 1 file skipped due to complete coverage. + + self.assertEqual(self.line_count(report), 4, report) + lines = self.report_lines(report) + self.assertEqual(lines[0], "Name Stmts Miss Branch BrPart Cover") + squeezed = self.squeezed_lines(report) + self.assertEqual(squeezed[3], "1 file skipped due to complete coverage.") + def test_report_skip_covered_no_data(self): report = self.report_from_command("coverage report --skip-covered") @@ -381,12 +402,12 @@ class SummaryTest(CoverageTest): self.make_file("mycode.py", "This isn't python at all!") report = self.report_from_command("coverage report mycode.py") + # mycode NotPython: Couldn't parse '...' as Python source: 'invalid syntax' at line 1 # Name Stmts Miss Cover # ---------------------------- - # mycode NotPython: Couldn't parse '...' as Python source: 'invalid syntax' at line 1 # No data to report. - last = self.squeezed_lines(report)[-2] + last = self.squeezed_lines(report)[0] # The actual file name varies run to run. last = re.sub(r"parse '.*mycode.py", "parse 'mycode.py", last) # The actual error message varies version to version @@ -405,12 +426,12 @@ class SummaryTest(CoverageTest): self.make_file(u"accented\xe2.py", "This isn't python at all!") report = self.report_from_command(u"coverage report accented\xe2.py") + # xxxx NotPython: Couldn't parse '...' as Python source: 'invalid syntax' at line 1 # Name Stmts Miss Cover # ---------------------------- - # xxxx NotPython: Couldn't parse '...' as Python source: 'invalid syntax' at line 1 # No data to report. - last = self.squeezed_lines(report)[-2] + last = self.squeezed_lines(report)[0] # The actual file name varies run to run. last = re.sub(r"parse '.*(accented.*?\.py)", r"parse '\1", last) # The actual error message varies version to version |