Commit 000e966
committed
Keep Ray integration tests runnable after the Ray 2.55.1 upgrade
Ray 2.55.1 no longer accepts `working_dir=None` in the session fixture, and `uv run pytest` also needs the uv runtime-env hook disabled before importing Ray. This switches the fixture to a temporary working directory and sets the environment variable early enough for the Ray integration tests to keep using the existing pytest entrypoint.
Constraint: Ray 2.55.1 rejects `runtime_env["working_dir"] = None`
Constraint: The project test entrypoint runs through `uv run pytest`
Rejected: Copy the repo into a temporary directory | adds global cwd side effects and more fixture overhead
Rejected: Keep `working_dir=None` and pin older Ray | does not address the upgrade target
Confidence: medium
Scope-risk: narrow
Reversibility: clean
Directive: If the Ray fixture changes again, verify behavior with `uv run pytest`, not only an activated virtualenv
Tested: `uv run pytest tests/integration/test_reads.py -m integration -k 'test_ray_' -vv`
Not-tested: Full integration suite and CI matrix outside the Ray-targeted cases1 parent 714c24f commit 000e966
1 file changed
Lines changed: 8 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3030 | 3030 | | |
3031 | 3031 | | |
3032 | 3032 | | |
3033 | | - | |
| 3033 | + | |
3034 | 3034 | | |
| 3035 | + | |
| 3036 | + | |
| 3037 | + | |
| 3038 | + | |
3035 | 3039 | | |
3036 | 3040 | | |
| 3041 | + | |
| 3042 | + | |
3037 | 3043 | | |
3038 | 3044 | | |
3039 | | - | |
| 3045 | + | |
3040 | 3046 | | |
3041 | 3047 | | |
3042 | 3048 | | |
| |||
0 commit comments