From 83780ddbffc8b64c1a1e3749618cd9296d7dfd59 Mon Sep 17 00:00:00 2001 From: Friel Date: Sat, 28 Mar 2026 11:53:11 -0700 Subject: [PATCH] test(core): annotate watchdog test literals --- codex-rs/app-server/tests/suite/v2/thread_read.rs | 2 +- codex-rs/core/src/codex_tests.rs | 10 ++++++---- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/codex-rs/app-server/tests/suite/v2/thread_read.rs b/codex-rs/app-server/tests/suite/v2/thread_read.rs index 98586983ee..1556f2bd09 100644 --- a/codex-rs/app-server/tests/suite/v2/thread_read.rs +++ b/codex-rs/app-server/tests/suite/v2/thread_read.rs @@ -438,7 +438,7 @@ async fn thread_read_loaded_ephemeral_thread_ignores_unrelated_rollout_mentions( &unrelated_preview, vec![], Some("mock_provider"), - None, + /*git_info*/ None, )?; let read_id = mcp diff --git a/codex-rs/core/src/codex_tests.rs b/codex-rs/core/src/codex_tests.rs index b72fad96f7..2833697b0f 100644 --- a/codex-rs/core/src/codex_tests.rs +++ b/codex-rs/core/src/codex_tests.rs @@ -5296,10 +5296,11 @@ async fn unified_exec_rejects_escalated_permissions_when_policy_not_on_request() async fn root_agent_prompt_only_includes_watchdog_fragment_when_enabled() { let codex_home = tempfile::tempdir().expect("create temp dir"); - let without_watchdog = load_root_agent_prompt(codex_home.path(), false).await; + let without_watchdog = + load_root_agent_prompt(codex_home.path(), /*include_watchdog*/ false).await; assert!(!without_watchdog.contains("## Watchdogs")); - let with_watchdog = load_root_agent_prompt(codex_home.path(), true).await; + let with_watchdog = load_root_agent_prompt(codex_home.path(), /*include_watchdog*/ true).await; assert!(with_watchdog.contains("## Watchdogs")); } @@ -5307,9 +5308,10 @@ async fn root_agent_prompt_only_includes_watchdog_fragment_when_enabled() { async fn subagent_prompt_only_includes_watchdog_fragment_when_enabled() { let codex_home = tempfile::tempdir().expect("create temp dir"); - let without_watchdog = load_subagent_prompt(codex_home.path(), false).await; + let without_watchdog = + load_subagent_prompt(codex_home.path(), /*include_watchdog*/ false).await; assert!(!without_watchdog.contains("## Watchdog-only Guidance")); - let with_watchdog = load_subagent_prompt(codex_home.path(), true).await; + let with_watchdog = load_subagent_prompt(codex_home.path(), /*include_watchdog*/ true).await; assert!(with_watchdog.contains("## Watchdog-only Guidance")); }