Scripts: Properly use CommonJS for default Playwright config (#54988)

* Fix path to `globalSetup` in default Playwright config Oversight from #54856 * `module.exports` * Fix default export usage
WordPress · Oct 4, 2023 · d36bd6d · d36bd6d
1 parent f112f14
commit d36bd6d
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/packages/scripts/config/playwright.config.js b/packages/scripts/config/playwright.config.js
@@ -25,7 +25,7 @@ const config = defineConfig( {
  snapshotPathTemplate:
  '{testDir}/{testFileDir}/__snapshots__/{arg}-{projectName}{ext}',
  globalSetup: fileURLToPath(
- new URL( './playwright/global-setup.ts', 'file:' + __filename ).href
+ new URL( './playwright/global-setup.js', 'file:' + __filename ).href
  ),
  use: {
  baseURL: process.env.WP_BASE_URL || 'http://localhost:8889',
@@ -60,4 +60,4 @@ const config = defineConfig( {
  ],
 } );
 
-export default config;
+module.exports = config;
diff --git a/packages/scripts/config/playwright/global-setup.js b/packages/scripts/config/playwright/global-setup.js
@@ -32,4 +32,4 @@ async function globalSetup( config ) {
  await requestContext.dispose();
 }
 
-export default globalSetup;
+module.exports = globalSetup;
diff --git a/test/e2e/playwright.config.ts b/test/e2e/playwright.config.ts
@@ -11,7 +11,7 @@ import { defineConfig, devices } from '@playwright/test';
 const baseConfig = require( '@wordpress/scripts/config/playwright.config' );
 
 const config = defineConfig( {
- ...baseConfig.default,
+ ...baseConfig,
  reporter: process.env.CI
  ? [ [ 'github' ], [ './config/flaky-tests-reporter.ts' ] ]
  : 'list',

diff --git a/test/performance/playwright.config.ts b/test/performance/playwright.config.ts
@@ -13,7 +13,7 @@ const baseConfig = require( '@wordpress/scripts/config/playwright.config' );
 process.env.ASSETS_PATH = path.join( __dirname, 'assets' );
 
 const config = defineConfig( {
- ...baseConfig.default,
+ ...baseConfig,
  reporter: process.env.CI
  ? './config/performance-reporter.ts'
  : [ [ 'list' ], [ './config/performance-reporter.ts' ] ],
@@ -26,7 +26,7 @@ const config = defineConfig( {
  new URL( './config/global-setup.ts', 'file:' + __filename ).href
  ),
  use: {
- ...baseConfig.default.use,
+ ...baseConfig.use,
  video: 'off',
  },
 } );