Fix type hinting

Author: Lode Rosseel

pytest_django/fixtures.py

@@ -205,14 +205,14 @@ def django_db_setup(
 
 
 def _build_pytest_django_test_case(
-    test_case_class,
+    test_case_class: type[django.test.TestCase],
     *,
     reset_sequences: bool,
     serialized_rollback: bool,
-    databases,
-    available_apps,
+    databases: _DjangoDbDatabases,
+    available_apps: _DjangoDbAvailableApps,
     skip_django_testcase_class_setup: bool,
-):
+) -> type[django.test.TestCase]:
     # Build a custom TestCase subclass with configured attributes and optional
     # overrides to skip Django's TestCase class-level setup/teardown.
     import django.test  # local import to avoid hard dependency at import time
@@ -319,10 +319,7 @@ def _sync_django_db_helper(
     import pytest_asyncio
 except ImportError:
 
-    async def _async_django_db_helper(
-        request: pytest.FixtureRequest,
-        django_db_blocker: DjangoDbBlocker,
-    ) -> AsyncGenerator[None, None]:
+    async def _async_django_db_helper() -> AsyncGenerator[None, None]:
         raise RuntimeError(
             "The `pytest_asyncio` plugin is required to use the `async_django_db` fixture."
         )
@@ -406,9 +403,9 @@ def _get_django_db_settings(request: pytest.FixtureRequest) -> _DjangoDb:
 @pytest.fixture
 def _django_db_helper(
     request: pytest.FixtureRequest,
-    django_db_setup: None,
-    django_db_blocker: DjangoDbBlocker,
-):
+    django_db_setup: None,  # noqa: ARG001
+    django_db_blocker: DjangoDbBlocker,  # noqa: ARG001
+) -> None:
     asyncio_marker = request.node.get_closest_marker("asyncio")
     transactional, *_ = _get_django_db_settings(request)
     if transactional or not asyncio_marker:

pytest_django/plugin.py

@@ -844,7 +844,7 @@ def _blocking_wrapper(self, *args: Any, **kwargs: Any) -> NoReturn:  # noqa: ARG
             '"db" or "transactional_db" fixtures to enable it. '
         )
 
-    def _unblocked_async_only(self, wrapper_self: Any, *args, **kwargs):
+    def _unblocked_async_only(self, wrapper_self: Any, *args: Any, **kwargs: Any) -> None:
         __tracebackhide__ = True
         from asgiref.sync import SyncToAsync
 
@@ -860,23 +860,12 @@ def _unblocked_async_only(self, wrapper_self: Any, *args, **kwargs):
         if self._real_ensure_connection is not None:
             self._real_ensure_connection(wrapper_self, *args, **kwargs)
 
-    def _unblocked_sync_only(self, wrapper_self: Any, *args, **kwargs):
-        __tracebackhide__ = True
-        if threading.current_thread() != threading.main_thread():
-            raise RuntimeError(
-                "Database access is only allowed in the main thread, "
-                "modify your test fixtures to be sync or use the transactional_db fixture."
-                "See https://pytest-django.readthedocs.io/en/latest/database.html#db-thread-safeguards for more information."
-            )
-        if self._real_ensure_connection is not None:
-            self._real_ensure_connection(wrapper_self, *args, **kwargs)
-
-    def unblock(self, async_only=False) -> AbstractContextManager[None]:
+    def unblock(self, async_only: bool = False) -> AbstractContextManager[None]:
         """Enable access to the Django database."""
         self._save_active_wrapper()
         if async_only:
 
-            def _method(wrapper_self, *args, **kwargs):
+            def _method(wrapper_self: Any, *args: Any, **kwargs: Any) -> None:
                 return self._unblocked_async_only(wrapper_self, *args, **kwargs)
 
             self._dj_db_wrapper.ensure_connection = _method