From 73d8cd52798bde05491bb51078ad3ea838fa86f0 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 7 Mar 2026 07:59:57 +0000
Subject: [PATCH 1/3] Initial plan


From 4d99d0e4cb4b988d0dce0c84758bb5295ccc1969 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 7 Mar 2026 08:11:05 +0000
Subject: [PATCH 2/3] Fix flaky Playwright tests: increase timeout, wait for
 logout nav, use unique retry data names

Co-authored-by: MichaelKohler <330324+MichaelKohler@users.noreply.github.com>
---
 e2e/account.spec.ts  | 38 +++++++++++++++++++-------------------
 e2e/shared-steps.ts  |  1 +
 playwright.config.ts |  2 +-
 3 files changed, 21 insertions(+), 20 deletions(-)

diff --git a/e2e/account.spec.ts b/e2e/account.spec.ts
index 16c8c2a..0c2abdb 100644
--- a/e2e/account.spec.ts
+++ b/e2e/account.spec.ts
@@ -5,51 +5,51 @@ test.beforeEach(async ({ page }) => {
   await page.goto("/");
 });
 
-test("Account flows", async ({ page }) => {
+test("Account flows", async ({ page }, testInfo) => {
+  const suffix = testInfo.retry > 0 ? `-retry${testInfo.retry}` : "";
+  const groupName = `Test Group for account${suffix}`;
+  const typeName = `Test Type for account${suffix}`;
+  const accountName = `Test Account for account creation${suffix}`;
+  const editedName = `Test Account Edited${suffix}`;
+
   // Login
   await login(page);
 
   // Create type and group so it can be chosen
-  await createGroup(page, "Test Group for account");
-  await createType(page, "Test Type for account");
+  await createGroup(page, groupName);
+  await createType(page, typeName);
 
   // Navigate to accounts page
   await page.getByRole("link", { name: "Accounts" }).click();
 
   // Create
   await page.getByRole("button", { name: "+ New Account" }).click();
-  await page.getByLabel("Name:").fill("Test Account for account creation");
-  await page
-    .getByLabel("Type:")
-    .selectOption({ label: "Test Type for account" });
-  await page
-    .getByLabel("Group:")
-    .selectOption({ label: "Test Group for account" });
+  await page.getByLabel("Name:").fill(accountName);
+  await page.getByLabel("Type:").selectOption({ label: typeName });
+  await page.getByLabel("Group:").selectOption({ label: groupName });
   await page.getByRole("button", { name: "Save" }).click();
-  await expect(
-    page.getByText("Test Account for account creation").first()
-  ).toBeVisible();
+  await expect(page.getByText(accountName).first()).toBeVisible();
 
   // Edit
   await page
-    .getByRole("row", { name: /Test Account for account creation/ })
+    .getByRole("row", { name: new RegExp(accountName) })
     .getByRole("button", { name: "Edit" })
     .first()
     .click();
-  await page.getByLabel("Name:").fill("Test Account Edited");
+  await page.getByLabel("Name:").fill(editedName);
   await page.getByRole("button", { name: "Save" }).click();
-  await expect(page.getByText("Test Account Edited").first()).toBeVisible();
+  await expect(page.getByText(editedName).first()).toBeVisible();
 
   // Delete
   await page
-    .getByRole("row", { name: /Test Account Edited/ })
+    .getByRole("row", { name: new RegExp(editedName) })
     .getByRole("button", { name: "X" })
     .first()
     .click();
   await page
-    .getByRole("row", { name: /Test Account Edited/ })
+    .getByRole("row", { name: new RegExp(editedName) })
     .getByRole("button", { name: "X?" })
     .first()
     .click();
-  await expect(page.getByText("Test Account Edited").first()).not.toBeVisible();
+  await expect(page.getByText(editedName).first()).not.toBeVisible();
 });
diff --git a/e2e/shared-steps.ts b/e2e/shared-steps.ts
index 0d56fe5..f7e325e 100644
--- a/e2e/shared-steps.ts
+++ b/e2e/shared-steps.ts
@@ -27,6 +27,7 @@ export async function login(
  */
 export async function logout(page: Page) {
   await page.getByRole("button", { name: "Logout" }).click();
+  await page.waitForURL("/");
   await expect(page.getByRole("link", { name: "Log In" })).toBeVisible();
 }
 
diff --git a/playwright.config.ts b/playwright.config.ts
index d5502bf..f523a11 100644
--- a/playwright.config.ts
+++ b/playwright.config.ts
@@ -7,7 +7,7 @@ import { devices } from "@playwright/test";
 const config: PlaywrightTestConfig = {
   testDir: "./e2e",
   /* Maximum time one test can run for. */
-  timeout: 30 * 1000,
+  timeout: 60 * 1000,
   expect: {
     /**
      * Maximum time expect() should wait for the condition to be met.

From 3eef9326aaa87104c896ab10a2a04d2878210d5b Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 7 Mar 2026 08:40:59 +0000
Subject: [PATCH 3/3] Fix logout waitForURL to match actual redirect
 destination (/login not /)

Co-authored-by: MichaelKohler <330324+MichaelKohler@users.noreply.github.com>
---
 e2e/shared-steps.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/e2e/shared-steps.ts b/e2e/shared-steps.ts
index f7e325e..702c573 100644
--- a/e2e/shared-steps.ts
+++ b/e2e/shared-steps.ts
@@ -27,7 +27,7 @@ export async function login(
  */
 export async function logout(page: Page) {
   await page.getByRole("button", { name: "Logout" }).click();
-  await page.waitForURL("/");
+  await page.waitForURL("**/login**");
   await expect(page.getByRole("link", { name: "Log In" })).toBeVisible();
 }