From 57b10f3940347316f3999ddbf539eacf8183d1be Mon Sep 17 00:00:00 2001
From: Thomas Petersen <thomasp@squareup.com>
Date: Mon, 4 May 2026 13:56:00 -0400
Subject: [PATCH] Update thread smoke coverage

Co-authored-by: Cursor <cursoragent@cursor.com>
---
 .../src/features/messages/ui/MessageThreadPanel.tsx |  2 +-
 desktop/tests/e2e/messaging.spec.ts                 | 13 ++++++++++++-
 desktop/tests/e2e/smoke.spec.ts                     |  2 +-
 3 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/desktop/src/features/messages/ui/MessageThreadPanel.tsx b/desktop/src/features/messages/ui/MessageThreadPanel.tsx
index a4fb0eb0b1..672fd77b31 100644
--- a/desktop/src/features/messages/ui/MessageThreadPanel.tsx
+++ b/desktop/src/features/messages/ui/MessageThreadPanel.tsx
@@ -219,7 +219,7 @@ export function MessageThreadPanel({
             >
               {threadReplies.length > 0 ? (
                 <div className="space-y-2">
-                  {threadReplies.map((entry, index) => {
+                  {threadReplies.map((entry) => {
                     return (
                       <div key={entry.message.id}>
                         <MessageRow
diff --git a/desktop/tests/e2e/messaging.spec.ts b/desktop/tests/e2e/messaging.spec.ts
index 7bb6494ff0..0380cd8cd1 100644
--- a/desktop/tests/e2e/messaging.spec.ts
+++ b/desktop/tests/e2e/messaging.spec.ts
@@ -424,7 +424,8 @@ test("opens a single-level thread panel with inline expansion", async ({
   const firstReplySummaryRow = threadReplies.locator(
     `[data-thread-head-id="${firstReplyId}"]`,
   );
-  await expect(firstReplySummaryRow).toHaveCount(0);
+  await expect(firstReplySummaryRow).toHaveCount(1);
+  await expect(firstReplySummaryRow).toContainText("2 replies");
 
   await expect(rootSummaryRow).toContainText("18 replies");
   await expect(
@@ -447,6 +448,16 @@ test("opens a single-level thread panel with inline expansion", async ({
       });
     })
     .toBeLessThan(160);
+
+  await firstReplySummaryRow.click();
+  await expect(
+    threadReplies.getByTestId("message-row").filter({ hasText: nestedReply }),
+  ).toHaveCount(0);
+  await expect(
+    threadReplies
+      .getByTestId("message-row")
+      .filter({ hasText: nestedReplyFromBob }),
+  ).toHaveCount(0);
 });
 
 test("thread panel width uses session storage and reset handle", async ({
diff --git a/desktop/tests/e2e/smoke.spec.ts b/desktop/tests/e2e/smoke.spec.ts
index 26b54da329..6876f8a230 100644
--- a/desktop/tests/e2e/smoke.spec.ts
+++ b/desktop/tests/e2e/smoke.spec.ts
@@ -361,7 +361,7 @@ test("supports multiline drafts with Ctrl+Enter and sends with Enter", async ({
   await page.goto("/");
   await page.getByTestId("channel-general").click();
   await expect(page.getByTestId("chat-title")).toHaveText("general");
-  await expect(page.getByTestId("send-message")).toContainText("Send");
+  await expect(page.getByRole("button", { name: "Send message" })).toBeVisible();
   const initialInputHeight = await input.evaluate(
     (element) => (element as HTMLElement).clientHeight,
   );