Skip to content

Use wait_for_event helpers in tests#4753

Merged
pakrym-oai merged 1 commit intomainfrom
codex/replace-loops-with-wait_for_event-calls
Oct 5, 2025
Merged

Use wait_for_event helpers in tests#4753
pakrym-oai merged 1 commit intomainfrom
codex/replace-loops-with-wait_for_event-calls

Conversation

@pakrym-oai
Copy link
Copy Markdown
Collaborator

Summary

  • replace manual event polling loops in several core test suites with the shared wait_for_event helpers
  • keep prior assertions intact by using closure captures for stateful expectations, including plan updates, patch lifecycles, and review flow checks
  • rely on wait_for_event_with_timeout where longer waits are required, simplifying timeout handling

Testing

  • just fmt

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

@pakrym-oai pakrym-oai added the codex Label used by connector to tag PRs that have been reviewed by Codex label Oct 5, 2025 — with ChatGPT Codex Connector
@pakrym-oai pakrym-oai merged commit 06853d9 into main Oct 5, 2025
20 checks passed
@pakrym-oai pakrym-oai deleted the codex/replace-loops-with-wait_for_event-calls branch October 5, 2025 05:04
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 5, 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