fix(tests): skip additional Windows-incompatible tests

- test_logging.py: Skip file logging tests due to Windows file locking
- test_reconcile.py: Skip path detection tests using Unix-style paths

Author: CybotTM

tests/test_logging.py

@@ -3,11 +3,18 @@
 """
 
 import logging
+import sys
 import tempfile
 from pathlib import Path
 
 import pytest
 
+# Skip marker for Windows (file locking issues with temp files)
+skip_on_windows = pytest.mark.skipif(
+    sys.platform == "win32",
+    reason="Windows file locking prevents temp file cleanup"
+)
+
 from cli_audit.logging_config import (
     setup_logging,
     get_logger,
@@ -46,6 +53,7 @@ def test_setup_logging_quiet(self):
         ]
         assert len(console_handlers) == 0
 
+    @skip_on_windows
     def test_setup_logging_with_file(self):
         """Test logging to file."""
         with tempfile.TemporaryDirectory() as tmpdir:
@@ -64,6 +72,7 @@ def test_setup_logging_with_file(self):
             content = log_file.read_text()
             assert "Test message" in content
 
+    @skip_on_windows
     def test_setup_logging_creates_log_directory(self):
         """Test that log directory is created if it doesn't exist."""
         with tempfile.TemporaryDirectory() as tmpdir:

tests/test_reconcile.py

@@ -5,12 +5,19 @@
 from __future__ import annotations
 
 import os
+import sys
 import tempfile
 from pathlib import Path
 from unittest.mock import MagicMock, patch, mock_open
 
 import pytest
 
+# Skip marker for Windows (Unix-style paths and PATH separator differences)
+skip_on_windows = pytest.mark.skipif(
+    sys.platform == "win32",
+    reason="Uses Unix-style paths and PATH separator (:)"
+)
+
 from cli_audit.reconcile import (
     Installation,
     ReconciliationResult,
@@ -124,6 +131,7 @@ def test_classify_via_path_unknown(self):
 class TestDetectInstallations:
     """Tests for installation detection."""
 
+    @skip_on_windows
     @patch("cli_audit.reconcile.validate_installation")
     @patch("cli_audit.reconcile.shutil.which")
     @patch("os.path.exists")
@@ -163,6 +171,7 @@ def exists_side_effect(path):
         assert installations[0].version == "14.1.0"
         assert installations[0].active is True
 
+    @skip_on_windows
     @patch("cli_audit.reconcile.validate_installation")
     @patch("cli_audit.reconcile.shutil.which")
     @patch("os.path.exists")
@@ -543,6 +552,7 @@ def reconcile_side_effect(tool, *args, **kwargs):
 class TestPathVerification:
     """Tests for PATH verification."""
 
+    @skip_on_windows
     def test_verify_path_ordering_correct(self, monkeypatch):
         """Test PATH ordering when correct."""
         monkeypatch.setenv("PATH", "/home/user/.cargo/bin:/home/user/.local/bin:/usr/local/bin:/usr/bin")
@@ -574,6 +584,7 @@ def expand_side_effect(path):
                 issues = verify_path_ordering()
                 assert len(issues) == 0
 
+    @skip_on_windows
     def test_verify_path_ordering_incorrect(self, monkeypatch):
         """Test PATH ordering when incorrect."""
         monkeypatch.setenv("PATH", "/usr/bin:/home/user/.cargo/bin")