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

Author: dgozman

package-lock.json

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

package.json

@@ -18,9 +18,7 @@
 import { browserTest as it, expect } from './config/browserTest';
 
 it.describe('device', () => {
-  it.beforeEach(async ({browserName}) => {
-    it.skip(browserName === 'firefox');
-  });
+  it.skip(({ browserName }) => browserName === 'firefox');
 
   it('should work', async ({playwright, browser, server}) => {
     const iPhone = playwright.devices['iPhone 6'];

tests/browsercontext-device.spec.ts

@@ -18,9 +18,8 @@ import { browserTest as it, expect } from './config/browserTest';
 import type { Browser } from '../index';
 
 let browser: Browser;
-it.beforeEach(async ({ browserType, browserOptions }) => {
-  if (!browser)
-    browser = await browserType.launch({ ...browserOptions, proxy: { server: 'per-context' } });
+it.beforeAll(async ({ browserType, browserOptions }) => {
+  browser = await browserType.launch({ ...browserOptions, proxy: { server: 'per-context' } });
 });
 it.afterAll(async () => {
   await browser.close();

tests/browsercontext-proxy.spec.ts

@@ -18,9 +18,7 @@
 import { browserTest as it, expect } from './config/browserTest';
 
 it.describe('mobile viewport', () => {
-  it.beforeEach(async ({ browserName }) => {
-    it.skip(browserName === 'firefox');
-  });
+  it.skip(({ browserName }) => browserName === 'firefox');
 
   it('should support mobile emulation', async ({playwright, browser, server}) => {
     const iPhone = playwright.devices['iPhone 6'];

tests/browsercontext-viewport-mobile.spec.ts

@@ -19,10 +19,8 @@ import { test as it, expect } from './config/pageTest';
 import { browserTest } from './config/browserTest';
 import { verifyViewport } from './config/utils';
 
-it.beforeEach(async ({ isElectron, isAndroid }) => {
-  it.skip(isAndroid, 'Default viewport is null');
-  it.skip(isElectron, 'Default viewport is null');
-});
+it.skip(({ isAndroid }) => isAndroid, 'Default viewport is null');
+it.skip(({ isElectron }) => isElectron, 'Default viewport is null');
 
 it('should get the proper default viewport size', async ({page, server}) => {
   await verifyViewport(page, 1280, 720);

tests/browsercontext-viewport.spec.ts

@@ -18,9 +18,7 @@
 import { playwrightTest as it, expect } from './config/browserTest';
 
 it.describe('launch server', () => {
-  it.beforeEach(async ({ mode}) => {
-    it.skip(mode !== 'default');
-  });
+  it.skip(({ mode}) => mode !== 'default');
 
   it('should work', async ({browserType, browserOptions}) => {
     const browserServer = await browserType.launchServer(browserOptions);

tests/browsertype-launch-server.spec.ts

@@ -20,11 +20,9 @@ import { playwrightTest } from '../config/browserTest';
 import http from 'http';
 
 pageTest.describe('chromium', () => {
-  pageTest.beforeEach(async ({ browserName, isElectron, isAndroid }) => {
-    pageTest.skip(browserName !== 'chromium');
-    pageTest.skip(isAndroid);
-    pageTest.skip(isElectron);
-  });
+  pageTest.skip(({ browserName }) => browserName !== 'chromium');
+  pageTest.skip(({ isAndroid }) => isAndroid);
+  pageTest.skip(({ isElectron }) => isElectron);
 
   pageTest('should create a worker from a service worker', async ({page, server}) => {
     const [worker] = await Promise.all([
@@ -85,9 +83,7 @@ pageTest.describe('chromium', () => {
 });
 
 playwrightTest.describe('chromium', () => {
-  playwrightTest.beforeEach(async ({ browserName }) => {
-    playwrightTest.skip(browserName !== 'chromium');
-  });
+  playwrightTest.skip(({ browserName }) => browserName !== 'chromium');
 
   playwrightTest('should close service worker together with the context', async ({browserType, browserOptions, server}) => {
     const browser = await browserType.launch(browserOptions);

tests/chromium/chromium.spec.ts

@@ -17,9 +17,7 @@
 import { test as it, expect } from '../config/pageTest';
 
 it.describe('CSS Coverage', () => {
-  it.beforeEach(async ({ browserName }) => {
-    it.skip(browserName !== 'chromium');
-  });
+  it.skip(({ browserName }) => browserName !== 'chromium');
 
   it('should work', async function({page, server}) {
     await page.coverage.startCSSCoverage();

tests/chromium/css-coverage.spec.ts

@@ -17,10 +17,8 @@
 import { test as it, expect } from '../config/pageTest';
 
 it.describe('JS Coverage', () => {
-  it.beforeEach(async ({ browserName, isElectron }) => {
-    it.skip(browserName !== 'chromium');
-    it.fixme(isElectron);
-  });
+  it.skip(({ browserName }) => browserName !== 'chromium');
+  it.fixme(({ isElectron }) => isElectron);
 
   it('should work', async function({page, server}) {
     await page.coverage.startJSCoverage();