diff --git a/CLAUDE.md b/CLAUDE.md index 684cbdb..fc2ba23 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 (see the Docker note below) +go test -short ./... # fast loop, no Docker 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 ``` -`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. +Docker-backed tests skip under `-short`; a `TestMain` that starts containers must call `flag.Parse()` and check `testing.Short()` (go test passes `-test.short=true`, so scanning `os.Args` for `-test.short` misses it). 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