Skip to content

Tests: split the state-layout move flush crash matrix into one test per flush - #676

Merged
ualtinok merged 2 commits into
mainfrom
test/split-move-flush
Nov 8, 2026
Merged

ualtinok merged 2 commits into
mainfrom
test/split-move-flush

Conversation

@antauth-alfonso

@antauth-alfonso antauth-alfonso Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

Tests: split the state-layout move flush crash matrix into one test per flush. Worker delivery rebased onto main; see the commit messages.


Summary by cubic

Splits the state-layout move flush crash/recovery matrix into one test per flush so no single test hits the per-test deadline.

  • The exact flush-order probe is now its own test, with the flush sites written out from the design rather than measured from the run.
  • Each of the eleven crash recovery cases runs in its own test; the slowest takes 1.15 s instead of the 6.79 s the combined matrix took.
  • Adds an Unreleased changelog entry.

Written for commit 8082d31. Summary will update on new commits.

View guided diff Turn on auto-fix

Give each of the eleven directory-flush crash/recovery cases its own test while retaining the independent exact-flush-sequence probe and every recovery assertion. The matrix-only measurement took 6.79 seconds altogether, but the slowest individual case took 1.15 seconds; no deadlines or production timing change.

The restore sharing-violation test already forwards an injected clock and sleep. Measurements found three attempts and fake 10/20 ms delays, not real backoff sleeps, so leave it unchanged. Its original timeout did not reproduce.
@ualtinok
ualtinok merged commit 8082d31 into main Nov 8, 2026
16 checks passed
@ualtinok
ualtinok deleted the test/split-move-flush branch November 8, 2026 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant