From 73af41479309c598ce4e39940ed4224471caea69 Mon Sep 17 00:00:00 2001 From: Jakub Zych Date: Sun, 11 Oct 2026 14:35:56 +0200 Subject: [PATCH] docs: correct the Docker note for go test -short in CLAUDE.md --- CLAUDE.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/CLAUDE.md b/CLAUDE.md index 467b35a..684cbdb 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -14,7 +14,7 @@ References: the WinterCMS starter at `../examples/golem15-wintercms-starter`, th ```sh go vet ./... -go test -short ./... # fast loop, no Docker +go test -short ./... # fast loop (see the Docker note below) go test ./... # also the testcontainers Postgres/Mailpit tests (needs Docker) go test -short ./modules/cabana -run TestName # one test cd examples/hello && go test ./... # separate workspace module, not covered by the root ./... @@ -31,7 +31,7 @@ scripts/check-admin-dist.sh # committed modules/boardwalk/dist scripts/check-phaseN.sh # per-phase verification gates ``` -Docker-backed tests skip themselves under `testing.Short()`. Keep `go vet` and `go test ./...` green at every commit. +`go test -short` still needs Docker for beachcomber, cabana, conga, lagoon and lighthouse: their `TestMain` starts a Postgres container regardless of `-short`. If the Docker socket is denied although the user is in the `docker` group, the shell session predates the group; run the tests through `sudo -u golem` to pick up a fresh group list. Keep `go vet` and `go test ./...` green at every commit. ## Architecture