Please address the comments from this code review:

## Individual Comments

### Comment 1
<location path="tests/test_drive_workers.py" line_range="85-94" />
<code_context>
+    assert drives == ["A:/", "C:/"]
+
+
+def test_get_mounted_drives_applies_exclude_patterns(
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    monkeypatch.setattr(drive_workers, "_get_posix_drives", lambda: ["/", "/mnt/usb"])
+    monkeypatch.setattr(drive_workers.path, "isdir", lambda p: True)
+
+    config = cast("RovrConfig", {"settings": {"drive_exclude": ["/mnt/*"]}})
+    drives = drive_workers.get_mounted_drives("Darwin", config)
+
+    assert drives == ["/"]
+
+
</code_context>
<issue_to_address>
**suggestion (testing):** Add coverage for path.isdir filtering in get_mounted_drives to ensure non-directory mountpoints are excluded.

The current tests patch `path.isdir` to always return `True`, so the directory filtering in `get_mounted_drives` is not actually validated. Please add a test where `_get_posix_drives`/`_get_linux_drives` returns both directory and non-directory mountpoints, and `path.isdir` is patched to return `False` for the invalid ones. This will verify that non-directory entries are excluded and protect against regressions in that logic.

Suggested implementation:

```python
from typing import cast

import pytest

from rovr.classes.config import RovrConfig
from rovr.functions import drive_workers

```

```python
@pytest.fixture
def config() -> RovrConfig:
    return cast("RovrConfig", {"settings": {"drive_exclude": []}})


def test_get_mounted_drives_filters_non_directory_posix_mounts(
    monkeypatch: pytest.MonkeyPatch,
    config: RovrConfig,
) -> None:
    """
    Ensure get_mounted_drives filters out mountpoints that are not directories
    on POSIX-like systems.
    """
    # Mix of directory and non-directory mountpoints
    monkeypatch.setattr(
        drive_workers,
        "_get_posix_drives",
        lambda: ["/", "/mnt/usb", "/not_a_directory"],
    )

    def fake_isdir(path: str) -> bool:
        # Treat "/not_a_directory" as a non-directory mountpoint
        return path != "/not_a_directory"

    monkeypatch.setattr(drive_workers.path, "isdir", fake_isdir)

    drives = drive_workers.get_mounted_drives("Darwin", config)

    assert drives == ["/", "/mnt/usb"]


def test_get_mounted_drives_filters_non_directory_linux_mounts(
    monkeypatch: pytest.MonkeyPatch,
    config: RovrConfig,
) -> None:
    """
    Ensure get_mounted_drives filters out mountpoints that are not directories
    on Linux.
    """
    monkeypatch.setattr(
        drive_workers,
        "_get_linux_drives",
        lambda: ["/", "/media/usb", "/proc", "/not_a_directory"],
    )

    def fake_isdir(path: str) -> bool:
        # Simulate "/not_a_directory" being invalid
        return path != "/not_a_directory"

    monkeypatch.setattr(drive_workers.path, "isdir", fake_isdir)

    drives = drive_workers.get_mounted_drives("Linux", config)

    assert drives == ["/", "/media/usb", "/proc"]

```
</issue_to_address>

### Comment 2
<location path="tests/test_drive_workers.py" line_range="110-119" />
<code_context>
+    assert drives == ["/"]
+
+
+def test_get_mounted_drives_linux_falls_back_to_posix_on_oserror(
+    monkeypatch: pytest.MonkeyPatch, config: RovrConfig
+) -> None:
+    monkeypatch.setattr(
+        drive_workers,
+        "_get_linux_drives",
+        lambda: (_ for _ in ()).throw(OSError("no procfs")),
+    )
+    monkeypatch.setattr(drive_workers, "_get_posix_drives", lambda: ["/data"])
+    monkeypatch.setattr(drive_workers.path, "isdir", lambda p: True)
+
+    drives = drive_workers.get_mounted_drives("Linux", config)
+
+    assert drives == ["/data"]
</code_context>
<issue_to_address>
**suggestion (testing):** There is no test for get_mounted_drives behavior on Windows, only for the Windows helper.

Please add a test that calls `get_mounted_drives("Windows", config)` directly, with `_get_windows_drives` patched to return a mix of drive letters. The test should assert that the function normalises drive paths correctly and respects `path.isdir` and any exclusion patterns, so the Windows integration path is covered end-to-end.

Suggested implementation:

```python
def test_get_mounted_drives_linux_falls_back_to_posix_on_oserror(
    monkeypatch: pytest.MonkeyPatch, config: RovrConfig
) -> None:
    monkeypatch.setattr(
        drive_workers,
        "_get_linux_drives",
        lambda: (_ for _ in ()).throw(OSError("no procfs")),
    )
    monkeypatch.setattr(drive_workers, "_get_posix_drives", lambda: ["/data"])
    monkeypatch.setattr(drive_workers.path, "isdir", lambda p: True)

    drives = drive_workers.get_mounted_drives("Linux", config)

    assert drives == ["/data"]


def test_get_mounted_drives_windows_normalises_and_applies_exclusions(
    monkeypatch: pytest.MonkeyPatch,
    config: RovrConfig,
) -> None:
    # Return a mix of drive specifications, some missing trailing slashes
    # and one that should be excluded via config.
    monkeypatch.setattr(
        drive_workers,
        "_get_windows_drives",
        lambda: ["C:", "D:\\", "E:\\Data", "F:", "G:\\"],
    )

    # Only some of the normalised drives should be considered "mounted".
    def fake_isdir(path_str: str) -> bool:
        # Expect get_mounted_drives to normalise to full root paths (with backslash)
        # and preserve subdirectory paths where appropriate.
        return path_str in {"C:\\", "D:\\", "E:\\Data"}

    monkeypatch.setattr(drive_workers.path, "isdir", fake_isdir)

    # Configure an exclusion pattern so that the E:\Data drive is filtered out,
    # even though path.isdir returns True for it.
    # Adjust this attribute name/pattern to match the real RovrConfig interface.
    config.drive_exclusion_patterns = [r"^E:\\Data$"]

    drives = drive_workers.get_mounted_drives("Windows", config)

    # C and D are mounted and not excluded; E:\Data is mounted but excluded;
    # F and G are not mounted (fake_isdir returns False after normalisation).
    assert drives == ["C:\\", "D:\\"]

```

The new Windows integration test assumes:
1. `get_mounted_drives("Windows", config)` normalises drive roots to include a trailing backslash (e.g. `"C:"` → `"C:\\"`).
2. `RovrConfig` exposes a `drive_exclusion_patterns` attribute that is consulted by `get_mounted_drives` to filter out matches. If your config uses a different attribute name or semantics (e.g. `exclude_paths`, `windows_drive_exclusions`, etc.), update the test to set the correct field and pattern.
3. The exclusion logic operates on the already-normalised drive path strings; if it currently matches raw drive strings from `_get_windows_drives`, you may want to adjust either the implementation or the test so they align.
</issue_to_address>