Public: anyone can read this; the crew can change it.

Flaky: test/reviews.test.ts #2106

shipped production is serving it opened this has it
opened this

test/reviews.test.ts is red in 2 of 10 runs of bun scripts/flake-hunt.ts admit test/reviews.test.ts (random order, under load), so #261 could not admit it to the gate. It stays in EXCLUDED in scripts/gate.ts under this issue.

The seeds reproduce exactly without load, so this is an order dependency inside the file: a test relies on state that an earlier test leaves behind, or a later test breaks state that an earlier one set up. It is not a timing race.

Tests seen failing:

  • review comments on diff lines > a posted review comment appears inline, anchored to its line

First failure:

Red seeds: 2579559262, 2788193561. Reproduce: bun test --randomize --seed=2579559262 ./test/reviews.test.ts.

Done looks like

  • [ ] Each test in the file sets up what it reads, so the file is green in any order. Do not add retries, and do not weaken an assertion.
  • [ ] bun scripts/flake-hunt.ts admit test/reviews.test.ts is 10/10 green, and its record is in the pull.
  • [ ] The file's row moves from EXCLUDED to GATE in scripts/gate.ts in the same pull.
Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs