From 38174907f30ca307be978cd9e3e274831f1c870b Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Thu, 28 May 2026 00:24:06 +0000 Subject: [PATCH] fix(addon-progress): stabilize WebKit integration test harness Remove unnecessary unawaited setViewportSize from ProgressAddon tests, which was the line reported in flaky CI failures. Use serial mode and scoped hooks per Playwright shared-page guidance, and guard afterAll teardown when setup fails. Fixes #5956 Co-authored-by: Daniel Imms --- .../addon-progress/test/ProgressAddon.test.ts | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/addons/addon-progress/test/ProgressAddon.test.ts b/addons/addon-progress/test/ProgressAddon.test.ts index 792c0445d..12ff7a38f 100644 --- a/addons/addon-progress/test/ProgressAddon.test.ts +++ b/addons/addon-progress/test/ProgressAddon.test.ts @@ -9,15 +9,18 @@ import { ITestContext, createTestContext, openTerminal } from '../../../test/pla let ctx: ITestContext; -test.beforeAll(async ({ browser }) => { - ctx = await createTestContext(browser); - ctx.page.setViewportSize({ width: 1024, height: 768 }); - await openTerminal(ctx); -}); -test.afterAll(async () => await ctx.page.close()); - test.describe('ProgressAddon', () => { + test.describe.configure({ mode: 'serial' }); + + test.beforeAll(async ({ browser }) => { + ctx = await createTestContext(browser); + await openTerminal(ctx); + }); + test.afterAll(async () => { + await ctx?.page?.close(); + }); + test.beforeEach(async function(): Promise { await ctx.page.evaluate(` window.progressStack = [];