test: fix a few tests on firefox
This commit is contained in:
parent
4bb3c41faa
commit
b43413b5dd
|
|
@ -221,7 +221,7 @@ module.exports.addTests = function({testRunner, expect, FFOX, CHROME, WEBKIT}) {
|
|||
// FIXME: WebKit doesn't provide remoteIPAddress in the response.
|
||||
it.skip(WEBKIT)('Page.Events.Response', async({page, server}) => {
|
||||
const responses = [];
|
||||
page.on('response', response => responses.push(response));
|
||||
page.on('response', response => !utils.isFavicon(response.request()) && responses.push(response));
|
||||
await page.goto(server.EMPTY_PAGE);
|
||||
expect(responses.length).toBe(1);
|
||||
expect(responses[0].url()).toBe(server.EMPTY_PAGE);
|
||||
|
|
@ -260,7 +260,7 @@ module.exports.addTests = function({testRunner, expect, FFOX, CHROME, WEBKIT}) {
|
|||
});
|
||||
it('Page.Events.RequestFinished', async({page, server}) => {
|
||||
const requests = [];
|
||||
page.on('requestfinished', request => requests.push(request));
|
||||
page.on('requestfinished', request => !utils.isFavicon(request) && requests.push(request));
|
||||
await page.goto(server.EMPTY_PAGE);
|
||||
expect(requests.length).toBe(1);
|
||||
expect(requests[0].url()).toBe(server.EMPTY_PAGE);
|
||||
|
|
@ -270,9 +270,9 @@ module.exports.addTests = function({testRunner, expect, FFOX, CHROME, WEBKIT}) {
|
|||
});
|
||||
it('should fire events in proper order', async({page, server}) => {
|
||||
const events = [];
|
||||
page.on('request', request => events.push('request'));
|
||||
page.on('response', response => events.push('response'));
|
||||
page.on('requestfinished', request => events.push('requestfinished'));
|
||||
page.on('request', request => !utils.isFavicon(request) && events.push('request'));
|
||||
page.on('response', response => !utils.isFavicon(response.request()) && events.push('response'));
|
||||
page.on('requestfinished', request => !utils.isFavicon(request) && events.push('requestfinished'));
|
||||
await page.goto(server.EMPTY_PAGE);
|
||||
expect(events).toEqual(['request', 'response', 'requestfinished']);
|
||||
});
|
||||
|
|
|
|||
|
|
@ -100,7 +100,6 @@ module.exports.addTests = function({testRunner, expect, headless, playwright, FF
|
|||
describe('Async stacks', () => {
|
||||
it('should work', async({page, server}) => {
|
||||
server.setRoute('/empty.html', (req, res) => {
|
||||
req.socket.write('deadbeef');
|
||||
req.socket.end();
|
||||
});
|
||||
let error = null;
|
||||
|
|
@ -178,7 +177,7 @@ module.exports.addTests = function({testRunner, expect, headless, playwright, FF
|
|||
page.on('popup', () => badSecondPopup = true);
|
||||
await popup.goto(server.CROSS_PROCESS_PREFIX + '/empty.html');
|
||||
expect(badSecondPopup).toBe(false);
|
||||
})
|
||||
});
|
||||
});
|
||||
|
||||
describe('Page.Events.Console', function() {
|
||||
|
|
|
|||
Loading…
Reference in a new issue