test: roll to [email protected] (#6897)

Author: dgozman

docs/src/test-reporters.md

@@ -39,10 +39,12 @@ You can use different reporters locally and on CI.
 // playwright.config.js
 module.exports = {
   reporter: !process.env.CI
-    // A list of tests for the terminal
+    // Default 'list' reporter for the terminal
     ? 'list'
-    // Very concise "dot" reporter and a comprehensive json report for CI
-    : ['dot', { name: 'json', outputFile: 'test-results.json' }],
+    // Two reporters for CI:
+    // - concise "dot"
+    // - comprehensive json report
+    : [ ['dot'], [ 'json', {  outputFile: 'test-results.json' }] ],
 };
 ```
 
@@ -51,11 +53,12 @@ module.exports = {
 import { PlaywrightTestConfig } from '@playwright/test';
 
 const config: PlaywrightTestConfig = {
-  reporter: !process.env.CI
-    // A list of tests for the terminal
+    // Default 'list' reporter for the terminal
     ? 'list'
-    // Very concise "dot" reporter and a comprehensive json report for CI
-    : ['dot', { name: 'json', outputFile: 'test-results.json' }],
+    // Two reporters for CI:
+    // - concise "dot"
+    // - comprehensive json report
+    : [ ['dot'], [ 'json', {  outputFile: 'test-results.json' }] ],
 };
 export default config;
 ```
@@ -190,7 +193,7 @@ In configuration file, pass options directly:
 ```js js-flavor=js
 // playwright.config.js
 module.exports = {
-  reporter: { name: 'json', outputFile: 'results.json' },
+  reporter: [ ['json', { outputFile: 'results.json' }] ],
 };
 ```
 
@@ -199,7 +202,7 @@ module.exports = {
 import { PlaywrightTestConfig } from '@playwright/test';
 
 const config: PlaywrightTestConfig = {
-  reporter: { name: 'json', outputFile: 'results.json' },
+  reporter: [ ['json', { outputFile: 'results.json' }] ],
 };
 export default config;
 ```
@@ -217,7 +220,7 @@ In configuration file, pass options directly:
 ```js js-flavor=js
 // playwright.config.js
 module.exports = {
-  reporter: { name: 'junit', outputFile: 'results.xml' },
+  reporter: [ ['junit', { outputFile: 'results.xml' }] ],
 };
 ```
 
@@ -226,7 +229,7 @@ module.exports = {
 import { PlaywrightTestConfig } from '@playwright/test';
 
 const config: PlaywrightTestConfig = {
-  reporter: { name: 'junit', outputFile: 'results.xml' },
+  reporter: [ ['junit', { outputFile: 'results.xml' }] ],
 };
 export default config;
 ```

package-lock.json

@@ -80,7 +80,7 @@
     "eslint-plugin-notice": "^0.9.10",
     "eslint-plugin-react-hooks": "^4.2.0",
     "file-loader": "^6.1.0",
-    "folio": "=0.4.0-alpha26",
+    "folio": "=0.4.0-alpha27",
     "formidable": "^1.2.2",
     "html-webpack-plugin": "^4.4.1",
     "ncp": "^2.0.0",

package.json

@@ -30,7 +30,7 @@ const tsConfig = 'playwright.config.ts';
 const jsConfig = 'playwright.config.js';
 const defaultConfig: Config = {
   preserveOutput: process.env.CI ? 'failures-only' : 'always',
-  reporter: [defaultReporter],
+  reporter: [ [defaultReporter] ],
   timeout: defaultTimeout,
   updateSnapshots: process.env.CI ? 'none' : 'missing',
   workers: Math.ceil(require('os').cpus().length / 2),
@@ -167,9 +167,7 @@ function overridesFromOptions(options: { [key: string]: any }): Config {
     quiet: options.quiet ? options.quiet : undefined,
     repeatEach: options.repeatEach ? parseInt(options.repeatEach, 10) : undefined,
     retries: options.retries ? parseInt(options.retries, 10) : undefined,
-    reporter: (options.reporter && options.reporter.length) ? options.reporter.split(',').map((r: string) => {
-      return builtinReporters.includes(r) ? r : { require: r };
-    }) : undefined,
+    reporter: (options.reporter && options.reporter.length) ? options.reporter.split(',').map((r: string) => [r]) : undefined,
     shard: shardPair ? { current: shardPair[0] - 1, total: shardPair[1] } : undefined,
     timeout: isDebuggerAttached ? 0 : (options.timeout ? parseInt(options.timeout, 10) : undefined),
     updateSnapshots: options.updateSnapshots ? 'all' as const : undefined,

src/cli/testRunner.ts

@@ -32,8 +32,8 @@ const config: folio.Config<CommonOptions & PlaywrightOptions> = {
   forbidOnly: !!process.env.CI,
   retries: process.env.CI ? 1 : 0,
   reporter: process.env.CI ? [
-    'dot',
-    { name: 'json', outputFile: path.join(outputDir, 'report.json') },
+    [ 'dot' ],
+    [ 'json', { outputFile: path.join(outputDir, 'report.json') } ],
   ] : 'line',
   projects: [],
 };

tests/config/android.config.ts

@@ -54,8 +54,8 @@ const config: folio.Config<CommonOptions & PlaywrightOptions> = {
   forbidOnly: !!process.env.CI,
   retries: process.env.CI ? 3 : 0,
   reporter: process.env.CI ? [
-    'dot',
-    { name: 'json', outputFile: path.join(outputDir, 'report.json') },
+    [ 'dot' ],
+    [ 'json', { outputFile: path.join(outputDir, 'report.json') } ],
   ] : 'line',
   projects: [],
 };

tests/config/default.config.ts

@@ -32,8 +32,8 @@ const config: folio.Config<CommonOptions & PlaywrightOptions> = {
   forbidOnly: !!process.env.CI,
   retries: process.env.CI ? 3 : 0,
   reporter: process.env.CI ? [
-    'dot',
-    { name: 'json', outputFile: path.join(outputDir, 'report.json') },
+    [ 'dot' ],
+    [ 'json', { outputFile: path.join(outputDir, 'report.json') } ],
   ] : 'line',
   projects: [],
 };