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

Author: dgozman

docs/src/test-snapshots.md

@@ -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-alpha27",
+    "folio": "=0.4.0-alpha28",
     "formidable": "^1.2.2",
     "html-webpack-plugin": "^4.4.1",
     "ncp": "^2.0.0",

package-lock.json

@@ -21,7 +21,8 @@ import type { PlaywrightTestArgs, PlaywrightTestOptions, PlaywrightWorkerArgs, P
 
 export * from 'folio';
 export const test = folio.test.extend<PlaywrightTestArgs & PlaywrightTestOptions, PlaywrightWorkerArgs & PlaywrightWorkerOptions>({
-  browserName: [ 'chromium', { scope: 'worker' } ],
+  defaultBrowserType: [ 'chromium', { scope: 'worker' } ],
+  browserName: [ ({ defaultBrowserType }, use) => use(defaultBrowserType), { scope: 'worker' } ],
   playwright: [ require('../inprocess'), { scope: 'worker' } ],
   headless: [ undefined, { scope: 'worker' } ],
   channel: [ undefined, { scope: 'worker' } ],
@@ -66,8 +67,8 @@ export const test = folio.test.extend<PlaywrightTestArgs & PlaywrightTestOptions
   viewport: undefined,
   contextOptions: {},
 
-  context: async ({ browserName, browser, screenshot, video, acceptDownloads, bypassCSP, colorScheme, deviceScaleFactor, extraHTTPHeaders, hasTouch, geolocation, httpCredentials, ignoreHTTPSErrors, isMobile, javaScriptEnabled, locale, offline, permissions, proxy, storageState, viewport, timezoneId, userAgent, contextOptions }, use, testInfo) => {
-    testInfo.snapshotSuffix = browserName + '-' + process.platform;
+  context: async ({ browser, screenshot, video, acceptDownloads, bypassCSP, colorScheme, deviceScaleFactor, extraHTTPHeaders, hasTouch, geolocation, httpCredentials, ignoreHTTPSErrors, isMobile, javaScriptEnabled, locale, offline, permissions, proxy, storageState, viewport, timezoneId, userAgent, contextOptions }, use, testInfo) => {
+    testInfo.snapshotSuffix = process.platform;
     if (process.env.PWDEBUG)
       testInfo.setTimeout(0);
 
@@ -148,3 +149,5 @@ export const test = folio.test.extend<PlaywrightTestArgs & PlaywrightTestOptions
   },
 });
 export default test;
+
+export const __baseTest = folio.test;

package.json

@@ -87,18 +87,16 @@ export function addTestCommand(program: commander.CommanderStatic) {
 }
 
 async function runTests(Runner: RunnerType, args: string[], opts: { [key: string]: any }) {
-  if (opts.browser) {
-    const browserOpt = opts.browser.toLowerCase();
-    if (!['all', 'chromium', 'firefox', 'webkit'].includes(browserOpt))
-      throw new Error(`Unsupported browser "${opts.browser}", must be one of "all", "chromium", "firefox" or "webkit"`);
-    const browserNames = browserOpt === 'all' ? ['chromium', 'firefox', 'webkit'] : [browserOpt];
-    defaultConfig.projects = browserNames.map(browserName => {
-      return {
-        name: browserName,
-        use: { browserName },
-      };
-    });
-  }
+  const browserOpt = opts.browser ? opts.browser.toLowerCase() : 'chromium';
+  if (!['all', 'chromium', 'firefox', 'webkit'].includes(browserOpt))
+    throw new Error(`Unsupported browser "${opts.browser}", must be one of "all", "chromium", "firefox" or "webkit"`);
+  const browserNames = browserOpt === 'all' ? ['chromium', 'firefox', 'webkit'] : [browserOpt];
+  defaultConfig.projects = browserNames.map(browserName => {
+    return {
+      name: browserName,
+      use: { browserName },
+    };
+  });
 
   const overrides = overridesFromOptions(opts);
   if (opts.headed)

src/cli/fixtures.ts

@@ -101,7 +101,7 @@ class DefaultMode {
   }
 }
 
-const baseFixtures: folio.Fixtures<{ __baseSetup: void }, BaseOptions & BaseFixtures> = {
+const baseFixtures: folio.Fixtures<{}, BaseOptions & BaseFixtures> = {
   mode: [ 'default', { scope: 'worker' } ],
   browserName: [ 'chromium' , { scope: 'worker' } ],
   channel: [ undefined, { scope: 'worker' } ],
@@ -123,10 +123,6 @@ const baseFixtures: folio.Fixtures<{ __baseSetup: void }, BaseOptions & BaseFixt
   isWindows: [ process.platform === 'win32', { scope: 'worker' } ],
   isMac: [ process.platform === 'darwin', { scope: 'worker' } ],
   isLinux: [ process.platform === 'linux', { scope: 'worker' } ],
-  __baseSetup: [ async ({ browserName }, run, testInfo) => {
-    testInfo.snapshotSuffix = browserName;
-    await run();
-  }, { auto: true } ],
 };
 
 type ServerOptions = {

src/cli/testRunner.ts

@@ -48,7 +48,7 @@ const metadata = {
 };
 
 config.projects.push({
-  name: 'electron',
+  name: 'chromium',  // We use 'chromium' here to share screenshots with chromium.
   use: {
     mode: 'default',
     browserName: 'chromium',
@@ -59,7 +59,7 @@ config.projects.push({
 });
 
 config.projects.push({
-  name: 'electron',
+  name: 'chromium',  // We use 'chromium' here to share screenshots with chromium.
   use: {
     mode: 'default',
     browserName: 'chromium',