fix(test): fix race in confusing confuse with previous navigation test (#730)

This commit is contained in:
Joel Einbinder 2020-01-28 17:25:07 -08:00 committed by GitHub
parent 89a93113f0
commit 4a3bd6008b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -539,8 +539,10 @@ module.exports.describe = function({testRunner, expect, headless, playwright, FF
let imgResponse = null;
server.setRoute(imgPath, (req, res) => imgResponse = res);
let loaded = false;
// get the global object to make sure that the main execution context is alive and well.
await page.evaluate(() => this);
// Trigger navigation which might resolve next setContent call.
page.evaluate(url => window.location.href = url, server.EMPTY_PAGE);
const evalPromise = page.evaluate(url => window.location.href = url, server.EMPTY_PAGE);
const contentPromise = page.setContent(`<img src="${server.PREFIX + imgPath}"></img>`).then(() => loaded = true);
await server.waitForRequest(imgPath);
@ -551,6 +553,7 @@ module.exports.describe = function({testRunner, expect, headless, playwright, FF
imgResponse.end();
await contentPromise;
await evalPromise;
});
it('should work with doctype', async({page, server}) => {
const doctype = '<!DOCTYPE html>';