test: simplify Windows drive-letter test to fix coverage

The previous tests mocked Path.resolve with a conditional that fell through
to the original implementation — but with the fix in place, only the full
path ever hits resolve(), so the fallthrough was dead code and missed
coverage (lines 129, 146).

Replace with a single parametrized test that records what resolve() was
called with. Before the fix, a bare 'C:\path\server.py' would rsplit on
the drive colon and call resolve(Path('C')); now it calls resolve on the
full path. The recorder asserts that directly — no dead branches.

Also covers the has_windows_drive=True + :object-suffix branch that was
previously missed.

Author: maxisbey

tests/cli/test_claude.py

@@ -117,37 +117,30 @@ def fake_which(cmd: str) -> str | None:
     assert get_uv_path() == expected
 
 
-def test_windows_drive_letter_without_object(config_dir: Path, monkeypatch: pytest.MonkeyPatch):
-    """Windows paths like C:\\path\\server.py without :object should not split on drive letter."""
-    # Mock Path.resolve to return a fake Windows-style path
-    original_resolve = Path.resolve
+@pytest.mark.parametrize(
+    "file_spec, expected_last_arg",
+    [
+        ("C:\\Users\\server.py", "C:\\Users\\server.py"),
+        ("C:\\Users\\server.py:app", "C:\\Users\\server.py:app"),
+    ],
+)
+def test_windows_drive_letter_not_split(
+    config_dir: Path, monkeypatch: pytest.MonkeyPatch, file_spec: str, expected_last_arg: str
+):
+    """Drive-letter paths like 'C:\\server.py' must not be split on the drive colon.
+
+    Before the fix, a bare 'C:\\path\\server.py' would hit rsplit(":", 1) and yield
+    ("C", "\\path\\server.py"), calling resolve() on Path("C") instead of the full path.
+    """
+    seen: list[str] = []
 
     def fake_resolve(self: Path) -> Path:
-        if str(self) == "C:\\Users\\foo\\server.py":
-            # Return the same path as if it were already resolved
-            return self
-        return original_resolve(self)
+        seen.append(str(self))
+        return self
 
     monkeypatch.setattr(Path, "resolve", fake_resolve)
 
-    assert update_claude_config(file_spec="C:\\Users\\foo\\server.py", server_name="s")
+    assert update_claude_config(file_spec=file_spec, server_name="s")
 
-    # Should use the full path without splitting on the drive letter colon
-    assert _read_server(config_dir, "s")["args"][-1] == "C:\\Users\\foo\\server.py"
-
-
-def test_windows_drive_letter_with_object(config_dir: Path, monkeypatch: pytest.MonkeyPatch):
-    """Windows paths like C:\\path\\server.py:app should only split on the :app suffix."""
-    original_resolve = Path.resolve
-
-    def fake_resolve(self: Path) -> Path:
-        if str(self) == "C:\\Users\\foo\\server.py":
-            return self
-        return original_resolve(self)
-
-    monkeypatch.setattr(Path, "resolve", fake_resolve)
-
-    assert update_claude_config(file_spec="C:\\Users\\foo\\server.py:app", server_name="s")
-
-    # Should split on :app but not on C:
-    assert _read_server(config_dir, "s")["args"][-1] == "C:\\Users\\foo\\server.py:app"
+    assert seen == ["C:\\Users\\server.py"]
+    assert _read_server(config_dir, "s")["args"][-1] == expected_last_arg