From 5624174c6d4781cf467b191e2048ee28f35ef1a5 Mon Sep 17 00:00:00 2001 From: Simone Locci Date: Fri, 29 Aug 2025 00:36:07 +0200 Subject: [PATCH 1/2] Fix assert --- tests/services/test_user_service.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/services/test_user_service.py b/tests/services/test_user_service.py index 36c4afc..a6e1bba 100644 --- a/tests/services/test_user_service.py +++ b/tests/services/test_user_service.py @@ -16,4 +16,4 @@ async def test_save_user(): async def test_get_all_users_empty(): users = await get_all_users() assert isinstance(users, list) - assert len(users) > 0 + assert len(users) == 0 From e69c609ea7c721f230c5a09cd9f3f15826e88c6f Mon Sep 17 00:00:00 2001 From: Simone Locci Date: Fri, 29 Aug 2025 00:40:11 +0200 Subject: [PATCH 2/2] Fix test --- app/services/user_service.py | 7 +++++++ tests/conftest.py | 7 +++++++ 2 files changed, 14 insertions(+) create mode 100644 tests/conftest.py diff --git a/app/services/user_service.py b/app/services/user_service.py index ee19239..e3cad92 100644 --- a/app/services/user_service.py +++ b/app/services/user_service.py @@ -19,3 +19,10 @@ async def save_user(user: UserCreate) -> User: async def get_all_users() -> list[User]: log.debug("Reading all users") return _fake_db + + +# Clears the in-memory user store. +# Used only in tests to ensure isolation between runs. +def _reset_users(): + global _fake_db + _fake_db = [] diff --git a/tests/conftest.py b/tests/conftest.py new file mode 100644 index 0000000..1f6d488 --- /dev/null +++ b/tests/conftest.py @@ -0,0 +1,7 @@ +import pytest +from app.services.user_service import _reset_users + + +@pytest.fixture(autouse=True) +def clean_fake_db(): + _reset_users()