refactor test

Author: solababs

backend/tests/unit/git/conftest.py

@@ -1371,3 +1371,11 @@ async def mock_create_branch_git_repo_01(db: InfrahubDatabase, default_branch: B
 @pytest.fixture
 async def mock_create_branch_git_repo_03(db: InfrahubDatabase, default_branch: Branch) -> None:
     await create_branch(branch_name="branch01", db=db)
+
+
+def git_user_config():
+    config.SETTINGS.git.user_email = "[email protected]"
+    config.SETTINGS.git.user_name = "Test User"
+    yield
+    config.SETTINGS.git.user_email = None
+    config.SETTINGS.git.user_name = None

backend/tests/unit/git/test_git_repository.py

@@ -12,7 +12,6 @@
 from infrahub_sdk.uuidt import UUIDT
 from pytest_httpx._httpx_mock import HTTPXMock
 
-from infrahub import config
 from infrahub.core.branch import Branch
 from infrahub.core.constants import InfrahubKind
 from infrahub.core.registry import registry
@@ -1107,10 +1106,7 @@ async def test_get_filtered_remote_branches__no_import_sync_branch_names(git_rep
     assert sorted(filtered_remote_branches.keys()) == ["branch01", "branch02", "clean-branch", "main"]
 
 
-async def test_new_repo_has_config(git_upstream_repo_01: dict[str, str | Path], git_repos_dir: Path):
-    config.SETTINGS.git.user_email = "[email protected]"
-    config.SETTINGS.git.user_name = "Test User"
-
+async def test_new_repo_has_config(git_upstream_repo_01: dict[str, str | Path], git_repos_dir: Path, git_user_config):
     repo = await InfrahubRepository.new(
         id=UUIDT.new(),
         name=git_upstream_repo_01["name"],
@@ -1127,6 +1123,3 @@ async def test_new_repo_has_config(git_upstream_repo_01: dict[str, str | Path],
     with repo.get_git_repo_main().config_reader() as git_config:
         assert git_config.get_value("user", "name") == "Test User"
         assert git_config.get_value("user", "email") == "[email protected]"
-
-    config.SETTINGS.git.user_email = None
-    config.SETTINGS.git.user_name = None