test(nextjs): Attempt to make Next.js integration tests less flakey

[lforst]

Partly adresses #6053

I guess there is a possibility that requests go out in between the page.goto() calls and the page.waitForRequest(), causing us to wait for the latter indefinitely.
If we await the request before we go to the page, there is no way a request is sent before page.waitForRequest() is called.

[AbhiPrasad]

l: Might be nice to make this a util

[lobsterkatie]

Could you include a sentence or two in the PR description explaining the logic here, please?

[lforst]

Ok so we didn't eliminate flakes completely with this but at least there were no timeouts anymore in 5 reruns so let's just roll with it.

[lobsterkatie]

Could you include a sentence or two in the PR description explaining the logic here, please?

I guess there is a possibility that requests go out in between the page.goto() calls and the page.waitForRequest(), causing us to wait for the latter indefinitely.
If we await the request before we go to the page, there is no way a request is sent before page.waitForRequest() is called.

Ah, right, okay, this makes sense. Unawaited waitForRequest() is really just startWaitingForRequest(). Because promises.

Thanks.