Skip to content

Commit 0c25603

Browse files
renemadsenclaude
andcommitted
fix: add early return in pairing beforeAll/afterAll to prevent execution
beforeAll/afterAll run even when tests are skipped via beforeEach. Add early returns to prevent the hooks from executing. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent cb85695 commit 0c25603

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

eform-client/playwright/e2e/plugins/items-planning-pn/c/items-planning.pairing.spec.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ test.describe.serial('Items planning plugin - Pairing', () => {
2525
test.describe.configure({ timeout: 600000 });
2626
test.beforeEach(() => { test.skip(true, 'pairing checkbox not working in CI'); });
2727
test.beforeAll(async ({ browser }, testInfo) => {
28+
return; // skipped — see TODO above
2829
testInfo.setTimeout(600000);
2930
page = await browser.newPage();
3031
const loginPage = new LoginPage(page);
@@ -80,6 +81,7 @@ test.describe.serial('Items planning plugin - Pairing', () => {
8081
});
8182

8283
test.afterAll(async ({}, testInfo) => {
84+
return; // skipped — see TODO above
8385
testInfo.setTimeout(600000);
8486
const myEformsPage = new MyEformsPage(page);
8587
const foldersPage = new FoldersPage(page);

0 commit comments

Comments
 (0)