From 67395428752f0a754439e3625b5882942eb060d5 Mon Sep 17 00:00:00 2001
From: Michael Suchacz <203725896+ibetitsmike@users.noreply.github.com>
Date: Sat, 23 May 2026 01:10:31 +0200
Subject: [PATCH] test(coderd/x/chatd): skip signal wake send flake (#25633)

Skips `TestSignalWakeSendMessage`, which flakes because the current
chatd control notification flow can deliver stale status notifications
after a new processing run starts.

This mirrors the existing CODAGT-353 skips for the same
stale-notification class and leaves the deterministic fix to that
notification-flow refactor.

Refs
https://linear.app/codercom/issue/ENG-2727/flake-testsignalwakesendmessage

> Generated by Coder Agents on behalf of @ibetitsmike.
---
 coderd/x/chatd/chatd_test.go | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/coderd/x/chatd/chatd_test.go b/coderd/x/chatd/chatd_test.go
index f35aba147a..5d1f373cd9 100644
--- a/coderd/x/chatd/chatd_test.go
+++ b/coderd/x/chatd/chatd_test.go
@@ -8372,6 +8372,9 @@ func TestSignalWakeImmediateAcquisition(t *testing.T) {
 // triggers immediate processing via signalWake.
 func TestSignalWakeSendMessage(t *testing.T) {
 	t.Parallel()
+	// TODO(CODAGT-353): Re-enable this after the chatd notification
+	// flow can distinguish stale status notifications from interrupts.
+	t.Skip("skipped until chatd notification flow refactor handles stale control notifications")
 
 	db, ps := dbtestutil.NewDB(t)
 	ctx := testutil.Context(t, testutil.WaitSuperLong)