Fix patching in pytest setup phase

- relevant for module and session-scoped fs fixtures

Author: mrbean-bremen

.github/workflows/testsuite.yml

@@ -134,6 +134,7 @@ jobs:
         run: |
           echo "$(python -m pytest pyfakefs/pytest_tests/pytest_plugin_failing_helper.py)" > ./testresult.txt
           pytest pyfakefs/pytest_tests
+          pytest pyfakefs/pytest_session_tests
           cd pyfakefs/pytest_tests/ns_package
           pytest --log-cli-level=INFO test
         shell: bash

CHANGES.md

@@ -25,7 +25,9 @@ The released versions correspond to PyPI releases.
 * fixed a regression that could break tests under Posix in Python 3.12
   (see [#1126](../../issues/1126))
 * fixed behavior for `os.access` for symlinks under Windows
-* fixes permission problem on querying file properties (see [#1122](../../issues/1122))
+* fixed permission problem on querying file properties (see [#1122](../../issues/1122))
+* fixed patching in pytest setup phase for module and session-scoped fs fixtures
+  (see [#1126](../../issues/1126))
 
 ### Documentation
 * use a theme for documentation supporting dark mode

pyfakefs/pytest_plugin.py

@@ -92,7 +92,7 @@ def pytest_runtest_logreport(report):
 
 
 @pytest.hookimpl(hookwrapper=True, trylast=True)
-def pytest_runtest_call(item):
+def pytest_runtest_setup(item):
     if Patcher.PATCHER is not None:
         Patcher.PATCHER.resume()
     yield

pyfakefs/pytest_session_tests/__init__.py

@@ -0,0 +1,61 @@
+from pathlib import Path
+
+import pytest
+
+from pyfakefs.fake_filesystem import FakeFilesystem
+
+
+def pytest_generate_tests(metafunc):
+    handlers = [a, b, c]
+    if "handler_class" in metafunc.fixturenames:
+        metafunc.parametrize("handler_class", handlers)
+
+
+def a():
+    pass
+
+
+def b():
+    pass
+
+
+def c():
+    pass
+
+
+@pytest.fixture
+def class_a():
+    pass
+
+
+@pytest.fixture
+def class_b():
+    pass
+
+
+@pytest.fixture
+def class_c():
+    pass
+
+
+@pytest.fixture
+def make_handler(request):
+    def _make_handler(cls):
+        return request.getfixturevalue(f"class_{cls.__name__}")
+
+    yield _make_handler
+
+
+@pytest.fixture
+def handler_and_check(handler_class, make_handler):
+    assert Path("/foo/bar").exists()
+    yield
+
+
+def test_handler_and_check_in_fixture(handler_and_check):
+    assert Path("/foo/bar").exists()
+
+
+@pytest.fixture(scope="session", autouse=True)
+def config(fs_session: FakeFilesystem):
+    fs_session.create_file("/foo/bar")