Skip to content

Commit 9df359e

Browse files
committed
fix: make test_write_note_project_path_validation cross-platform
The test was failing on Windows because it checked for forward slashes in paths, but Windows uses backslashes. Changed to use Path.parts which works cross-platform regardless of path separator. Fixes CI failures on Windows. Signed-off-by: phernandez <paul@basicmachines.co>
1 parent e1c5f2d commit 9df359e

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

test-int/mcp/test_write_note_integration.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -469,8 +469,8 @@ async def test_write_note_project_path_validation(mcp_server, test_project):
469469
# Verify tilde was expanded (won't contain ~)
470470
assert "~" not in str(home_path)
471471

472-
# Verify it ends with the expected structure
473-
assert str(home_path).endswith("Documents/Test BiSync")
472+
# Verify it ends with the expected structure (use Path.parts for cross-platform)
473+
assert home_path.parts[-2:] == ("Documents", "Test BiSync")
474474

475475
# Also test that write_note works with regular project
476476
async with Client(mcp_server) as client:

0 commit comments

Comments
 (0)