From 9f9a280fc7534fd736c0be13c20a86b7f6b28bcc Mon Sep 17 00:00:00 2001 From: QA Runner Date: Mon, 20 Jul 2026 09:25:34 -0700 Subject: [PATCH] test(e2e): update built-in workflow id after upstream rename The workflows UI refresh (olp/main fa21ac8) renamed the built-in workflow id builtin-cp-checklist -> builtin-draft-cp-checklist (title 'Draft CP Checklist' unchanged). The read-only-detail spec navigated to the old id URL and got a 'workflow not found' page. Point it at the new id. Co-Authored-By: Claude Opus 4.8 (1M context) --- e2e/workflows-account.spec.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/e2e/workflows-account.spec.ts b/e2e/workflows-account.spec.ts index e21b46bf..9b9d1722 100644 --- a/e2e/workflows-account.spec.ts +++ b/e2e/workflows-account.spec.ts @@ -6,7 +6,7 @@ * Key source facts used by these selectors: * - WorkflowList.tsx: h1 "Workflows"; Plus icon button (no aria-label) opens NewWorkflowModal * - NewWorkflowModal.tsx: placeholder "Workflow name"; submit button text "Create workflow" - * - systemWorkflows.ts (generated): built-in id "builtin-cp-checklist", title "Draft CP Checklist" + * - systemWorkflows.ts (generated): built-in id "builtin-draft-cp-checklist", title "Draft CP Checklist" * - WorkflowDetailPage ([id]/page.tsx): readOnly badge renders Read-only; * WorkflowPromptEditor passes editable:!readOnly to Tiptap → contenteditable="false" when readOnly * - WorkflowPromptEditor.tsx: editorProps class = "workflow-editor-content" on the ProseMirror div @@ -81,7 +81,7 @@ test.describe("Workflows", () => { ).toBeVisible({ timeout: 10_000 }); // System workflows are generated into backend/src/lib/systemWorkflows.ts — - // "Draft CP Checklist" (id: builtin-cp-checklist) is always present + // "Draft CP Checklist" (id: builtin-draft-cp-checklist) is always present // is always present; its title appears as a row in the table. // REGRESSION: fails if the workflow list page or built-in workflow rendering is broken await expect(page.getByText("Draft CP Checklist")).toBeVisible({ @@ -134,11 +134,11 @@ test.describe("Workflows", () => { page, }) => { // Navigate directly to the known built-in ID; this avoids having to click - // through the DisplayWorkflowModal "View Page" button. builtin-cp-checklist + // through the DisplayWorkflowModal "View Page" button. builtin-draft-cp-checklist // is an assistant-type workflow, so use the typed detail route // (/workflows/assistant/[id]) that the app itself links to via // workflowDetailPath — the flat /workflows/[id] path does not exist here. - await page.goto("/workflows/assistant/builtin-cp-checklist"); + await page.goto("/workflows/assistant/builtin-draft-cp-checklist"); // The page loads and shows the built-in workflow title await expect(page.getByText("Draft CP Checklist")).toBeVisible({