From ba9e2e8a4bda0345f815a78e154ea581d30ec6d8 Mon Sep 17 00:00:00 2001 From: StillKnotKnown Date: Sat, 7 Feb 2026 16:29:19 +0200 Subject: [PATCH] test: fix test_IdeationConfigManager_default_values path comparison Use string comparison instead of resolve() for path comparison. On Windows, /tmp/test resolves to D:/tmp/test, and on macOS /tmp/test resolves to /private/tmp/test, but the implementation stores paths as-is without resolving. --- tests/ideation/test_config.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/ideation/test_config.py b/tests/ideation/test_config.py index 509c5278..f049f3f3 100644 --- a/tests/ideation/test_config.py +++ b/tests/ideation/test_config.py @@ -64,8 +64,8 @@ def test_IdeationConfigManager_default_values(mock_init): config = IdeationConfigManager(project_dir=project_dir) - # Use resolve() for cross-platform compatibility (macOS /tmp -> /private/tmp) - assert config.project_dir == project_dir.resolve() + # Compare paths without resolve() since the implementation stores paths as-is + assert str(config.project_dir) == str(project_dir) assert config.model == "sonnet" assert config.thinking_level == "medium" assert config.refresh is False