Skip to content

test: control lazy recovery timing - #134

Merged
KyleJune merged 1 commit into
mainfrom
test/lazy-recovery-clock
Sep 12, 2026
Merged

KyleJune merged 1 commit into
mainfrom
test/lazy-recovery-clock

Conversation

@KyleJune

Copy link
Copy Markdown
Member

Summary

The macOS main run 34714681380 checked document recovery after a 10 ms wall-clock wait and saw no navigation yet. Advance a fake clock through import rejection and the scheduled handoff so the recovery assertions do not depend on runner timing.

Changes

  • Use FakeTime throughout the eight lazy-route recovery tests.
  • Exercise a 25 ms delayed import rejection in the route-preservation test.
  • Preserve assertions for the destination, pending navigation, current page, error suppression, cancellation retries, and reload budget. Production code and sanitizer settings are unchanged.

Testing

  • deno task check passed; deno task test --parallel --reporter=dot passed (34 suites, 388 steps).
  • Red: deno task test --reporter=dot client.test.tsx, with only the delayed-import case selected and the original 10 ms helper, failed at assertEquals(assigned, ["/destination?q=1#top"]): actual [].
  • Green: deno task test --reporter=dot --filter 'createLazyRoute failure recovery' client.test.tsx passed all eight cases with the clock fix.
  • Mutation: resolving the recovery promise prematurely made the route-preservation test fail in assertPendingRecovery: actual resolved, expected pending. The mutation was restored before the full gates.

Closes

Nothing.

@KyleJune
KyleJune merged commit cf26b02 into main Sep 12, 2026
10 checks passed
@KyleJune
KyleJune deleted the test/lazy-recovery-clock branch September 12, 2026 20:12
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 0.11.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant