Skip to content

[fix] app server flaky thread/resume tests#8870

Merged
celia-oai merged 1 commit intomainfrom
dev/cc/fix-tests
Jan 8, 2026
Merged

[fix] app server flaky thread/resume tests#8870
celia-oai merged 1 commit intomainfrom
dev/cc/fix-tests

Conversation

@celia-oai
Copy link
Collaborator

@celia-oai celia-oai commented Jan 7, 2026

Fix flakiness of CI tests: https://github.com/openai/codex/actions/runs/20350530276/job/58473691443?pr=8282

This PR does two things:

  1. test with responses API instead of chat completions API in thread_resume tests;
  2. have a new responses API fixture that mocks out arbitrary numbers of responses API calls (including no calls) and have the same repeated response.

Tested by CI

@celia-oai celia-oai changed the title draft [fix] app server flaky thread/resume tests Jan 8, 2026
@celia-oai celia-oai marked this pull request as ready for review January 8, 2026 00:38
@celia-oai celia-oai merged commit c4af304 into main Jan 8, 2026
26 checks passed
@celia-oai celia-oai deleted the dev/cc/fix-tests branch January 8, 2026 18:17
@github-actions github-actions bot locked and limited conversation to collaborators Jan 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants