Refactor and extend skipif.

Author: tobiasraabe

tests/conftest.py

@@ -79,3 +79,10 @@ def pytest_collection_modifyitems(session, config, items) -> None:  # noqa: ARG0
     for item in items:
         if isinstance(item, NotebookItem):
             item.add_marker(pytest.mark.xfail(reason="The tests are flaky."))
+
+
+skip_if_deadlock = pytest.mark.skipif(
+    (sys.version_info[:2] in [(3, 12), (3, 13)] and sys.platform == "win32")
+    or (sys.version_info[:2] == (3, 13) and sys.platform == "linux"),
+    reason="Deadlock in loky/backend/resource_tracker.py, line 181, maybe related to https://github.com/joblib/loky/pull/450",
+)

tests/test_capture.py

@@ -1,11 +1,11 @@
-import sys
 import textwrap
 
 import pytest
 from pytask import ExitCode
 from pytask import cli
 
 from pytask_parallel import ParallelBackend
+from tests.conftest import skip_if_deadlock
 
 
 @pytest.mark.parametrize(
@@ -17,14 +17,7 @@
                 reason="dask cannot handle dynamically imported modules."
             ),
         ),
-        pytest.param(
-            ParallelBackend.LOKY,
-            marks=pytest.mark.skipif(
-                (sys.version_info[:2] == (3, 12) and sys.platform == "win32")
-                or (sys.version_info[:2] == (3, 13) and sys.platform == "linux"),
-                reason="Deadlock in loky/backend/resource_tracker.py, line 181, maybe related to https://github.com/joblib/loky/pull/450",  # noqa: E501
-            ),
-        ),
+        pytest.param(ParallelBackend.LOKY, marks=skip_if_deadlock),
         ParallelBackend.PROCESSES,
     ],
 )

tests/test_config.py

@@ -1,14 +1,14 @@
 from __future__ import annotations
 
 import os
-import sys
 import textwrap
 
 import pytest
 from pytask import ExitCode
 from pytask import build
 
 from pytask_parallel import ParallelBackend
+from tests.conftest import skip_if_deadlock
 
 
 @pytest.mark.parametrize(
@@ -37,11 +37,7 @@ def test_interplay_between_debugging_and_parallel(tmp_path, pdb, n_workers, expe
             "parallel_backend",
             ParallelBackend.LOKY,
             ExitCode.OK,
-            marks=pytest.mark.skipif(
-                (sys.version_info[:2] == (3, 12) and sys.platform == "win32")
-                or (sys.version_info[:2] == (3, 13) and sys.platform == "linux"),
-                reason="Deadlock in loky/backend/resource_tracker.py, line 181, maybe related to https://github.com/joblib/loky/pull/450",  # noqa: E501
-            ),
+            marks=skip_if_deadlock,
         ),
         ("parallel_backend", ParallelBackend.PROCESSES, ExitCode.OK),
         ("parallel_backend", ParallelBackend.THREADS, ExitCode.OK),

tests/test_execute.py

@@ -1,6 +1,5 @@
 from __future__ import annotations
 
-import sys
 import textwrap
 from time import time
 
@@ -12,6 +11,7 @@
 from pytask_parallel import ParallelBackend
 from pytask_parallel.execute import _Sleeper
 from tests.conftest import restore_sys_path_and_module_after_test_execution
+from tests.conftest import skip_if_deadlock
 
 _IMPLEMENTED_BACKENDS = [
     pytest.param(
@@ -20,14 +20,7 @@
             reason="dask cannot handle dynamically imported modules."
         ),
     ),
-    pytest.param(
-        ParallelBackend.LOKY,
-        marks=pytest.mark.skipif(
-            (sys.version_info[:2] == (3, 12) and sys.platform == "win32")
-            or (sys.version_info[:2] == (3, 13) and sys.platform == "linux"),
-            reason="Deadlock in loky/backend/resource_tracker.py, line 181, maybe related to https://github.com/joblib/loky/pull/450",  # noqa: E501
-        ),
-    ),
+    pytest.param(ParallelBackend.LOKY, marks=skip_if_deadlock),
     ParallelBackend.PROCESSES,
     ParallelBackend.THREADS,
 ]

tests/test_remote.py

@@ -1,16 +1,13 @@
 import pickle
-import sys
 import textwrap
 
 import pytest
 from pytask import ExitCode
 from pytask import cli
 
-pytestmark = pytest.mark.skipif(
-    (sys.version_info[:2] == (3, 12) and sys.platform == "win32")
-    or (sys.version_info[:2] == (3, 13) and sys.platform == "linux"),
-    reason="Deadlock in loky/backend/resource_tracker.py, line 181, maybe related to https://github.com/joblib/loky/pull/450",
-)
+from tests.conftest import skip_if_deadlock
+
+pytestmark = skip_if_deadlock
 
 
 @pytest.fixture(autouse=True)