From b9bc0ad6df2ff4726944cf468d859974ba236b7f Mon Sep 17 00:00:00 2001 From: Ethan <39577870+ethanndickson@users.noreply.github.com> Date: Thu, 16 Apr 2026 23:55:41 +1000 Subject: [PATCH] test: skip TestSubscribeRelayEstablishedMidStream (#24431) Relates to https://github.com/coder/internal/issues/1455 From that issue: > Going to skip this test until the underlying race in chatd is fixed. https://github.com/coder/coder/pull/24279 was a band-aid fix that I no longer think is valuable pursuing short term. Hugo is working on a RFC for a redesign of the system to prevent the class of race condition into the future. --- enterprise/coderd/x/chatd/chatd_test.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/enterprise/coderd/x/chatd/chatd_test.go b/enterprise/coderd/x/chatd/chatd_test.go index fa44ea9d8a..f192995e91 100644 --- a/enterprise/coderd/x/chatd/chatd_test.go +++ b/enterprise/coderd/x/chatd/chatd_test.go @@ -1435,6 +1435,8 @@ func TestSubscribeRelayDialCanceledOnFastCompletion(t *testing.T) { // condition where the relay is too slow. func TestSubscribeRelayEstablishedMidStream(t *testing.T) { t.Parallel() + // TODO(hugodutka): Unskip when chatd is free of race conditions. + t.Skip("skipped due to inherent race condition; see https://github.com/coder/internal/issues/1455") db, ps := dbtestutil.NewDB(t) workerID := uuid.New()