test: cleanup some test files (#1195)

Author: dgozman

test/chromium/headful.spec.js

@@ -22,6 +22,7 @@ const readFileAsync = util.promisify(fs.readFile);
 const rmAsync = util.promisify(require('rimraf'));
 const mkdtempAsync = util.promisify(fs.mkdtemp);
 const statAsync = util.promisify(fs.stat);
+const { makeUserDataDir, removeUserDataDir } = require('../utils');
 
 const TMP_FOLDER = path.join(os.tmpdir(), 'pw_tmp_folder-');
 
@@ -33,24 +34,60 @@ module.exports.describe = function({testRunner, expect, defaultBrowserOptions, p
   const {it, fit, xit, dit} = testRunner;
   const {beforeAll, beforeEach, afterAll, afterEach} = testRunner;
 
-  describe('CrPlaywright', function() {
-    describe('Playwright.launch webSocket option', function() {
-      it('should support the remote-debugging-port argument', async() => {
-        const options = Object.assign({}, defaultBrowserOptions);
-        const browserServer = await playwright.launchServer({ ...options, port: 0 });
-        const browser = await playwright.connect({ wsEndpoint: browserServer.wsEndpoint() });
-        expect(browserServer.wsEndpoint()).not.toBe(null);
-        const page = await browser.newPage();
-        expect(await page.evaluate('11 * 11')).toBe(121);
-        await page.close();
-        await browserServer.close();
-      });
-      it('should throw with remote-debugging-pipe argument and webSocket', async() => {
-        const options = Object.assign({}, defaultBrowserOptions);
-        options.args = ['--remote-debugging-pipe'].concat(options.args || []);
-        const error = await playwright.launchServer(options).catch(e => e);
-        expect(error.message).toContain('Playwright manages remote debugging connection itself');
-      });
+  const headfulOptions = Object.assign({}, defaultBrowserOptions, {
+    headless: false
+  });
+  const extensionPath = path.join(__dirname, '..', 'assets', 'simple-extension');
+  const extensionOptions = Object.assign({}, defaultBrowserOptions, {
+    headless: false,
+    args: [
+      `--disable-extensions-except=${extensionPath}`,
+      `--load-extension=${extensionPath}`,
+    ],
+  });
+
+  describe('launcher', function() {
+    it('should throw with remote-debugging-pipe argument', async() => {
+      const options = Object.assign({}, defaultBrowserOptions);
+      options.args = ['--remote-debugging-pipe'].concat(options.args || []);
+      const error = await playwright.launchServer(options).catch(e => e);
+      expect(error.message).toContain('Playwright manages remote debugging connection itself');
+    });
+    it('should throw with remote-debugging-port argument', async() => {
+      const options = Object.assign({}, defaultBrowserOptions);
+      options.args = ['--remote-debugging-port=9222'].concat(options.args || []);
+      const error = await playwright.launchServer(options).catch(e => e);
+      expect(error.message).toContain('Playwright manages remote debugging connection itself');
+    });
+    it('should open devtools when "devtools: true" option is given', async({server}) => {
+      const browser = await playwright.launch(Object.assign({devtools: true}, headfulOptions));
+      const context = await browser.newContext();
+      const browserSession = await browser.createBrowserSession();
+      await browserSession.send('Target.setDiscoverTargets', { discover: true });
+      const devtoolsPagePromise = new Promise(fulfill => browserSession.on('Target.targetCreated', async ({targetInfo}) => {
+        if (targetInfo.type === 'other' && targetInfo.url.includes('devtools://'))
+           fulfill();
+      }));
+      await Promise.all([
+        devtoolsPagePromise,
+        context.newPage()
+      ]);
+      await browser.close();
+    });
+  });
+
+  describe('extensions', () => {
+    it('should return background pages', async() => {
+      const userDataDir = await makeUserDataDir();
+      const context = await playwright.launchPersistent(userDataDir, extensionOptions);
+      const backgroundPages = await context.backgroundPages();
+      let backgroundPage = backgroundPages.length
+          ? backgroundPages[0]
+          : await new Promise(fulfill => context.once('backgroundpage', async event => fulfill(await event.page())));
+      expect(backgroundPage).toBeTruthy();
+      expect(await context.backgroundPages()).toContain(backgroundPage);
+      expect(await context.pages()).not.toContain(backgroundPage);
+      await removeUserDataDir(userDataDir);
     });
   });
 

test/chromium/launcher.spec.js

@@ -22,6 +22,10 @@ module.exports.describe = function({testRunner, expect, defaultBrowserOptions, p
   const {it, fit, xit, dit} = testRunner;
   const {beforeAll, beforeEach, afterAll, afterEach} = testRunner;
 
+  const headfulOptions = Object.assign({}, defaultBrowserOptions, {
+    headless: false
+  });
+
   describe('OOPIF', function() {
     beforeAll(async function(state) {
       state.browser = await playwright.launch(Object.assign({}, defaultBrowserOptions, {
@@ -54,7 +58,7 @@ module.exports.describe = function({testRunner, expect, defaultBrowserOptions, p
       expect(page.frames().length).toBe(2);
     });
     it('should load oopif iframes with subresources and request interception', async function({browser, page, server, context}) {
-      await page.route('*', request => request.continue());
+      await page.route('**/*', request => request.continue());
       const browserSession = await browser.createBrowserSession();
       await browserSession.send('Target.setDiscoverTargets', { discover: true });
       const oopifs = [];
@@ -66,5 +70,28 @@ module.exports.describe = function({testRunner, expect, defaultBrowserOptions, p
       expect(oopifs.length).toBe(1);
       await browserSession.detach();
     });
+    it.fail(true)('should report google.com frame with headful', async({server}) => {
+      // TODO: Support OOOPIF. @see https://github.com/GoogleChrome/puppeteer/issues/2548
+      // https://google.com is isolated by default in Chromium embedder.
+      const browser = await playwright.launch(headfulOptions);
+      const page = await browser.newPage();
+      await page.goto(server.EMPTY_PAGE);
+      await page.route('**/*', request => {
+        request.fulfill({body: 'YO, GOOGLE.COM'});
+      });
+      await page.evaluate(() => {
+        const frame = document.createElement('iframe');
+        frame.setAttribute('src', 'https://google.com/');
+        document.body.appendChild(frame);
+        return new Promise(x => frame.onload = x);
+      });
+      await page.waitForSelector('iframe[src="https://google.com/"]');
+      const urls = page.frames().map(frame => frame.url()).sort();
+      expect(urls).toEqual([
+        server.EMPTY_PAGE,
+        'https://google.com/'
+      ]);
+      await browser.close();
+    });
   });
 };

test/chromium/oopif.spec.js

@@ -392,6 +392,21 @@ module.exports.describe = function({testRunner, expect, playwright, MAC, WIN, FF
       const error = await failed;
       expect(error.message).toBeTruthy();
     });
+    it('extraHttpHeaders should be pushed to provisional page', async({page, server}) => {
+      await page.goto(server.EMPTY_PAGE);
+      const pagePath = '/one-style.html';
+      server.setRoute(pagePath, async (req, res) => {
+        page.setExtraHTTPHeaders({ foo: 'bar' });
+        server.serveFile(req, res, pagePath);
+      });
+      const [htmlReq, cssReq] = await Promise.all([
+        server.waitForRequest(pagePath),
+        server.waitForRequest('/one-style.css'),
+        page.goto(server.CROSS_PROCESS_PREFIX + pagePath)
+      ]);
+      expect(htmlReq.headers['foo']).toBe(undefined);
+      expect(cssReq.headers['foo']).toBe('bar');
+    });
 
     describe('network idle', function() {
       it('should navigate to empty page with networkidle0', async({page, server}) => {
@@ -404,11 +419,11 @@ module.exports.describe = function({testRunner, expect, playwright, MAC, WIN, FF
       });
 
       /**
-       * @param {import('../src/frames').Frame} frame 
-       * @param {TestServer} server 
-       * @param {'networkidle0'|'networkidle2'} signal 
-       * @param {() => Promise<void>} action 
-       * @param {boolean} isSetContent 
+       * @param {import('../src/frames').Frame} frame
+       * @param {TestServer} server
+       * @param {'networkidle0'|'networkidle2'} signal
+       * @param {() => Promise<void>} action
+       * @param {boolean} isSetContent
        */
       async function networkIdleTest(frame, server, signal, action, isSetContent) {
         const finishResponse = response => {

test/navigation.spec.js

@@ -181,17 +181,16 @@ module.exports.describe = ({testRunner, product, playwrightPath}) => {
         testRunner.loadTests(require('./workers.spec.js'), testOptions);
         testRunner.loadTests(require('./capabilities.spec.js'), testOptions);
       });
+      describe('[Permissions]', () => {
+        testRunner.loadTests(require('./permissions.spec.js'), testOptions);
+      });
 
       describe.skip(!CHROMIUM)('[Chromium]', () => {
         testRunner.loadTests(require('./chromium/chromium.spec.js'), testOptions);
         testRunner.loadTests(require('./chromium/coverage.spec.js'), testOptions);
         testRunner.loadTests(require('./chromium/pdf.spec.js'), testOptions);
         testRunner.loadTests(require('./chromium/session.spec.js'), testOptions);
       });
-
-      describe('[Permissions]', () => {
-        testRunner.loadTests(require('./features/permissions.spec.js'), testOptions);
-      });
     });
 
     // Browser-level tests that are given a browser.
@@ -214,7 +213,6 @@ module.exports.describe = ({testRunner, product, playwrightPath}) => {
 
   describe.skip(!CHROMIUM)('[Chromium]', () => {
     testRunner.loadTests(require('./chromium/launcher.spec.js'), testOptions);
-    testRunner.loadTests(require('./chromium/headful.spec.js'), testOptions);
     testRunner.loadTests(require('./chromium/oopif.spec.js'), testOptions);
     testRunner.loadTests(require('./chromium/tracing.spec.js'), testOptions);
   });