Skip to content

Commit 0eb24eb

Browse files
jope-bmclaude
andcommitted
fix: Normalize path comparison in v2 project router test for Windows
The test was directly comparing path strings, which fails on Windows because the API returns paths with forward slashes while Path() creates paths with backslashes on Windows. Changed to use Path() objects for comparison, which normalizes path separators across platforms. Fixes test_update_project_path_by_id on Windows. 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Joe P <joe@basicmemory.com>
1 parent a32d0c8 commit 0eb24eb

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

tests/api/v2/test_project_router.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,8 @@ async def test_update_project_path_by_id(
5353
status_response = ProjectStatusResponse.model_validate(response.json())
5454
assert status_response.status == "success"
5555
assert status_response.new_project.id == test_project.id
56-
assert status_response.new_project.path == new_path
56+
# Normalize paths for cross-platform comparison (Windows uses backslashes, API returns forward slashes)
57+
assert Path(status_response.new_project.path) == Path(new_path)
5758
assert status_response.old_project.id == test_project.id
5859

5960

0 commit comments

Comments
 (0)