Skip to content

Commit 28bf37b

Browse files
HanSur94claude
andcommitted
fix: cross-platform test path + suppress asyncio warnings in CI
- test_absolute_path_unchanged: use tmp_path instead of hardcoded Unix path /absolute/path/results (fails on Windows as C:\absolute...) - Suppress PytestUnraisableExceptionWarning (Event loop is closed) in Windows CI tests Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 959e351 commit 28bf37b

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ jobs:
9797
run: |
9898
call .venv\Scripts\activate.bat
9999
pip install pytest pytest-asyncio pytest-cov --quiet
100-
pytest tests/ -v -k "not matlab" --ignore=tests/test_integration.py || echo Tests completed
100+
pytest tests/ -v -k "not matlab" --ignore=tests/test_integration.py -W ignore::pytest.PytestUnraisableExceptionWarning
101101
102102
docker:
103103
needs: lint

tests/test_config.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -166,9 +166,9 @@ def test_relative_temp_dir_resolved(self, tmp_path: Path):
166166
assert Path(cfg.execution.temp_dir).is_absolute()
167167

168168
def test_absolute_path_unchanged(self, tmp_path: Path):
169-
abs_dir = "/absolute/path/results"
169+
abs_dir = str(tmp_path / "absolute" / "path" / "results")
170170
config_file = tmp_path / "config.yaml"
171-
config_file.write_text(f"server:\n result_dir: {abs_dir}\n")
171+
config_file.write_text(f"server:\n result_dir: '{abs_dir}'\n")
172172
cfg = load_config(config_file)
173173
assert cfg.server.result_dir == abs_dir
174174

0 commit comments

Comments
 (0)