test(cli): Fix conflicting logging fixtures between test files

why: Test suite was failing when run together due to fixture conflicts
what:
- Add vcspull.cli.fmt to list of loggers cleared in fixtures
- Rename clear_logging_handlers to reset_logging in test_fmt.py to avoid conflicts
- Update test_fmt.py tests to use reset_logging fixture

refs: Fixture isolation issue - clear_logging_handlers with autouse=True affects all tests

Author: tony

tests/cli/test_add.py

@@ -30,6 +30,7 @@ def clear_logging_handlers() -> t.Generator[None, None, None]:
         "vcspull.cli.add",
         "vcspull.cli.add_from_fs",
         "vcspull.cli.sync",
+        "vcspull.cli.fmt",
     ]
     for logger_name in cli_loggers:
         logger = logging.getLogger(logger_name)

tests/cli/test_add_from_fs.py

@@ -32,6 +32,7 @@ def clear_logging_handlers() -> t.Generator[None, None, None]:
         "vcspull.cli.add",
         "vcspull.cli.add_from_fs",
         "vcspull.cli.sync",
+        "vcspull.cli.fmt",
     ]
     for logger_name in cli_loggers:
         logger = logging.getLogger(logger_name)

tests/cli/test_fmt.py

@@ -16,8 +16,8 @@
 
 
 @pytest.fixture
-def clear_logging_handlers() -> t.Generator[None, None, None]:
-    """Clear logging handlers to prevent duplicate log messages in tests."""
+def reset_logging() -> t.Generator[None, None, None]:
+    """Reset logging handlers to prevent duplicate log messages in tests."""
     # Store original handlers
     original_handlers = logging.root.handlers[:]
     # Clear all handlers
@@ -181,7 +181,7 @@ def test_format_file_no_write(
         self,
         tmp_path: pathlib.Path,
         caplog: LogCaptureFixture,
-        clear_logging_handlers: None,
+        reset_logging: None,
     ) -> None:
         """Test formatting without writing changes."""
         config_file = tmp_path / ".vcspull.yaml"
@@ -214,7 +214,7 @@ def test_format_file_with_write(
         self,
         tmp_path: pathlib.Path,
         caplog: LogCaptureFixture,
-        clear_logging_handlers: None,
+        reset_logging: None,
     ) -> None:
         """Test formatting with writing changes."""
         config_file = tmp_path / ".vcspull.yaml"
@@ -252,7 +252,7 @@ def test_already_formatted(
         self,
         tmp_path: pathlib.Path,
         caplog: LogCaptureFixture,
-        clear_logging_handlers: None,
+        reset_logging: None,
     ) -> None:
         """Test when file is already correctly formatted."""
         config_file = tmp_path / ".vcspull.yaml"
@@ -277,7 +277,7 @@ def test_nonexistent_file(
         self,
         tmp_path: pathlib.Path,
         caplog: LogCaptureFixture,
-        clear_logging_handlers: None,
+        reset_logging: None,
     ) -> None:
         """Test handling of nonexistent config file."""
         config_file = tmp_path / "nonexistent.yaml"
@@ -291,7 +291,7 @@ def test_invalid_yaml(
         self,
         tmp_path: pathlib.Path,
         caplog: LogCaptureFixture,
-        clear_logging_handlers: None,
+        reset_logging: None,
     ) -> None:
         """Test handling of invalid YAML."""
         config_file = tmp_path / ".vcspull.yaml"
@@ -307,7 +307,7 @@ def test_no_config_found(
         tmp_path: pathlib.Path,
         caplog: LogCaptureFixture,
         monkeypatch: pytest.MonkeyPatch,
-        clear_logging_handlers: None,
+        reset_logging: None,
     ) -> None:
         """Test when no config file is found."""
         monkeypatch.chdir(tmp_path)
@@ -321,7 +321,7 @@ def test_detailed_change_reporting(
         self,
         tmp_path: pathlib.Path,
         caplog: LogCaptureFixture,
-        clear_logging_handlers: None,
+        reset_logging: None,
     ) -> None:
         """Test detailed reporting of changes."""
         config_file = tmp_path / ".vcspull.yaml"