Michael Bolin b4787bf4c0 fix: changes to test that should help them pass on Windows under Bazel (#16662) ...

https://github.com/openai/codex/pull/16460 was a large PR created by
Codex to try to get the tests to pass under Bazel on Windows. Indeed, it
successfully ran all of the tests under `//codex-rs/core:` with its
changes to `codex-rs/core/`, though the full set of changes seems to be
too broad.

This PR tries to port the key changes, which are:

- Under Bazel, the `USERNAME` environment variable is not guaranteed to
be set on Windows, so for tests that need a non-empty env var as a
convenient substitute for an env var containing an API key, just use
`PATH`. Note that `PATH` is unlikely to contain characters that are not
allowed in an HTTP header value.
- Specify `"powershell.exe"` instead of just `"powershell"` in case the
`PATHEXT` env var gets lost in the shuffle.

b4787bf4c0 · 2026-04-02 23:06:36 -07:00

History

..

common

extract models manager and related ownership from core (#16508)

2026-04-02 23:00:02 -07:00

fixtures

Update tests to stop using sse_completed fixture (#10638)

2026-02-04 08:38:06 -08:00

suite

fix: changes to test that should help them pass on Windows under Bazel (#16662)

2026-04-02 23:06:36 -07:00

all.rs

extract models manager and related ownership from core (#16508)

2026-04-02 23:00:02 -07:00

cli_responses_fixture.sse

Add CLI streaming integration tests (#1542)

2025-07-12 18:05:58 -07:00

responses_headers.rs

core: remove cross-crate re-exports from lib.rs (#16512)

2026-04-01 23:06:24 -07:00