refactor: Removed event_loop_fixture_id from cache key of async fixtures.

Author: seifertm

pytest_asyncio/plugin.py

@@ -238,8 +238,8 @@ def _preprocess_async_fixtures(
     for fixtures in fixturemanager._arg2fixturedefs.values():
         for fixturedef in fixtures:
             func = fixturedef.func
-            if not _is_coroutine_or_asyncgen(func) and not getattr(
-                func, "_async_fixture", False
+            if fixturedef in processed_fixturedefs or not _is_coroutine_or_asyncgen(
+                func
             ):
                 continue
             if not _is_asyncio_fixture_function(func) and asyncio_mode == Mode.STRICT:
@@ -254,17 +254,12 @@ def _preprocess_async_fixtures(
             if scope == "function":
                 event_loop_fixture_id: Optional[str] = "event_loop"
             else:
-                try:
-                    event_loop_node = _retrieve_scope_root(collector, scope)
-                except Exception:
-                    continue
+                event_loop_node = _retrieve_scope_root(collector, scope)
                 event_loop_fixture_id = event_loop_node.stash.get(
                     # Type ignored because of non-optimal mypy inference.
                     _event_loop_fixture_id,  # type: ignore[arg-type]
                     None,
                 )
-            if (fixturedef, event_loop_fixture_id) in processed_fixturedefs:
-                continue
             _make_asyncio_fixture_function(func, scope)
             function_signature = inspect.signature(func)
             if "event_loop" in function_signature.parameters:
@@ -286,7 +281,7 @@ def _preprocess_async_fixtures(
                 fixturedef.argnames += ("event_loop",)
             _synchronize_async_fixture(fixturedef)
             assert _is_asyncio_fixture_function(fixturedef.func)
-            processed_fixturedefs.add((fixturedef, event_loop_fixture_id))
+            processed_fixturedefs.add(fixturedef)
 
 
 def _synchronize_async_fixture(fixturedef: FixtureDef) -> None: