test: retry known flaky cases per #122 - #138
Conversation
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
…ays open (#200) (#203) * fix(ci): keep route-unit proofs out of the example plain test pool * fix(ci): defer to the mainline Skill IR digest canonicalization (#191) * test: retry the overview handoff e2e while the reload-budget flake stays open (#200) Verify (Node 24) alone exceeded the owned-reload generation budget on a loaded hosted runner (run 33557117407) while the identical commit passed on Node 22 and 26 — the #122-class watch-delivery skew. Same interim treatment as #138 gave the known family.
1 participant
Summary
Adds per-test rstest retries (
{ retry: 2 }) to the two load-sensitive flakes tracked in #122, each annotated with a one-line comment. Root causes stay open; this only mitigates the gate noise.packages/agent-bundle/tests/dev-watcher.test.ts::waits for the real watcher root before reporting create, change, and delete source inputs(chokidar create-event coalescing under full-pool load)packages/workbench/tests/mcp-app-real.e2e.test.ts::opens the real RSC runtime timeline App from provider-owned run evidence(expect.pollbudget under gate load;retry: 2merged into the existing{ timeout: 120_000 }options)No changeset (test-only). Refs #122 — do not close.
Test plan
pnpm exec rstest --config rstest.config.ts packages/agent-bundle/tests/dev-watcher.test.ts packages/workbench/tests/mcp-app-real.e2e.test.ts— 2 files, 6 tests, all passed