openai/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-09-14 11:57:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

tests(core): remove extra shell spawn from approval writes

Browse Source 
Co-authored-by: Codex <noreply@openai.com>
This commit is contained in:
Ahmed Ibrahim
2026-03-13 17:09:19 +00:00
parent cb046f2927
commit 1c084a71af
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
codex-rs/core/tests/suite/approvals.rs
View File

@@ -122,9 +122,11 @@ impl ActionKind {
ActionKind::WriteFile { target, content } => {
let (path, _) = target.resolve_for_patch(test);
let _ = fs::remove_file(&path);
let command = format!("printf {content:?} > {path:?} && cat {path:?}");
// Keep this fixture on shell builtins only so approval-flow timing
// does not depend on launching an extra `cat` subprocess.
let command = format!("printf {content:?} > {path:?} && printf {content:?}");
let event =
approval_matrix_shell_event(call_id, &command, 5_000, sandbox_permissions)?;
approval_matrix_shell_event(call_id, &command, 1_000, sandbox_permissions)?;
Ok((event, Some(command)))
}
ActionKind::FetchUrl {