Skip to content

Use TestCodex builder in stream retry tests#4096

Merged
pakrym-oai merged 1 commit intomainfrom
codex/update-tests-to-use-testcodex
Sep 23, 2025
Merged

Use TestCodex builder in stream retry tests#4096
pakrym-oai merged 1 commit intomainfrom
codex/update-tests-to-use-testcodex

Conversation

@pakrym-oai
Copy link
Copy Markdown
Collaborator

Summary

  • refactor the stream retry integration tests to construct conversations through TestCodex
  • remove bespoke config and tempdir setup now handled by the shared builder

Testing

  • cargo test -p codex-core --test all stream_error_allows_next_turn::continue_after_stream_error
  • cargo test -p codex-core --test all stream_no_completed::retries_on_early_close

https://chatgpt.com/codex/tasks/task_i_68d2b94d83888320bc75a0bc3bd77b49

@pakrym-oai pakrym-oai added the codex Label used by connector to tag PRs that have been reviewed by Codex label Sep 23, 2025 — with ChatGPT Codex Connector
@pakrym-oai pakrym-oai merged commit 76ecbb3 into main Sep 23, 2025
19 checks passed
@pakrym-oai pakrym-oai deleted the codex/update-tests-to-use-testcodex branch September 23, 2025 15:57
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

codex Label used by connector to tag PRs that have been reviewed by Codex

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants