test: fix CI failures for path resolution and enum iteration
- Fix test_init_with_absolute_path in test_search_comprehensive.py: Use .resolve() for cross-platform path comparison (macOS /tmp -> /private/tmp) - Fix test_enum_iteration in test_phase_event.py: Add missing enum values rate_limit_paused and auth_failure_paused These are the same platform-specific issues fixed earlier for other tests.
This commit is contained in:
@@ -28,7 +28,9 @@ class TestCodeSearcherInit:
|
||||
"""Test initialization with absolute path"""
|
||||
project_dir = Path("/tmp/test_project")
|
||||
searcher = CodeSearcher(project_dir)
|
||||
assert searcher.project_dir == project_dir
|
||||
# On macOS, /tmp is a symlink to /private/tmp, so compare resolved paths
|
||||
# On Windows, absolute paths may include drive letter
|
||||
assert searcher.project_dir == project_dir.resolve()
|
||||
|
||||
|
||||
class TestIterCodeFiles:
|
||||
|
||||
@@ -513,5 +513,5 @@ class TestExecutionPhaseEnum:
|
||||
def test_enum_iteration(self):
|
||||
"""Test iterating over all enum values."""
|
||||
phases = [phase.value for phase in ExecutionPhase]
|
||||
expected = ["planning", "coding", "qa_review", "qa_fixing", "complete", "failed"]
|
||||
expected = ["planning", "coding", "qa_review", "qa_fixing", "complete", "failed", "rate_limit_paused", "auth_failure_paused"]
|
||||
assert phases == expected
|
||||
|
||||
Reference in New Issue
Block a user