Skip to content

Commit 91d8ede

Browse files
fix: remove custom test extensions causing Playwright conflicts in CI
- Remove test.extend() from all base-test.ts files - This was causing "test.describe() called in unexpected place" errors - Keep only the BasePage class exports - Tests now use standard @playwright/test imports without extensions This resolves the Playwright version conflict issues in CI environment. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 7b7be59 commit 91d8ede

File tree

4 files changed

+4
-27
lines changed

4 files changed

+4
-27
lines changed

advanced-api/automatic-vendor-sharing/e2e/utils/base-test.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { test as base, Page } from '@playwright/test';
1+
import { Page } from '@playwright/test';
22

33
export class BasePage {
44
constructor(public page: Page) {}
@@ -44,8 +44,3 @@ export class BasePage {
4444
}
4545
}
4646

47-
export const test = base.extend<{ basePage: BasePage }>({
48-
basePage: async ({ page }, use) => {
49-
await use(new BasePage(page));
50-
},
51-
});

advanced-api/dynamic-remotes-runtime-environment-variables/e2e/utils/base-test.ts

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { test as base, expect, Page } from '@playwright/test';
1+
import { Page } from '@playwright/test';
22

33
export class BasePage {
44
constructor(public page: Page) {}
@@ -44,10 +44,3 @@ export class BasePage {
4444
}
4545
}
4646

47-
export const test = base.extend<{ basePage: BasePage }>({
48-
basePage: async ({ page }, use) => {
49-
await use(new BasePage(page));
50-
},
51-
});
52-
53-
export { expect };

advanced-api/dynamic-remotes-synchronous-imports/e2e/utils/base-test.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { test as base, Page } from '@playwright/test';
1+
import { Page } from '@playwright/test';
22

33
export class BasePage {
44
constructor(public page: Page) {}
@@ -63,8 +63,3 @@ export class BasePage {
6363
}
6464
}
6565

66-
export const test = base.extend<{ basePage: BasePage }>({
67-
basePage: async ({ page }, use) => {
68-
await use(new BasePage(page));
69-
},
70-
});

advanced-api/dynamic-remotes/e2e/utils/base-test.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { test as base, expect, Page } from '@playwright/test';
1+
import { Page } from '@playwright/test';
22

33
export class BasePage {
44
constructor(public page: Page) {}
@@ -87,11 +87,5 @@ export class BasePage {
8787
}
8888
}
8989

90-
export const test = base.extend<{ basePage: BasePage }>({
91-
basePage: async ({ page }, use) => {
92-
const basePage = new BasePage(page);
93-
await use(basePage);
94-
},
95-
});
9690

9791
export { expect } from '@playwright/test';

0 commit comments

Comments
 (0)