From 526cda9ebb490ea079065852d0212db054e072d7 Mon Sep 17 00:00:00 2001 From: sneak Date: Thu, 1 Oct 2026 23:52:17 +0000 Subject: [PATCH] Raise the go test timeout so internal/handlers fits With the two races fixed, internal/handlers still hit the 30s -timeout in the Docker build. Its time is real work, not clock waits: each test starts its own server and in-memory database, and each new session writes its welcome lines to the database one row at a time, which the race detector slows down. The only clock waits are the one-second long poll and a 200ms pause. Measured in docker build --no-cache .: internal/handlers took 21-30s under -race, and over 30s in the review builds that timed out. Both go test runs in the test target now use -timeout 120s. The retry stays (https://git.eeqj.de/sneak/neoirc/issues/101). Model: opus-5-5 --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 67ceb3e..f00e4b7 100644 --- a/Makefile +++ b/Makefile @@ -32,7 +32,7 @@ fmt-check: @test -z "$$(gofmt -l .)" || (echo "Files not formatted:" && gofmt -l . && exit 1) test: ensure-web-dist - go test -timeout 30s -race -cover ./... || go test -timeout 30s -race -v ./... + go test -timeout 120s -race -cover ./... || go test -timeout 120s -race -v ./... # check runs all validation without making changes # Used by CI and Docker build — fails if anything is wrong