Skip to content

test: fix flow state clean test on windows #13013

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Cristhianzl merged 1 commit into from
May 7, 2026
Merged

test: fix flow state clean test on windows #13013

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
44 changes: 30 additions & 14 deletions src/frontend/tests/core/features/user-flow-state-cleanup.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,17 +44,19 @@ test(
await page.evaluate(() => {
sessionStorage.removeItem("testMockAutoLogin");
});
await page.getByRole("button", { name: "Sign In" }).click();

// Create User A — wait for the homepage Loading state to clear before
// checking mainpage_title (mainpage_title only renders after data load,
// which can outlast a 30s wait on slower runners like Windows CI).
await page.waitForSelector('text="Loading"', {
state: "hidden",
timeout: 60000,
});
await Promise.all([
page.waitForResponse(
(response) =>
response.url().includes("/api/v1/login") && response.status() === 200,
{ timeout: 60000 },
),
page.getByRole("button", { name: "Sign In" }).click(),
]);

// mainpage_title only renders after the homepage data finishes loading,
// and on slower runners (Windows CI) this can outlast a 60s wait.
await page.waitForSelector('[data-testid="mainpage_title"]', {
timeout: 60000,
timeout: 90000,
});
await page.getByTestId("user-profile-settings").click();
await page.getByText("Admin Page", { exact: true }).click();
Expand Down Expand Up @@ -88,10 +90,17 @@ test(
await page.evaluate(() => {
sessionStorage.removeItem("testMockAutoLogin");
});
await page.getByRole("button", { name: "Sign In" }).click();
await Promise.all([
page.waitForResponse(
(response) =>
response.url().includes("/api/v1/login") && response.status() === 200,
{ timeout: 60000 },
),
page.getByRole("button", { name: "Sign In" }).click(),
]);

// Create a flow for User A
await page.waitForSelector('[id="new-project-btn"]', { timeout: 30000 });
await page.waitForSelector('[id="new-project-btn"]', { timeout: 60000 });
// Check that User A starts with an empty flows list
expect(
(
Expand Down Expand Up @@ -129,7 +138,7 @@ test(
await basicPromptingHeading.click();
try {
await page.waitForSelector('[data-testid="canvas_controls_dropdown"]', {
timeout: attempt === maxClickAttempts ? 180000 : 45000,
timeout: attempt === maxClickAttempts ? 240000 : 60000,
});
canvasMounted = true;
break;
Expand Down Expand Up @@ -172,7 +181,14 @@ test(
await page.evaluate(() => {
sessionStorage.removeItem("testMockAutoLogin");
});
await page.getByRole("button", { name: "Sign In" }).click();
await Promise.all([
page.waitForResponse(
(response) =>
response.url().includes("/api/v1/login") && response.status() === 200,
{ timeout: 60000 },
),
page.getByRole("button", { name: "Sign In" }).click(),
]);

// Verify admin can't see User A's flow
await expect(page.getByText(userAFlowName, { exact: true })).toBeVisible({
Expand Down
Loading