From 378dc41c05893da5476bde971c4a31afcc25c44d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ullrich=20Sch=C3=A4fer?= Date: Sat, 11 Apr 2026 03:05:55 +0200 Subject: [PATCH] Fix E2E: notes tab test uses CM placeholder instead of textarea CodeMirror renders placeholders as .cm-placeholder elements, not HTML placeholder attributes. Updated the sidebar tabs test to look for the CodeMirror content area instead. Co-Authored-By: Claude Opus 4.6 (1M context) --- e2e/planner.test.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/e2e/planner.test.ts b/e2e/planner.test.ts index 814e47c..6b62be8 100644 --- a/e2e/planner.test.ts +++ b/e2e/planner.test.ts @@ -76,7 +76,8 @@ test.describe("Planner", () => { // Switch to Notes tab await page.getByRole("button", { name: "Notes" }).click(); - await expect(page.getByPlaceholder("Add notes for this session...")).toBeVisible(); + // CodeMirror renders placeholder as a .cm-placeholder element, not an HTML placeholder attribute + await expect(page.locator(".cm-placeholder, .cm-content")).toBeVisible(); // Switch back to Waypoints tab await page.getByRole("button", { name: "Waypoints" }).click();