From dfa0e8a56665973668fd7830478bbd02d6595ea0 Mon Sep 17 00:00:00 2001 From: BaaRai <130491490+BaaRai@users.noreply.github.com> Date: Sat, 14 Nov 2026 19:11:10 +0100 Subject: [PATCH] ci: run ./server under the race detector The server suite now passes go test -race against postgres, and its test teardown no longer leaks live servers (#227, #228). Add ./server to the go test -race step; the DSN and -p 1 are unchanged. --- .github/workflows/ci.yml | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index ac6e007..bbe5ad4 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -45,8 +45,9 @@ jobs: # keeps the database state deterministic. run: go test -p 1 ./... - name: go test -race - # Race detector on the packages that pass under -race today. ./server is - # left out until it has been checked with -race against postgres. + # Race detector on the packages that pass under -race today. ./server + # joined once its suite was checked with -race against postgres and its + # test teardown fixed (#227, #228). # Its own database: the plain run above leaves rows behind and the db tests # are not idempotent across runs (duplicate llm_profiles names). env: @@ -54,7 +55,8 @@ jobs: run: >- go test -race -p 1 ./agent ./cmd/artex ./config ./db ./evidence ./guard - ./intercept ./llmpool ./llmrec ./mcphttp ./notify ./sidequestion ./traffic + ./intercept ./llmpool ./llmrec ./mcphttp ./notify ./server ./sidequestion + ./traffic shell: runs-on: ubuntu-latest