From 6fa7e847616e5b3f7f8a8972ad5304e5cbf787d5 Mon Sep 17 00:00:00 2001
From: Ethan <39577870+ethanndickson@users.noreply.github.com>
Date: Thu, 7 May 2026 17:56:17 +1000
Subject: [PATCH] test(coderd/x/chatd): skip
 TestExploreChatSendMessageCannotMutateMCPSnapshot (#25023)

Skips `TestExploreChatSendMessageCannotMutateMCPSnapshot` while the
chatd redesign is in flight. The test exposes a self-interrupt race in
`processChat`'s control-pubsub subscriber that is structurally fixed by
the redesign in #24444; skipping until then matches the existing
`TestSubscribeRelayEstablishedMidStream` skip in
`enterprise/coderd/x/chatd/chatd_test.go`.

Relates to https://github.com/coder/internal/issues/1493.
---
 coderd/x/chatd/chatd_test.go | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/coderd/x/chatd/chatd_test.go b/coderd/x/chatd/chatd_test.go
index eb6501447b..03bf22c092 100644
--- a/coderd/x/chatd/chatd_test.go
+++ b/coderd/x/chatd/chatd_test.go
@@ -1092,6 +1092,8 @@ func TestRootExploreChatExcludesWebSearchProviderToolAtRuntime(t *testing.T) {
 
 func TestExploreChatSendMessageCannotMutateMCPSnapshot(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/1493")
 
 	db, ps := dbtestutil.NewDB(t)
 	ctx := testutil.Context(t, testutil.WaitLong)