internal/handlers tests make make test miss its 20-second target #113

Open
opened 2026-10-06 01:32:43 +02:00 by clawbot · 0 comments
Collaborator

REPO_POLICIES.md sets a target of under 20 seconds for make test. On next (729c671) the go test -race run inside the Docker build spends about 29 seconds in internal/handlers alone; every other package finishes in about 2 seconds or less. The suite is still under the policy's 60-second hard cap, so this is an improvement, not a red gate.

Definition of done

  1. Find where internal/handlers tests spend their time and remove that cost at its cause (for example a wait, a long-poll timeout or per-test setup that does not need to be that slow), so the whole go test -race run in the Docker build finishes in under 20 seconds.
  2. No test is removed, skipped or weakened, -race stays, and the test timeout is not changed.
  3. One reviewed PR against next, landed after #112, which moves the tests into their own Docker phase where this is measured.

Model: opus-5-5

`REPO_POLICIES.md` sets a target of under 20 seconds for `make test`. On `next` (`729c671`) the `go test -race` run inside the Docker build spends about 29 seconds in `internal/handlers` alone; every other package finishes in about 2 seconds or less. The suite is still under the policy's 60-second hard cap, so this is an improvement, not a red gate. ## Definition of done 1. Find where `internal/handlers` tests spend their time and remove that cost at its cause (for example a wait, a long-poll timeout or per-test setup that does not need to be that slow), so the whole `go test -race` run in the Docker build finishes in under 20 seconds. 2. No test is removed, skipped or weakened, `-race` stays, and the test timeout is not changed. 3. One reviewed PR against `next`, landed after https://git.eeqj.de/sneak/neoirc/issues/112, which moves the tests into their own Docker phase where this is measured. Model: opus-5-5
clawbot self-assigned this 2026-10-06 01:32:43 +02:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/neoirc#113