Skip to content

Commit 68f5479

Browse files
charley-oaicodex
andcommitted
Fix rebased guardian approvals tests
Co-authored-by: Codex <noreply@openai.com>
1 parent 938aaeb commit 68f5479

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

codex-rs/tui/src/app.rs

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -5584,21 +5584,21 @@ mod tests {
55845584
.permissions
55855585
.sandbox_policy
55865586
.get(),
5587-
&smart_approvals.sandbox_policy
5587+
&guardian_approvals.sandbox_policy
55885588
);
55895589
assert_eq!(
55905590
app.chat_widget.config_ref().approvals_reviewer,
5591-
smart_approvals.approvals_reviewer
5591+
guardian_approvals.approvals_reviewer
55925592
);
55935593
assert_eq!(app.runtime_approval_policy_override, None);
55945594
assert_eq!(app.runtime_sandbox_policy_override, None);
55955595
assert_eq!(
55965596
op_rx.try_recv(),
55975597
Ok(Op::OverrideTurnContext {
55985598
cwd: None,
5599-
approval_policy: Some(smart_approvals.approval_policy),
5600-
approvals_reviewer: Some(smart_approvals.approvals_reviewer),
5601-
sandbox_policy: Some(smart_approvals.sandbox_policy.clone()),
5599+
approval_policy: Some(guardian_approvals.approval_policy),
5600+
approvals_reviewer: Some(guardian_approvals.approvals_reviewer),
5601+
sandbox_policy: Some(guardian_approvals.sandbox_policy.clone()),
56025602
windows_sandbox_level: None,
56035603
model: None,
56045604
effort: None,
@@ -5619,10 +5619,10 @@ mod tests {
56195619
.map(|line| line.to_string())
56205620
.collect::<Vec<_>>()
56215621
.join("\n");
5622-
assert!(rendered.contains("Permissions updated to Smart Approvals"));
5622+
assert!(rendered.contains("Permissions updated to Guardian Approvals"));
56235623

56245624
let config = std::fs::read_to_string(codex_home.path().join("config.toml"))?;
5625-
assert!(config.contains("smart_approvals = true"));
5625+
assert!(config.contains("guardian_approval = true"));
56265626
assert!(config.contains("approvals_reviewer = \"guardian_subagent\""));
56275627
assert!(config.contains("approval_policy = \"on-request\""));
56285628
assert!(config.contains("sandbox_mode = \"workspace-write\""));
@@ -5634,7 +5634,7 @@ mod tests {
56345634
let (mut app, mut app_event_rx, mut op_rx) = make_test_app_with_channels().await;
56355635
let codex_home = tempdir()?;
56365636
app.config.codex_home = codex_home.path().to_path_buf();
5637-
let smart_approvals = super::smart_approvals_mode();
5637+
let guardian_approvals = super::guardian_approvals_mode();
56385638

56395639
app.update_feature_flags(vec![(Feature::GuardianApproval, true)])
56405640
.await;
@@ -5648,19 +5648,19 @@ mod tests {
56485648
);
56495649
assert_eq!(
56505650
app.config.approvals_reviewer,
5651-
smart_approvals.approvals_reviewer
5651+
guardian_approvals.approvals_reviewer
56525652
);
56535653
assert_eq!(
56545654
app.config.permissions.approval_policy.value(),
5655-
smart_approvals.approval_policy
5655+
guardian_approvals.approval_policy
56565656
);
56575657
assert_eq!(
56585658
app.chat_widget
56595659
.config_ref()
56605660
.permissions
56615661
.approval_policy
56625662
.value(),
5663-
smart_approvals.approval_policy
5663+
guardian_approvals.approval_policy
56645664
);
56655665
assert_eq!(
56665666
app.chat_widget

0 commit comments

Comments
 (0)