page.waitForNavigation returns null when interceptors are enabled

import puppeteer from "puppeteer";
import {pause, disableImage} from "../../helpers";
import parseUrl from "parse-url";
import env from './../../env';

let page;
let browser;


beforeAll(async () => {
    browser = await puppeteer.launch({
        slowMo: 80,
        args: [
            `--window-size=${env.width},${env.height}`,
            ...env.args,
        ]
    });
    page = await browser.newPage();
    await page.setRequestInterception(true);
    page.on('request', request => {
        // add stylesheet, font
        if (request.resourceType === 'image')
            request.abort();
        else
            request.continue();
    });
    await page.setViewport({width: env.width, height: env.height});
});


afterAll(() => {
    browser.close();
});


describe("failure-in-login", () => {
    test("should fail due to wrong user", async () => {
        await page.goto(env.host);
        await page.waitForSelector('input#user');
        await pause(100);
        await page.type('input#user', 'admiin');
        await page.type('input#password', env.admin.password);
        const [response] = await Promise.all([
            page.waitForNavigation(),
            page.click('#submit')
        ]);

        expect(parseUrl(response.url).query.user).toEqual('admiin');
    }, 10000 * env.time);
});

I have an error on expect(parseUrl(response.url).query.user).toEqual(‘admiin’); caused by response variable to be null. and if I only remove interceptor things work just fine.

[aslushnikov]

@hasangilak we’ve fixed a few bugs related to this in 1.0.0. Please, try updating.

Beware: all getters in 0.13 became methods in 1.0. For example, request.url getter became request.url() method. Full list of changes could be found here: https://github.com/GoogleChrome/puppeteer/releases/tag/v1.0.0

[aslushnikov]

is there any way to drastically reduce test time?

Not a general one; you can try caching the page or disabling javascript though.