From 371d2970657086a3313a171129c1ad69513ff8ff Mon Sep 17 00:00:00 2001
From: yousefed <yousefdardiry@gmail.com>
Date: Sun, 11 Oct 2026 10:52:45 +0200
Subject: [PATCH] test(comments): fix flaky thread-mark clicks

ProseMirror counts a click within 500ms and 10px of the previous click in
the editor as a double/triple click and skips handleClick, so the thread
never opened. Clicks on the comment UI in between are outside the editor,
so clicking the mark's centre right after double-clicking the word was a
race. Click near the mark's left edge instead.
---
 .../src/end-to-end/comments/comments.test.tsx | 19 +++++++++++++++++--
 1 file changed, 17 insertions(+), 2 deletions(-)

diff --git a/tests/src/end-to-end/comments/comments.test.tsx b/tests/src/end-to-end/comments/comments.test.tsx
index fd7303ad03..214e1cec32 100644
--- a/tests/src/end-to-end/comments/comments.test.tsx
+++ b/tests/src/end-to-end/comments/comments.test.tsx
@@ -17,6 +17,21 @@ async function doubleClickElement(element: Element) {
   await clickAt(x + width / 2, y + height / 2, 2);
 }
 
+/**
+ * Clicks the comment mark near its left edge. ProseMirror counts a click within
+ * 500ms and 10px of the previous one in the editor as a double/triple click
+ * (which skips `handleClick`, so no thread opens). The clicks on the comment
+ * UI in between happen outside the editor and don't reset that count, so
+ * clicking the centre right after double-clicking the word to select it is
+ * flaky.
+ */
+async function clickThreadMark() {
+  const { x, y, height } = getRect(
+    await waitForSelector("span.bn-thread-mark"),
+  );
+  await clickAt(x + 2, y + height / 2);
+}
+
 /** Resolves once exactly `count` elements match `selector`. */
 function expectSelectorCount(selector: string, count: number) {
   return vi.waitFor(() => {
@@ -90,7 +105,7 @@ describe("Check Comments functionality", () => {
 
     await userEvent.keyboard("{Enter}");
     await expectSelectorCount(".bn-thread", 0);
-    await userEvent.click(await waitForSelector("span.bn-thread-mark"));
+    await clickThreadMark();
     await expectSelectorCount(".bn-thread-comment", 1);
     expect(document.querySelector(".bn-thread-comment")?.textContent).toContain(
       "second line",
@@ -131,7 +146,7 @@ describe("Check Comments functionality", () => {
     // Wait for comment composer to close.
     await expectSelectorCount(".bn-thread", 0);
 
-    await userEvent.click(await waitForSelector("span.bn-thread-mark"));
+    await clickThreadMark();
     await expectElement(
       await waitForSelector(".bn-thread-comment"),
     ).toBeVisible();