From c8d872f4e5780aba66169d3a324d987f4ad6d480 Mon Sep 17 00:00:00 2001 From: Tim Froehlich Date: Tue, 5 May 2026 15:51:38 -0500 Subject: [PATCH] =?UTF-8?q?test(e2e):=20unsilence=20PP-jsh=20and=20PP-v7g?= =?UTF-8?q?=20=E2=80=94=20verify=20PP-awg=20absorbs=20flakes?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Removes test.fixme calls for the Radix DropdownMenu (logout) and Radix Select (status) flake clusters. PP-awg (#1280) bumped Playwright's actionTimeout to 30s in CI; this experiment confirms whether that absorbed the portal mount timing variance. Affected: - e2e/full/machine-details-extended.spec.ts (4 tests using logout()) → PP-jsh - e2e/full/status-overhaul.spec.ts (1 test using Status Select) → PP-v7g If CI passes cleanly: PP-awg absorbed the flakes; close PP-jsh and PP-v7g. If CI flakes: revert this commit and file Layer 2 (per-spec helpers). Co-Authored-By: Claude Sonnet 4.6 --- e2e/full/machine-details-extended.spec.ts | 16 ---------------- e2e/full/status-overhaul.spec.ts | 4 ---- 2 files changed, 20 deletions(-) diff --git a/e2e/full/machine-details-extended.spec.ts b/e2e/full/machine-details-extended.spec.ts index 819d0e5c0..76e4e6b1c 100644 --- a/e2e/full/machine-details-extended.spec.ts +++ b/e2e/full/machine-details-extended.spec.ts @@ -27,10 +27,6 @@ test.describe("Machine Details - Extended", () => { test("should show description placeholder for owner", async ({ page, }, testInfo) => { - test.fixme( - true, - "PP-jsh — Radix DropdownMenu portal mount timing under CI load; awaiting verification of PP-awg/#1280 absorption" - ); // Login as admin (owner of Medieval Madness) await logout(page, testInfo); await loginAs(page, testInfo, { @@ -121,10 +117,6 @@ test.describe("Machine Details - Extended", () => { test("should show owner notes to machine owner", async ({ page, }, testInfo) => { - test.fixme( - true, - "PP-jsh — Radix DropdownMenu portal mount timing under CI load; awaiting verification of PP-awg/#1280 absorption" - ); // Login as admin (owns Medieval Madness) await logout(page, testInfo); await loginAs(page, testInfo, { @@ -151,10 +143,6 @@ test.describe("Machine Details - Extended", () => { test("should hide owner requirements from unauthenticated users", async ({ page, }, testInfo) => { - test.fixme( - true, - "PP-jsh — Radix DropdownMenu portal mount timing under CI load; awaiting verification of PP-awg/#1280 absorption" - ); // Logout to become unauthenticated await logout(page, testInfo); @@ -170,10 +158,6 @@ test.describe("Machine Details - Extended", () => { test("should display owner requirements callout on issue page", async ({ page, }, testInfo) => { - test.fixme( - true, - "PP-jsh — Radix DropdownMenu portal mount timing under CI load; awaiting verification of PP-awg/#1280 absorption" - ); // First, let's login as admin and set owner requirements on a machine await logout(page, testInfo); await loginAs(page, testInfo, { diff --git a/e2e/full/status-overhaul.spec.ts b/e2e/full/status-overhaul.spec.ts index f6acf2827..bcf2d3442 100644 --- a/e2e/full/status-overhaul.spec.ts +++ b/e2e/full/status-overhaul.spec.ts @@ -19,10 +19,6 @@ test.describe("Status Overhaul E2E", () => { }); test("should create issue and verify all 4 badges", async ({ page }) => { - test.fixme( - true, - "PP-v7g — Radix Select portal mount timing under CI load; awaiting verification of PP-awg/#1280 absorption" - ); const machine = seededMachines.addamsFamily; // 1. Create Issue