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

The audit paging test depends on other tests' rows: bun test --randomize can fail it #86

open not picked up yet opened this
opened this

the audit trail is an admin's, and it pages from a cursor in src/server.test.ts asserts that /admin/audit offers an older link. That link only appears past PAGE = 100 rows in src/audit.ts, and the test writes none of them: it is reading the rows every other test in the file happened to leave behind. In the default order there are enough; under bun test --randomize there need not be, and the test fails with expected older toBeDefined (seen on seed 2745955017).

AGENTS.md asks that the suite pass in any order, so the fix is for that test to make the trail it pages through — enough rows of its own before it asks for a page — or for the page size to be something the request can ask for. Nothing else in the file looked order-dependent, but --randomize a few times is the way to find out.

Found while adding the body deadline in #54; unrelated to it.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs