increate user flow state timeout

Author: Cristhianzl

src/frontend/tests/core/features/traces.spec.ts

@@ -20,7 +20,19 @@ test(
     await awaitBootstrapTest(page);
 
     await page.getByTestId("side_nav_options_all-templates").click();
+    // Tie the wait to the actual flow-creation network response. On Windows
+    // CI the click → canvas mount can outlast a 100s wait when the backend
+    // is under load, but if the POST /flows succeeds we know the page will
+    // navigate; we then wait for the canvas with a generous budget.
+    const flowCreatePromise = page.waitForResponse(
+      (response) =>
+        /\/api\/v1\/flows\/?(?:\?|$)/.test(response.url()) &&
+        response.request().method() === "POST" &&
+        response.status() === 201,
+      { timeout: 120000 },
+    );
     await page.getByRole("heading", { name: "Basic Prompting" }).click();
+    await flowCreatePromise;
     await page.waitForSelector('[data-testid="canvas_controls_dropdown"]', {
       timeout: 100000,
     });
@@ -76,7 +88,19 @@ test.skip(
     await awaitBootstrapTest(page);
 
     await page.getByTestId("side_nav_options_all-templates").click();
+    // Tie the wait to the actual flow-creation network response. On Windows
+    // CI the click → canvas mount can outlast a 100s wait when the backend
+    // is under load, but if the POST /flows succeeds we know the page will
+    // navigate; we then wait for the canvas with a generous budget.
+    const flowCreatePromise = page.waitForResponse(
+      (response) =>
+        /\/api\/v1\/flows\/?(?:\?|$)/.test(response.url()) &&
+        response.request().method() === "POST" &&
+        response.status() === 201,
+      { timeout: 120000 },
+    );
     await page.getByRole("heading", { name: "Basic Prompting" }).click();
+    await flowCreatePromise;
     await page.waitForSelector('[data-testid="canvas_controls_dropdown"]', {
       timeout: 100000,
     });

src/frontend/tests/core/features/user-flow-state-cleanup.spec.ts

@@ -123,35 +123,19 @@ test(
     await page.waitForSelector('[data-testid="modal-title"]', {
       timeout: 30000,
     });
-    await page.getByTestId("side_nav_options_all-templates").click();
-
-    const basicPromptingHeading = page.getByRole("heading", {
-      name: "Basic Prompting",
-    });
-    await basicPromptingHeading.waitFor({ state: "visible", timeout: 30000 });
-
-    // Retry the template click if the canvas never mounts: on Windows CI the
-    // first click occasionally lands before the template grid is interactive.
-    let canvasMounted = false;
-    const maxClickAttempts = 3;
-    for (let attempt = 1; attempt <= maxClickAttempts; attempt++) {
-      await basicPromptingHeading.click();
-      try {
-        await page.waitForSelector('[data-testid="canvas_controls_dropdown"]', {
-          timeout: attempt === maxClickAttempts ? 240000 : 60000,
-        });
-        canvasMounted = true;
-        break;
-      } catch (_error) {
-        if (attempt === maxClickAttempts) throw _error;
-        const modalStillOpen =
-          (await page.getByTestId("modal-title").count()) > 0;
-        if (!modalStillOpen) throw _error;
-      }
-    }
-    if (!canvasMounted) {
-      throw new Error("canvas_controls_dropdown never appeared");
-    }
+
+    // Use blank-flow instead of the Basic Prompting template. The template
+    // path provisions multiple components on the backend and, on Windows CI
+    // shards under load, the new-flow canvas can stay un-mounted past 240s.
+    // This cleanup test only needs *some* flow owned by the user, so a blank
+    // flow is equivalent in scope while avoiding the Windows-specific stall.
+    await page.waitForSelector('[data-testid="blank-flow"]', {
+      timeout: 30000,
+    });
+    await page.getByTestId("blank-flow").click();
+    await page.waitForSelector('[data-testid="canvas_controls_dropdown"]', {
+      timeout: 60000,
+    });
 
     await renameFlow(page, { flowName: userAFlowName });