From 652e8a946bffb465c212586616e2215e29af17bf Mon Sep 17 00:00:00 2001 From: Joe Florencio Date: Tue, 2 Jun 2026 14:03:00 -0700 Subject: [PATCH] fix: update app-server image generation test helper PR #25701 renamed the app-server test process helper from McpProcess to TestAppServer and updated existing tests, but image_generation.rs still referenced the old app_test_support::McpProcess export. Update that test to use TestAppServer so the app-server integration test binary compiles. --- codex-rs/app-server/tests/suite/v2/image_generation.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/codex-rs/app-server/tests/suite/v2/image_generation.rs b/codex-rs/app-server/tests/suite/v2/image_generation.rs index 4752f1497ecc..e1f4cfbc9af3 100644 --- a/codex-rs/app-server/tests/suite/v2/image_generation.rs +++ b/codex-rs/app-server/tests/suite/v2/image_generation.rs @@ -4,7 +4,7 @@ use std::time::Duration; use anyhow::Context; use anyhow::Result; use app_test_support::ChatGptAuthFixture; -use app_test_support::McpProcess; +use app_test_support::TestAppServer; use app_test_support::to_response; use app_test_support::write_chatgpt_auth; use codex_app_server_protocol::ItemCompletedNotification; @@ -76,7 +76,8 @@ async fn standalone_image_generation_persists_image_and_returns_it_to_model() -> AuthCredentialsStoreMode::File, )?; - let mut mcp = McpProcess::new_with_env(codex_home.path(), &[("OPENAI_API_KEY", None)]).await?; + let mut mcp = + TestAppServer::new_with_env(codex_home.path(), &[("OPENAI_API_KEY", None)]).await?; timeout(DEFAULT_READ_TIMEOUT, mcp.initialize()).await??; let thread_req = mcp @@ -157,7 +158,7 @@ async fn standalone_image_generation_persists_image_and_returns_it_to_model() -> } async fn wait_for_image_generation_completed( - mcp: &mut McpProcess, + mcp: &mut TestAppServer, ) -> Result { loop { let notification = mcp