relink tests: Move some fixtures to conftest.py.

Author: samsrabin

tests/relink/conftest.py

@@ -0,0 +1,29 @@
+"""
+Shared fixtures for relink tests.
+"""
+
+import os
+import tempfile
+import shutil
+
+import pytest
+
+
+@pytest.fixture(scope="function", name="temp_dirs")
+def fixture_temp_dirs():
+    """Create temporary source and target directories for testing."""
+    source_dir = tempfile.mkdtemp(prefix="test_source_")
+    target_dir = tempfile.mkdtemp(prefix="test_target_")
+
+    yield source_dir, target_dir
+
+    # Cleanup
+    shutil.rmtree(source_dir, ignore_errors=True)
+    shutil.rmtree(target_dir, ignore_errors=True)
+
+
+@pytest.fixture(name="current_user")
+def fixture_current_user():
+    """Get the current user's username."""
+    username = os.environ["USER"]
+    return username

tests/relink/test_dryrun.py

@@ -4,8 +4,6 @@
 
 import os
 import sys
-import tempfile
-import shutil
 import logging
 
 import pytest
@@ -18,19 +16,6 @@
 import relink  # noqa: E402
 
 
-@pytest.fixture(scope="function", name="temp_dirs")
-def fixture_temp_dirs():
-    """Create temporary source and target directories for testing."""
-    source_dir = tempfile.mkdtemp(prefix="test_source_")
-    target_dir = tempfile.mkdtemp(prefix="test_target_")
-
-    yield source_dir, target_dir
-
-    # Cleanup
-    shutil.rmtree(source_dir, ignore_errors=True)
-    shutil.rmtree(target_dir, ignore_errors=True)
-
-
 @pytest.fixture(name="dry_run_setup")
 def fixture_dry_run_setup(temp_dirs):
     """Set up directories and files for dry-run tests."""

tests/relink/test_find_and_replace_owned_files.py

@@ -5,13 +5,10 @@
 import os
 import sys
 import tempfile
-import shutil
 import pwd
 import logging
 from unittest.mock import patch
 
-import pytest
-
 # Add parent directory to path to import relink module
 sys.path.insert(
     0, os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
@@ -20,26 +17,6 @@
 import relink  # noqa: E402
 
 
-@pytest.fixture(scope="function", name="temp_dirs")
-def fixture_temp_dirs():
-    """Create temporary source and target directories for testing."""
-    source_dir = tempfile.mkdtemp(prefix="test_source_")
-    target_dir = tempfile.mkdtemp(prefix="test_target_")
-
-    yield source_dir, target_dir
-
-    # Cleanup
-    shutil.rmtree(source_dir, ignore_errors=True)
-    shutil.rmtree(target_dir, ignore_errors=True)
-
-
-@pytest.fixture(name="current_user")
-def fixture_current_user():
-    """Get the current user's username."""
-    username = os.environ["USER"]
-    return username
-
-
 def test_basic_file_replacement(temp_dirs, current_user):
     """Test basic functionality: replace owned file with symlink."""
     source_dir, target_dir = temp_dirs

tests/relink/test_verbosity.py

@@ -5,12 +5,9 @@
 import os
 import sys
 import tempfile
-import shutil
 import logging
 from unittest.mock import patch
 
-import pytest
-
 # Add parent directory to path to import relink module
 sys.path.insert(
     0, os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
@@ -19,19 +16,6 @@
 import relink  # noqa: E402
 
 
-@pytest.fixture(scope="function", name="temp_dirs")
-def fixture_temp_dirs():
-    """Create temporary source and target directories for testing."""
-    source_dir = tempfile.mkdtemp(prefix="test_source_")
-    target_dir = tempfile.mkdtemp(prefix="test_target_")
-
-    yield source_dir, target_dir
-
-    # Cleanup
-    shutil.rmtree(source_dir, ignore_errors=True)
-    shutil.rmtree(target_dir, ignore_errors=True)
-
-
 def test_quiet_mode_suppresses_info_messages(temp_dirs, caplog):
     """Test that quiet mode suppresses INFO level messages."""
     source_dir, target_dir = temp_dirs