New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: add automatic wait for vite client connected #5437
Conversation
|
unfortunately it looks like that it now hangs the firefox tests. investigating tomorrow. |
will mark this as draft until it's ready |
firefox may be hanging due to microsoft/playwright#15550 |
I've updated to Playwright 1.25.0, so I think that issue should be fixed now |
What's the status of this — is it still needed? Ready for review? |
oh, didn't realize the firefox issue might be fixed. Not sure if it is still needed, i think separating the write tests into their own app made this mostly obsolete. lets see what happens after bringing this up 2 date. |
83289ed
to
9bf8191
Compare
now linux+chrome has fails with waiting for that console message. at this point it's safe to say that this does not improve stability |
I suspect the reason this was not working is vitejs/vite#7733 |
i saw that but vite plugin svelte still uses this check, looks like playwrights console api gives you debug too |
to page navigation functions
This isn't the nicest way to do it and doesn't recognize cases where goBack leads to a load.
But it can stabilize tests that rely on the vite client and previously may have started work before that was ready.
logic taken from vite-plugin-svelte. cc @bluwy
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
Tests
pnpm test
and lint the project withpnpm lint
andpnpm check
Changesets
pnpm changeset
and following the prompts. All changesets should bepatch
until SvelteKit 1.0