From 0b907fe3fb2eececbec16dccc17aeabb9c64dc15 Mon Sep 17 00:00:00 2001 From: Ned Batchelder Date: Sat, 15 Jan 2022 17:07:44 -0500 Subject: test: minor cleanup from start_method fixture work We only had to use remove_files because try_multiprocessing_code used to be called twice in one test. Now that it's only called once, we don't have to clean out stale data files. --- tests/test_concurrency.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'tests') diff --git a/tests/test_concurrency.py b/tests/test_concurrency.py index 35f0e384..001455f2 100644 --- a/tests/test_concurrency.py +++ b/tests/test_concurrency.py @@ -22,7 +22,6 @@ from coverage.files import abs_file from coverage.misc import import_local_file from tests.coveragetest import CoverageTest -from tests.helpers import remove_files # These libraries aren't always available, we'll skip tests if they aren't. @@ -443,6 +442,7 @@ def start_method_fixture(request): """Parameterized fixture to choose the start_method for multiprocessing.""" start_method = request.param if start_method not in multiprocessing.get_all_start_methods(): + # Windows doesn't support "fork". pytest.skip(f"start_method={start_method} not supported here") return start_method @@ -470,7 +470,6 @@ class MultiprocessingTest(CoverageTest): source = . """) - remove_files(".coverage", ".coverage.*") cmd = "coverage run {args} multi.py {start_method}".format( args=args, start_method=start_method, ) -- cgit v1.2.1