openai/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-09-15 12:08:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(mcp-server): make Chat Completions WireMock emit [DONE]

Browse Source 
Windows CI sometimes flakes in `codex-mcp-server` tests that stub `/v1/chat/completions` with WireMock.

The mock SSE payload ended with `data: DONE` (nonstandard) instead of the upstream terminal sentinel `data: [DONE]`. Depending on platform/timing, that can leave the client waiting for stream termination via connection close.

Fix the fixture to emit `data: [DONE]` to match server wire format. No changes to production protocol handling.
This commit is contained in:
Josh McKinney
2026-01-05 18:30:06 -08:00
parent cabf85aa18
commit ee3659372c
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
codex-rs/mcp-server/tests/common/responses.rs
View File

@@ -34,7 +34,7 @@ pub fn create_shell_command_sse_response(
});
let sse = format!(
"data: {}\n\ndata: DONE\n\n",
"data: {}\n\ndata: [DONE]\n\n",
serde_json::to_string(&tool_call)?
);
Ok(sse)
@@ -53,7 +53,7 @@ pub fn create_final_assistant_message_sse_response(message: &str) -> anyhow::Res
});
let sse = format!(
"data: {}\n\ndata: DONE\n\n",
"data: {}\n\ndata: [DONE]\n\n",
serde_json::to_string(&assistant_message)?
);
Ok(sse)
@@ -89,7 +89,7 @@ pub fn create_apply_patch_sse_response(
});
let sse = format!(
"data: {}\n\ndata: DONE\n\n",
"data: {}\n\ndata: [DONE]\n\n",
serde_json::to_string(&tool_call)?
);
Ok(sse)