script/test always runs verbose and never measures coverage, against the policy pattern #315

New Issue

2026-09-21T09:20:20+02:00

clawbot commented

2026-09-21 09:20:20 +02:00

script/test runs go test -v -race -timeout 90s ./... unconditionally. REPO_POLICIES.md requires the conditional verbose rerun: a quiet first run with -cover, and a -v rerun only on failure, exiting non-zero regardless. Every green CI build and every local make check currently prints every test name, which buries the one line that matters when something is red.

Definition of done

script/test runs quietly with -race -cover -timeout 90s first and reruns with -v only when that fails, then exits 1.
The header comment in script/test about the 90s timeout survives as it stands.
Verified by running the script once green and once with a deliberately failing test, and confirming the second exits non-zero.

Model: fable-5-1

`script/test` runs `go test -v -race -timeout 90s ./...` unconditionally. `REPO_POLICIES.md` requires the conditional verbose rerun: a quiet first run with `-cover`, and a `-v` rerun only on failure, exiting non-zero regardless. Every green CI build and every local `make check` currently prints every test name, which buries the one line that matters when something is red. ## Definition of done - `script/test` runs quietly with `-race -cover -timeout 90s` first and reruns with `-v` only when that fails, then exits 1. - The header comment in `script/test` about the 90s timeout survives as it stands. - Verified by running the script once green and once with a deliberately failing test, and confirming the second exits non-zero. Model: fable-5-1

clawbot added this to the 1.0.0 milestone 2026-09-21 09:20:20 +02:00

clawbot referenced this issue

2026-09-21 09:20:58 +02:00

Review toward 1.0: identify and file all next steps #311

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sneak/webhooker#315