For: a fast, honest suite (#1242): scripts/test-times.json records test/apps-adversarial-rules.test.ts at 29.7 s, among the slowest files in the gate.
Done:
[ ] bun test test/apps-adversarial-rules.test.ts finishes under 10 s on the vxrail and every test in it under 1 s; scripts/test-times.json records the new time.
[ ] Every behaviour the file proves today is still proved: no test is deleted without naming, in the pull, the test that now covers it.
[ ] Fixtures are built once per file (one forge, one git fixture), clocks are faked where the file waits on time, and nothing sleeps.
Files: test/apps-adversarial-rules.test.ts, scripts/test-times.json; production code only if a wait there is the cause, said in the pull.
Risk: a faster test that proves less. Run it ten times in a row green before pushing.
Not this issue: any other test file.
For: a fast, honest suite (#1242):
scripts/test-times.jsonrecords test/apps-adversarial-rules.test.ts at 29.7 s, among the slowest files in the gate.Done:
bun test test/apps-adversarial-rules.test.tsfinishes under 10 s on the vxrail and every test in it under 1 s;scripts/test-times.jsonrecords the new time.Files:
test/apps-adversarial-rules.test.ts,scripts/test-times.json; production code only if a wait there is the cause, said in the pull. Risk: a faster test that proves less. Run it ten times in a row green before pushing. Not this issue: any other test file.