Skip to content

test: raise the 100k deep-cascade timeout to 60s (CI flake under load)#90

Merged
andymai merged 1 commit into
mainfrom
fix/flaky-deep-cascade-timeout
Jun 8, 2026
Merged

test: raise the 100k deep-cascade timeout to 60s (CI flake under load)#90
andymai merged 1 commit into
mainfrom
fix/flaky-deep-cascade-timeout

Conversation

@andymai
Copy link
Copy Markdown
Owner

@andymai andymai commented Jun 8, 2026

The recursion-free deep-cascade test (m8-relations.property.test.ts) builds a 100k-link chain before despawning — ~8s locally, but it overran the 30s timeout on a contended CI runner and spuriously failed a docs PR. Bump to 60s (7.5× the local time); the assertion and the decisive DEPTH=100000 are unchanged. Pure CI-robustness fix.

The recursion-free deep-cascade test builds a 100k-link chain (100k spawn +
addPair) before despawning — ~8s locally, but it can exceed the 30s timeout on a
heavily contended CI runner, producing spurious failures unrelated to the code
under test. Bump to 60s (7.5x the local time) for headroom; the assertion and the
decisive DEPTH=100k are unchanged.
@andymai andymai enabled auto-merge (squash) June 8, 2026 10:17
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Jun 8, 2026

Greptile Summary

Bumps the timeout on the a DEEP (100k) deleteSubject chain despawns without recursion test from 30 s to 60 s to address intermittent CI failures on contended runners. No logic, assertions, or test parameters are changed.

  • The only change is timeout: 30_000timeout: 60_000 in a single it(...) call in m8-relations.property.test.ts.
  • The test itself — including the DEPTH=100000 constant, the BFS-cascade assertion, and the aliveCount check — is untouched.

Confidence Score: 5/5

Safe to merge — this is a one-line timeout increase with no changes to test logic or application code.

The change is narrowly scoped: one timeout value in one test, with all assertions, constants, and cascade logic left intact. There is no risk of masking real slowness at 60 s given that the test runs in ~8 s locally.

No files require special attention.

Important Files Changed

Filename Overview
packages/relations/test/m8-relations.property.test.ts Single-line timeout bump for the 100k deep-cascade test from 30s to 60s to prevent spurious CI failures on slow runners.

Reviews (1): Last reviewed commit: "test: raise the 100k deep-cascade timeou..." | Re-trigger Greptile

@andymai andymai merged commit 2d87e87 into main Jun 8, 2026
9 checks passed
@andymai andymai deleted the fix/flaky-deep-cascade-timeout branch June 8, 2026 10:19
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