fix: suffix=False will suppress the suffix even with multiprocessing. #…

…989

CHANGES.rst

@@ -37,13 +37,18 @@ Unreleased
 - API: The exceptions raised by Coverage.py have been specialized, to provide
   finer-grained catching of exceptions by third-party code.
 
+- API: Using ``suffix=False`` when constructing a Coverage object with
+  multiprocessing wouldn't suppress the data file suffix (`issue 989`_).  This
+  is now fixed.
+
 - Debug: The `coverage debug data` command will now sniff out combinable data
   files, and report on all of them.
 
 - Debug: The `coverage debug` command used to accept a number of topics at a
   time, and show all of them, though this was never documented.  This no longer
   works, to allow for command-line options in the future.
 
+.. _issue 989: https://github.com/nedbat/coveragepy/issues/989
 .. _issue 1203: https://github.com/nedbat/coveragepy/issues/1203
 
 

coverage/control.py

@@ -482,10 +482,15 @@ def _init_for_start(self):
             )
 
         suffix = self._data_suffix_specified
-        if suffix or self.config.parallel:
+        if suffix:
             if not isinstance(suffix, str):
                 # if data_suffix=True, use .machinename.pid.random
                 suffix = True
+        elif self.config.parallel:
+            if suffix is None:
+                suffix = True
+            elif not isinstance(suffix, str):
+                suffix = bool(suffix)
         else:
             suffix = None
 

tests/test_api.py

@@ -1199,3 +1199,23 @@ def test_combine_relative(self):
         assert files == {'foo.py', 'bar.py', os_sep('modsrc/__init__.py')}
         res = cov.report()
         assert res == 100
+
+    def test_combine_no_suffix_multiprocessing(self):
+        self.make_file(".coveragerc", """\
+            [run]
+            branch = True
+            """)
+        cov = coverage.Coverage(
+            config_file=".coveragerc",
+            concurrency="multiprocessing",
+            data_suffix=False,
+            )
+        cov.start()
+        cov.stop()
+        # The warning isn't the point of this test, but suppress it.
+        with pytest.warns(Warning) as warns:
+            cov.combine()
+        assert_coverage_warnings(warns, "No data was collected. (no-data-collected)")
+        cov.save()
+        self.assert_file_count(".coverage.*", 0)
+        self.assert_exists(".coverage")