mirror of
https://github.com/openai/codex.git
synced 2026-05-19 02:33:10 +00:00
[codex] fix builtin MCP Windows path test (#21350)
## Summary
- make the builtin MCP config test derive the expected `--codex-home`
argument from `AbsolutePathBuf`
## Why
`AbsolutePathBuf::try_from("/tmp/codex-home")` is rendered as
`D:\\tmp\\codex-home` on Windows, but the test asserted the Unix literal
`"/tmp/codex-home"`. That made the Windows Bazel job fail even though
the production code was behaving correctly.
## Impact
This keeps the test cross-platform while preserving the same transport
assertion on Unix and Windows.
## Validation
- `cargo test -p codex-builtin-mcps`
Co-authored-by: Codex <noreply@openai.com>
This commit is contained in:
@@ -109,7 +109,7 @@ mod tests {
|
||||
"builtin-mcp".to_string(),
|
||||
"memories".to_string(),
|
||||
"--codex-home".to_string(),
|
||||
"/tmp/codex-home".to_string(),
|
||||
codex_home.as_path().to_string_lossy().into_owned(),
|
||||
],
|
||||
env: None,
|
||||
env_vars: Vec::new(),
|
||||
|
||||
Reference in New Issue
Block a user