diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 7a8737c..b3d18fe 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -37,13 +37,13 @@ jobs: # fresh database. # - TestSideUsageRecordedOnceOnConsumerCancellation: concurrency/cancellation # test that passes locally but is timing-sensitive on 2-core CI runners. - # Skipping keeps the baseline green so real regressions stand out. + # They now skip themselves in code (t.Skip with a pointer to the issue). - name: go test # -p 1: package test suites share the single postgres service; running # them in parallel makes packages delete each other's rows (observed # as rotating failures in the agent package). Serial package execution # keeps the database state deterministic. - run: go test -p 1 ./... -skip 'TestInterceptFilterHTTP|TestGraphOverviewExpansAssociatedCompanyScope|TestCaptureApprovalLifecycle|TestSideUsageRecordedOnceOnConsumerCancellation' + run: go test -p 1 ./... web: runs-on: ubuntu-latest diff --git a/agent/blackboard_inheritance_test.go b/agent/blackboard_inheritance_test.go index 25e7c8f..94c099f 100644 --- a/agent/blackboard_inheritance_test.go +++ b/agent/blackboard_inheritance_test.go @@ -26,6 +26,7 @@ func callReadJSON(t *testing.T, tool actool.CoreTool, input string) any { } func TestGraphOverviewExpandsAssociatedCompanyScope(t *testing.T) { + t.Skip("known failure with a fresh database, pre-existing on unmodified upstream; tracked in BaaRai/ARTEX#9") d := testDB(t) defer d.Close() diff --git a/agent/capture_approval_test.go b/agent/capture_approval_test.go index 617fe56..706fc2b 100644 --- a/agent/capture_approval_test.go +++ b/agent/capture_approval_test.go @@ -17,6 +17,7 @@ import ( // Exercise the actual SDK event -> hook -> execution -> result path. No real // model or command is used; the probe tool only returns a fixed string. func TestCaptureApprovalLifecycle(t *testing.T) { + t.Skip("known failure with a fresh database, pre-existing on unmodified upstream; tracked in BaaRai/ARTEX#9") dsn, _, err := db.DSN() if err != nil { t.Skip("no test database configured") diff --git a/llmrec/llmrec_test.go b/llmrec/llmrec_test.go index eaec56d..2fd0e0e 100644 --- a/llmrec/llmrec_test.go +++ b/llmrec/llmrec_test.go @@ -83,6 +83,7 @@ func (meteredStreamProvider) Stream(context.Context, llm.CompletionRequest) iter } func TestSideUsageRecordedOnceOnConsumerCancellation(t *testing.T) { + t.Skip("timing-sensitive: passes locally, flaky on CI runners; tracked in BaaRai/ARTEX#9") dsn := os.Getenv("ARTEX_PG_DSN") if dsn == "" { t.Skip("requires isolated ARTEX_PG_DSN") diff --git a/server/intercept_filter_test.go b/server/intercept_filter_test.go index 432e23f..b948335 100644 --- a/server/intercept_filter_test.go +++ b/server/intercept_filter_test.go @@ -11,6 +11,7 @@ import ( ) func TestInterceptFilterHTTP(t *testing.T) { + t.Skip("known failure with a fresh database, pre-existing on unmodified upstream; tracked in BaaRai/ARTEX#9") dsn, _, err := db.DSN() if err != nil { t.Skip("no test database configured")