test: skip flaky 'Page.goto extraHttpHeaders should be pushed to prov…

…isional page' (#1203)
microsoft · Mar 4, 2020 · 33f3e57 · 33f3e57
1 parent 1c4619e
commit 33f3e57
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/test/navigation.spec.js b/test/navigation.spec.js
@@ -392,7 +392,9 @@ 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}) => {
+    it.skip(true)('extraHttpHeaders should be pushed to provisional page', async({page, server}) => {
+      // This test is flaky, because we cannot await page.setExtraHTTPHeaders.
+      // We need a way to test our implementation by more than just public api.
       await page.goto(server.EMPTY_PAGE);
       const pagePath = '/one-style.html';
       server.setRoute(pagePath, async (req, res) => {