A different (better) way to avoid warnings when no code has been run.

1 files changed, 5 insertions, 16 deletions

diff --git a/coverage/control.py b/coverage/control.py
index 6fc1b98..514f23d 100644
--- a/coverage/control.py
+++ b/coverage/control.py
@@ -172,10 +172,6 @@ class coverage(object):
 
         # Only _harvest_data once per measurement cycle.
         self._harvested = False
-        
-        # When stop() is called, we can tell it that in fact no product code
-        # was run, to make the warnings more reasonable.
-        self._never_run = False
 
         # Set the reporting precision.
         Numbers.set_precision(self.config.precision)
@@ -365,14 +361,8 @@ class coverage(object):
         self._harvested = False
         self.collector.start()
 
-    def stop(self, never_run=False):
-        """Stop measuring code coverage.
-        
-        Set `never_run` to True to indicate that no product code was run, so
-        we don't warn unnecessarily.
-
-        """
-        self._never_run = never_run
+    def stop(self):
+        """Stop measuring code coverage."""
         self.collector.stop()
         self._harvest_data()
 
@@ -454,10 +444,9 @@ class coverage(object):
                 self._warn("Module %s was never imported." % pkg)
 
             # Find out if we got any data.
-            if not self._never_run:
-                summary = self.data.summary()
-                if not summary:
-                    self._warn("No data was collected.")
+            summary = self.data.summary()
+            if not summary:
+                self._warn("No data was collected.")
 
             # Find files that were never executed at all.
             for src in self.source: