Update skip_covered handling, add info when files are skipped. Tests.

1 files changed, 29 insertions, 4 deletions

diff --git a/tests/test_summary.py b/tests/test_summary.py
index 850f4dfd..15e948ab 100644
--- a/tests/test_summary.py
+++ b/tests/test_summary.py
@@ -259,14 +259,16 @@ class SummaryTest(CoverageTest):
         # Name             Stmts   Miss  Cover
         # ------------------------------------
         # not_covered.py       2      1    50%
+        #
+        # 1 file skipped because of 'skip covered' option.
 
-        self.assertEqual(self.line_count(report), 3, report)
+        self.assertEqual(self.line_count(report), 5, report)
         squeezed = self.squeezed_lines(report)
         self.assertEqual(squeezed[2], "not_covered.py 2 1 50%")
 
     def test_report_skip_covered_branches(self):
         self.make_file("main.py", """
-            import not_covered
+            import not_covered, covered
 
             def normal(z):
                 if z:
@@ -280,6 +282,11 @@ class SummaryTest(CoverageTest):
                     print("n")
             not_covered(True)
         """)
+        self.make_file("covered.py", """
+            def foo():
+                pass
+            foo()
+        """)
         out = self.run_command("coverage run --branch main.py")
         self.assertEqual(out, "n\nz\n")
         report = self.report_from_command("coverage report --skip-covered")
@@ -287,8 +294,10 @@ class SummaryTest(CoverageTest):
         # Name             Stmts   Miss Branch BrPart  Cover
         # --------------------------------------------------
         # not_covered.py       4      0      2      1    83%
+        #
+        # 2 files skipped because of 'skip covered' option.
 
-        self.assertEqual(self.line_count(report), 3, report)
+        self.assertEqual(self.line_count(report), 5, report)
         squeezed = self.squeezed_lines(report)
         self.assertEqual(squeezed[2], "not_covered.py 4 0 2 1 83%")
 
@@ -323,12 +332,28 @@ class SummaryTest(CoverageTest):
         # not_covered.py       4      0      2      1    83%
         # --------------------------------------------------
         # TOTAL                6      1      2      1    75%
+        #
+        # 1 file skipped because of 'skip covered' option.
 
-        self.assertEqual(self.line_count(report), 6, report)
+        self.assertEqual(self.line_count(report), 8, report)
         squeezed = self.squeezed_lines(report)
         self.assertEqual(squeezed[2], "also_not_run.py 2 1 0 0 50%")
         self.assertEqual(squeezed[3], "not_covered.py 4 0 2 1 83%")
         self.assertEqual(squeezed[5], "TOTAL 6 1 2 1 75%")
+        self.assertEqual(squeezed[7],
+                         "1 file skipped because of 'skip covered' option.")
+
+    def test_report_skip_covered_no_data(self):
+        self.make_file("main.py", "")
+        out = self.run_command("coverage run --branch main.py")
+        self.assertEqual(out, "")
+        report = self.report_from_command("coverage report --skip-covered")
+        # Name      Stmts   Miss Branch BrPart  Cover
+        # -------------------------------------------
+        # No data to report.
+        self.assertEqual(self.line_count(report), 3, report)
+        squeezed = self.squeezed_lines(report)
+        self.assertEqual(squeezed[2], "No data to report.")
 
     def test_dotpy_not_python(self):
         # We run a .py file, and when reporting, we can't parse it as Python.