Repository navigation
test(renderer): explicit budgets for the remaining full-App edge tests - #534
Merged
Merged
Conversation
After the toolbar test got its 15s budget (#531), the next-slowest full-App tests surfaced at the 5s edge on macOS runners: the turn-link re-anchor test timed out on four consecutive runs across #527/#522/#513, and the HireDrawer binding and docs undo/redo tests each timed out once under the same runner load. Give all three the same explicit 15s budget the other heavy tests in these files already carry.
This was referenced Sep 29, 2026
PeterGuy326
added a commit
that referenced
this pull request
Sep 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Preflight
main@ed94fca.Summary
After the toolbar test got its explicit 15s budget (#531), the next-slowest tests surfaced at the 5s vitest default on macOS runners. This gives the three observed edge-dwellers the same explicit 15s budget the other heavy tests in these files already carry:
App.test.tsx > reanchors a repeated project turn link…App.test.tsx > binds the ready %s runtime in HireDrawer…109243787847)docs-lifecycle.test.tsx > preserves undo and redo history…109243787847)No production code changes; test files only.
Evidence
Test timed out in 5000ms(job logs109243787847,109247014992,109247113086), each followed by a green rerun or a pass on the other platform — the signature of cold, heavy tests at the 5s edge under runner load, not hangs.reanchorscase timed out on every macOS run observed today (4/4), so it sits furthest above the default.Assumptions
Acceptance
Node 24 / ubuntu-latestandNode 24 / macos-14pass on this PR and on fresh runs ofmainafter merge.Refs #529, #531