From 4f0133b8c305fec379c010495a29622369622aef Mon Sep 17 00:00:00 2001
From: Alexandre Teixeira <111787685+alteixeira20@users.noreply.github.com>
Date: Fri, 5 Jun 2026 11:10:41 +0100
Subject: [PATCH] refactor(tests): reuse import-state helper in auth tests

Test-only refactor continuing #2523. Replaces a repeated core.auth cache eviction pattern in three auth tests with the shared clear_module helper, preserving behavior.
---
 tests/test_auth_session_revocation.py        | 6 +++---
 tests/test_delete_user_revokes_api_tokens.py | 6 +++---
 tests/test_rename_user_case_insensitive.py   | 6 +++---
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/tests/test_auth_session_revocation.py b/tests/test_auth_session_revocation.py
index 3ec9d1a..e2f75c8 100644
--- a/tests/test_auth_session_revocation.py
+++ b/tests/test_auth_session_revocation.py
@@ -11,6 +11,8 @@ from unittest.mock import MagicMock
 import pytest
 from fastapi import HTTPException
 
+from tests.helpers.import_state import clear_module
+
 
 def _real_core_package():
     root = Path(__file__).resolve().parent.parent
@@ -20,9 +22,7 @@ def _real_core_package():
         core = types.ModuleType("core")
         sys.modules["core"] = core
     core.__path__ = [core_path]
-    if hasattr(core, "auth"):
-        delattr(core, "auth")
-    sys.modules.pop("core.auth", None)
+    clear_module("core.auth")
     return core
 
 
diff --git a/tests/test_delete_user_revokes_api_tokens.py b/tests/test_delete_user_revokes_api_tokens.py
index 3d646c7..dab753f 100644
--- a/tests/test_delete_user_revokes_api_tokens.py
+++ b/tests/test_delete_user_revokes_api_tokens.py
@@ -13,6 +13,8 @@ from pathlib import Path
 
 import pytest
 
+from tests.helpers.import_state import clear_module
+
 
 def _real_core_package():
     root = Path(__file__).resolve().parent.parent
@@ -22,9 +24,7 @@ def _real_core_package():
         core = types.ModuleType("core")
         sys.modules["core"] = core
     core.__path__ = [core_path]
-    if hasattr(core, "auth"):
-        delattr(core, "auth")
-    sys.modules.pop("core.auth", None)
+    clear_module("core.auth")
     return core
 
 
diff --git a/tests/test_rename_user_case_insensitive.py b/tests/test_rename_user_case_insensitive.py
index 624bc87..292085f 100644
--- a/tests/test_rename_user_case_insensitive.py
+++ b/tests/test_rename_user_case_insensitive.py
@@ -14,6 +14,8 @@ from unittest.mock import MagicMock
 
 import pytest
 
+from tests.helpers.import_state import clear_module
+
 
 def _real_core_package():
     root = Path(__file__).resolve().parent.parent
@@ -23,9 +25,7 @@ def _real_core_package():
         core = types.ModuleType("core")
         sys.modules["core"] = core
     core.__path__ = [core_path]
-    if hasattr(core, "auth"):
-        delattr(core, "auth")
-    sys.modules.pop("core.auth", None)
+    clear_module("core.auth")
     return core