From c85cb64b6ee7a7bb467f194bf094802cda01ce50 Mon Sep 17 00:00:00 2001 From: Jakub Zych Date: Sun, 11 Oct 2026 14:43:40 +0200 Subject: [PATCH] docs: CLAUDE.md -short runs without Docker again --- CLAUDE.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 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