Implement code comments

Author: Lode Rosseel

pytest_django/fixtures.py

@@ -3,7 +3,7 @@
 from __future__ import annotations
 
 import os
-from collections.abc import AsyncGenerator, Generator, Iterable, Sequence
+from collections.abc import Generator, Iterable, Sequence
 from contextlib import AbstractContextManager, contextmanager
 from functools import partial
 from typing import TYPE_CHECKING, Protocol
@@ -16,6 +16,7 @@
 
 
 if TYPE_CHECKING:
+    from collections.abc import AsyncGenerator
     from typing import Any, Callable, Literal, Optional, Union
 
     import django

tests/test_async_db.py

@@ -22,7 +22,7 @@
 @pytest.mark.parametrize("run_number", [1, 2])
 @pytestmark
 @pytest.mark.django_db
-async def test_async_db(db, run_number) -> None:
+async def test_async_db(run_number: int) -> None:  # noqa: ARG001
     # test async database usage remains isolated between tests
 
     assert await Item.objects.acount() == 0
@@ -32,18 +32,19 @@ async def test_async_db(db, run_number) -> None:
 
 
 @fixturemark
-async def db_item(db) -> Any:
+async def db_item() -> Any:
     return await Item.objects.acreate(name="async")
 
 
 @pytest.fixture
-def sync_db_item(db) -> Any:
+def sync_db_item() -> Any:
     return Item.objects.create(name="sync")
 
 
 @pytestmark
 @pytest.mark.xfail(strict=True, reason="Sync fixture used in async test")
-async def test_db_item(db_item: Item, sync_db_item) -> None:
+@pytest.mark.usefixtures("db_item", "sync_db_item")
+async def test_db_item() -> None:
     pass