test(chromium): mark passing popup tests as passing (#1297)

microsoft · Mar 9, 2020 · cf46f1b · cf46f1b
1 parent ca5ce7d
commit cf46f1b
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/test/popup.spec.js b/test/popup.spec.js
@@ -83,7 +83,6 @@ module.exports.describe = function({testRunner, expect, playwright, CHROMIUM, WE
       expect(online).toBe(false);
     });
     it('should inherit http credentials from browser context', async function({browser, server}) {
-      // Use unique user/password since Chromium caches credentials per origin.
       server.setAuth('/title.html', 'user', 'pass');
       const context = await browser.newContext({
         httpCredentials: { username: 'user', password: 'pass' }
@@ -123,7 +122,7 @@ module.exports.describe = function({testRunner, expect, playwright, CHROMIUM, WE
       await context.close();
       expect(size).toEqual({width: 400, height: 500});
     });
-    it.fail(CHROMIUM)('should apply addInitScript from browser context', async function({browser, server}) {
+    it('should apply addInitScript from browser context', async function({browser, server}) {
       const context = await browser.newContext();
       await context.addInitScript(() => window.injected = 123);
       const page = await context.newPage();
@@ -135,7 +134,7 @@ module.exports.describe = function({testRunner, expect, playwright, CHROMIUM, WE
       await context.close();
       expect(injected).toBe(123);
     });
-    it.fail(CHROMIUM || FFOX)('should expose function from browser context', async function({browser, server}) {
+    it.fail(FFOX)('should expose function from browser context', async function({browser, server}) {
       const context = await browser.newContext();
       await context.exposeFunction('add', (a, b) => a + b);
       const page = await context.newPage();