Fix unit tests for base resolvers

- Add pytest-asyncio default fixture loop scope configuration
- Extract implementation methods from Strawberry fields for testability
- Fix Task creation test to include required patient_id
- Tests now call implementation methods directly instead of Strawberry fields

Author: felixevers

backend/api/resolvers/base.py

@@ -18,16 +18,22 @@ def __init__(self, model: type[ModelType]):
     def get_repository(self, db: AsyncSession) -> BaseRepository[ModelType]:
         return BaseRepository(db, self.model)
 
-    @strawberry.field
-    async def get_by_id(self, info: Info, id: strawberry.ID) -> ModelType | None:
+    async def _get_by_id_impl(self, info: Info, id: strawberry.ID) -> ModelType | None:
         repo = self.get_repository(info.context.db)
         return await repo.get_by_id(id)
 
-    @strawberry.field
-    async def get_all(self, info: Info) -> list[ModelType]:
+    async def _get_all_impl(self, info: Info) -> list[ModelType]:
         repo = self.get_repository(info.context.db)
         return await repo.get_all()
 
+    @strawberry.field
+    async def get_by_id(self, info: Info, id: strawberry.ID) -> ModelType | None:
+        return await self._get_by_id_impl(info, id)
+
+    @strawberry.field
+    async def get_all(self, info: Info) -> list[ModelType]:
+        return await self._get_all_impl(info)
+
 
 class BaseMutationResolver(Generic[ModelType]):
     def __init__(self, model: type[ModelType], entity_name: str):

backend/pytest.ini

@@ -1,5 +1,6 @@
 [pytest]
 asyncio_mode = auto
+asyncio_default_fixture_loop_scope = function
 testpaths = tests
 python_files = test_*.py
 python_classes = Test*

backend/tests/unit/test_base_resolvers.py

@@ -13,7 +13,7 @@ def __init__(self, db):
 async def test_base_query_resolver_get_by_id(db_session, sample_task):
     resolver = BaseQueryResolver(Task)
     info = MockInfo(db_session)
-    result = await resolver.get_by_id(info, sample_task.id)
+    result = await resolver._get_by_id_impl(info, sample_task.id)
     assert result is not None
     assert result.id == sample_task.id
 
@@ -22,19 +22,20 @@ async def test_base_query_resolver_get_by_id(db_session, sample_task):
 async def test_base_query_resolver_get_all(db_session, sample_task):
     resolver = BaseQueryResolver(Task)
     info = MockInfo(db_session)
-    results = await resolver.get_all(info)
+    results = await resolver._get_all_impl(info)
     assert len(results) >= 1
     assert any(t.id == sample_task.id for t in results)
 
 
 @pytest.mark.asyncio
-async def test_base_mutation_resolver_create_and_notify(db_session):
+async def test_base_mutation_resolver_create_and_notify(db_session, sample_patient):
     resolver = BaseMutationResolver(Task, "task")
     info = MockInfo(db_session)
-    new_task = Task(title="New Task", description="Description")
+    new_task = Task(title="New Task", description="Description", patient_id=sample_patient.id)
     result = await resolver.create_and_notify(info, new_task)
     assert result.id is not None
     assert result.title == "New Task"
+    assert result.patient_id == sample_patient.id
 
 
 @pytest.mark.asyncio