correction

Author: matteius

src/pythonfinder/utils/path_utils.py

@@ -238,14 +238,14 @@ def exists_and_is_accessible(path: Path) -> bool:
     """
     try:
         return path.exists()
-    except PermissionError:
-        # Treat any permission-denied error (including Windows WinError 5) as
-        # inaccessible rather than crashing.
-        return False
-    except OSError:
-        # Catch other OS-level errors (e.g. Windows ERROR_ACCESS_DENIED variants
-        # that may surface as OSError rather than PermissionError).
-        return False
+    except PermissionError as error:
+        if error.errno == errno.EACCES or getattr(error, "winerror", None) == 5:
+            return False
+        raise
+    except OSError as error:
+        if error.errno == errno.EACCES or getattr(error, "winerror", None) == 5:
+            return False
+        raise
 
 
 def is_in_path(path: str | Path, parent_path: str | Path) -> bool:

tests/test_path_utils.py

@@ -241,6 +241,19 @@ def test_exists_and_is_accessible():
         ):
             exists_and_is_accessible(Path("/usr/bin/python"))
 
+    class WindowsAccessDenied(OSError):
+        def __init__(self):
+            super().__init__("Access is denied")
+            self.winerror = 5
+            self.errno = None
+
+    with mock.patch("pathlib.Path.exists", side_effect=WindowsAccessDenied()):
+        assert not exists_and_is_accessible(Path("/usr/bin/python"))
+
+    with pytest.raises(OSError):
+        with mock.patch("pathlib.Path.exists", side_effect=OSError(1, "Other error")):
+            exists_and_is_accessible(Path("/usr/bin/python"))
+
 
 def test_is_in_path():
     """Test that is_in_path correctly checks if a path is inside another path."""