fail if asyncio test requests async pytest fixture in strict mode

Author: jakkdl

pytest_asyncio/plugin.py

@@ -880,7 +880,20 @@ def pytest_pyfunc_call(pyfuncitem: Function) -> object | None:
     """
     if pyfuncitem.get_closest_marker("asyncio") is not None:
         if isinstance(pyfuncitem, PytestAsyncioFunction):
-            pass
+            asyncio_mode = _get_asyncio_mode(pyfuncitem.config)
+            for fixtures in pyfuncitem._fixtureinfo.name2fixturedefs.values():
+                # name2fixturedefs is a dict between fixture name and a list of matching
+                # fixturedefs. The last entry in the list is closest and the one used.
+                func = fixtures[-1].func
+                if (
+                    _is_coroutine_or_asyncgen(func)
+                    and not _is_asyncio_fixture_function(func)
+                    and asyncio_mode == Mode.STRICT
+                ):
+                    pytest.fail(
+                        "asyncio test requested async fixture not marked asyncio, "
+                        "in strict mode. You might want to use @pytest_asyncio.fixture"
+                    )
         else:
             pyfuncitem.warn(
                 pytest.PytestWarning(

tests/modes/test_strict_mode.py

@@ -124,3 +124,31 @@ async def test_anything(any_fixture):
             "*coroutine 'any_fixture' was never awaited*",
         ],
     )
+
+
+def test_strict_mode_marked_test_errors_unmarked_fixture(pytester: Pytester):
+    pytester.makeini("[pytest]\nasyncio_default_fixture_loop_scope = function")
+    pytester.makepyfile(
+        dedent(
+            """\
+        import pytest
+
+        # Not using pytest_asyncio.fixture
+        @pytest.fixture()
+        async def any_fixture():
+            raise RuntimeError()
+
+        @pytest.mark.asyncio
+        async def test_anything(any_fixture):
+            pass
+        """
+        )
+    )
+    result = pytester.runpytest_subprocess("--asyncio-mode=strict", "-W default")
+    result.assert_outcomes(failed=1, skipped=0, warnings=1)
+    result.stdout.fnmatch_lines(
+        [
+            "*asyncio test requested async fixture not marked asyncio in strict mode*",
+            "*coroutine 'any_fixture' was never awaited*",
+        ],
+    )