fix: Remove pytest_generate_tests.

This also fixes an issue that could cause the warning for the deprecated *scope* argument to the asyncio marker to be reported multiple times.

Author: seifertm

changelog.d/+b22c903a.fixed.rst

@@ -0,0 +1 @@
+An error that could cause duplicate warnings to be issued

pytest_asyncio/plugin.py

@@ -42,7 +42,6 @@
     Function,
     Item,
     Mark,
-    Metafunc,
     MonkeyPatch,
     Parser,
     PytestCollectionWarning,
@@ -547,32 +546,6 @@ def _temporary_event_loop_policy(policy: AbstractEventLoopPolicy) -> Iterator[No
         _set_event_loop(old_loop)
 
 
-@pytest.hookimpl(tryfirst=True)
-def pytest_generate_tests(metafunc: Metafunc) -> None:
-    marker = metafunc.definition.get_closest_marker("asyncio")
-    if not marker:
-        return
-    default_loop_scope = _get_default_test_loop_scope(metafunc.config)
-    loop_scope = _get_marked_loop_scope(marker, default_loop_scope)
-    runner_fixture_id = f"_{loop_scope}_scoped_runner"
-    # This specific fixture name may already be in metafunc.argnames, if this
-    # test indirectly depends on the fixture. For example, this is the case
-    # when the test depends on an async fixture, both of which share the same
-    # event loop fixture mark.
-    if runner_fixture_id in metafunc.fixturenames:
-        return
-    fixturemanager = metafunc.config.pluginmanager.get_plugin("funcmanage")
-    assert fixturemanager is not None
-    # Add the scoped event loop fixture to Metafunc's list of fixture names and
-    # fixturedefs and leave the actual parametrization to pytest
-    # The fixture needs to be appended to avoid messing up the fixture evaluation
-    # order
-    metafunc.fixturenames.append(runner_fixture_id)
-    metafunc._arg2fixturedefs[runner_fixture_id] = fixturemanager._arg2fixturedefs[
-        runner_fixture_id
-    ]
-
-
 def _get_event_loop_policy() -> AbstractEventLoopPolicy:
     with warnings.catch_warnings():
         warnings.simplefilter("ignore", DeprecationWarning)

tests/markers/test_function_scope.py

@@ -88,7 +88,7 @@ async def test_warns():
         )
     )
     result = pytester.runpytest_subprocess("--asyncio-mode=strict")
-    result.assert_outcomes(passed=1, warnings=2)
+    result.assert_outcomes(passed=1, warnings=1)
     result.stdout.fnmatch_lines("*DeprecationWarning*")