diff --git a/codex-rs/core/src/agent/role_tests.rs b/codex-rs/core/src/agent/role_tests.rs index 624b8b9ba134..6f359eb38d67 100644 --- a/codex-rs/core/src/agent/role_tests.rs +++ b/codex-rs/core/src/agent/role_tests.rs @@ -71,21 +71,6 @@ async fn apply_role_returns_error_for_unknown_role() { assert_eq!(err, "unknown agent_type 'missing-role'"); } -#[tokio::test] -#[ignore = "No role requiring it for now"] -async fn apply_explorer_role_sets_model_and_adds_session_flags_layer() { - let (_home, mut config) = test_config_with_cli_overrides(Vec::new()).await; - let before_layers = session_flags_layer_count(&config); - - apply_role_to_config(&mut config, Some("explorer")) - .await - .expect("explorer role should apply"); - - assert_eq!(config.model.as_deref(), Some("gpt-5.4-mini")); - assert_eq!(config.model_reasoning_effort, Some(ReasoningEffort::Medium)); - assert_eq!(session_flags_layer_count(&config), before_layers + 1); -} - #[tokio::test] async fn apply_empty_explorer_role_preserves_current_model_and_reasoning_effort() { let (_home, mut config) = test_config_with_cli_overrides(Vec::new()).await; diff --git a/codex-rs/core/src/unified_exec/mod_tests.rs b/codex-rs/core/src/unified_exec/mod_tests.rs index 389164a43913..22c08f4aeb8d 100644 --- a/codex-rs/core/src/unified_exec/mod_tests.rs +++ b/codex-rs/core/src/unified_exec/mod_tests.rs @@ -552,67 +552,6 @@ async fn unified_exec_pause_blocks_yield_timeout() -> anyhow::Result<()> { Ok(()) } -#[tokio::test] -#[ignore] // Ignored while we have a better way to test this. -async fn requests_with_large_timeout_are_capped() -> anyhow::Result<()> { - let (session, turn) = test_session_and_turn().await; - - let result = exec_command( - &session, - &turn, - "echo codex", - /*yield_time_ms*/ 120_000, - /*workdir*/ None, - ) - .await?; - - assert!(result.process_id.is_some()); - assert!( - result - .truncated_output(DEFAULT_MAX_OUTPUT_TOKENS) - .contains("codex") - ); - - Ok(()) -} - -#[tokio::test] -#[ignore] // Ignored while we have a better way to test this. -async fn completed_commands_do_not_persist_sessions() -> anyhow::Result<()> { - let (session, turn) = test_session_and_turn().await; - let result = exec_command( - &session, - &turn, - "echo codex", - /*yield_time_ms*/ 2_500, - /*workdir*/ None, - ) - .await?; - - assert!( - result.process_id.is_some(), - "completed command should report a process id" - ); - assert!( - result - .truncated_output(DEFAULT_MAX_OUTPUT_TOKENS) - .contains("codex") - ); - - assert!( - session - .services - .unified_exec_manager - .process_store - .lock() - .await - .processes - .is_empty() - ); - - Ok(()) -} - #[tokio::test(flavor = "multi_thread", worker_threads = 2)] async fn reusing_completed_process_returns_unknown_process() -> anyhow::Result<()> { skip_if_sandbox!(Ok(()));