🐛 Fix: Update fake API key generation to use _asdict() for row data

Author: pcrespov

services/api-server/tests/unit/_with_db/conftest.py

@@ -301,7 +301,9 @@ async def _generate_fake_api_key(n: PositiveInt):
 
             _generate_fake_api_key.row_ids.append(row.id)
 
-            yield ApiKeyInDB.model_validate({"api_secret": plain_api_secret, **row})
+            yield ApiKeyInDB.model_validate(
+                {"api_secret": plain_api_secret, **row._asdict()}
+            )
 
     _generate_fake_api_key.row_ids = []
     yield _generate_fake_api_key

services/api-server/tests/unit/_with_db/test_repository_api_keys.py

@@ -1,22 +1,36 @@
+# pylint: disable=redefined-outer-name
+# pylint: disable=unused-argument
+# pylint: disable=unused-variable
+# pylint: disable=too-many-arguments
+
 from collections.abc import AsyncGenerator, Callable
 
+import httpx
+import pytest
 from models_library.api_schemas_api_server.api_keys import ApiKeyInDB
 from pydantic import PositiveInt
 from simcore_service_api_server.repository.api_keys import ApiKeysRepository
 from sqlalchemy.ext.asyncio import AsyncEngine
 
 
-async def test_get_user_with_valid_credentials(
+@pytest.fixture
+def api_key_repo(
+    client: httpx.AsyncClient,  # ensures app context is available
     async_engine: AsyncEngine,
+) -> ApiKeysRepository:
+
+    return ApiKeysRepository(db_engine=async_engine)
+
+
+async def test_get_user_with_valid_credentials(
     create_fake_api_keys: Callable[[PositiveInt], AsyncGenerator[ApiKeyInDB, None]],
+    api_key_repo: ApiKeysRepository,
 ):
-    # Arrange
-    repo = ApiKeysRepository(db_engine=async_engine)
 
     # Generate a fake API key
     async for api_key_in_db in create_fake_api_keys(1):
         # Act
-        result = await repo.get_user(
+        result = await api_key_repo.get_user(
             api_key=api_key_in_db.api_key, api_secret=api_key_in_db.api_secret
         )
 
@@ -28,16 +42,14 @@ async def test_get_user_with_valid_credentials(
 
 
 async def test_get_user_with_invalid_credentials(
-    async_engine: AsyncEngine,
+    api_key_repo: ApiKeysRepository,
     create_fake_api_keys: Callable[[PositiveInt], AsyncGenerator[ApiKeyInDB, None]],
 ):
-    # Arrange
-    repo = ApiKeysRepository(db_engine=async_engine)
 
     # Generate a fake API key
     async for api_key_in_db in create_fake_api_keys(1):
         # Act - use wrong secret
-        result = await repo.get_user(
+        result = await api_key_repo.get_user(
             api_key=api_key_in_db.api_key, api_secret="wrong_secret"
         )