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

signin-throttle.test.ts takes ~58 s alone and times out at 60 s under load #1070

shipped production is serving it opened this has it
opened this

Replaces #864, parked twice:

  • the check failed ("capacity: only a forge admin reads or changes it > an admin changes each one from the page, the API and MCP; a wrong value is refused; every change is kept") and the decider saw no point in another try (pull #1013 closed). Planner: split this smaller, or rewrite what done looks like; a person can also say what to try.
  • the check failed ("every open pull's preview on the studio page > a change's page names its preview's data as its own copy and gives the moment it was taken; production's card does not") and the decider saw no point in another try (pull #1036 closed).

Those checks went red on tests this change does not touch. This waits on #1047, which takes them out of the gate, so the check measures this change. Start from today's main. What done looks like is #864's, unchanged:

---

Found while admitting test/[p-s]*.test.ts (#563). bun test test/signin-throttle.test.ts --timeout 60000 passes alone in ~58 s (twenty wrong guesses, each paying the full password hash) and fails with a timeout when other files run beside it. Make the test cheap (lower hash cost in the test, or fewer guesses via an injectable limit) so it fits well inside the bound, then admit it to GATE in scripts/gate.ts.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs