From dde55b025746d35311b55a885793f113b410c4f6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dar=C3=ADo=20Kondratiuk?= Date: Fri, 19 Apr 2019 15:29:29 -0300 Subject: [PATCH] test: Make intervention test Edge friendly (#4314) All tests are green when running against [Microsoft Edge Insider](https://www.microsoftedgeinsider.com/en-us/), except this test. Edge returns `https://to-be-replaced.invalid/feature/5718547946799104` instead of `https://www.chromestatus.com/feature/5718547946799104` I think that checking for `feature/5718547946799104` is a small test and will do the job for both browsers. ``` $env:CHROME = "C:\Program Files (x86)\Microsoft\Edge Dev\Application\msedge.exe"; node .\test.js ``` --- test/chromiumonly.spec.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/test/chromiumonly.spec.js b/test/chromiumonly.spec.js index cd3a5ccd8c4d6..b19d3942a90ea 100644 --- a/test/chromiumonly.spec.js +++ b/test/chromiumonly.spec.js @@ -118,7 +118,9 @@ module.exports.addPageTests = function({testRunner, expect}) { await page.setRequestInterception(true); page.on('request', request => request.continue()); await page.goto(server.PREFIX + '/intervention'); - expect(serverRequest.headers.intervention).toContain('www.chromestatus.com'); + // Check for feature URL substring rather than https://www.chromestatus.com to + // make it work with Edgium. + expect(serverRequest.headers.intervention).toContain('feature/5718547946799104'); }); }); };