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
try to get some tests passing #5333
Conversation
|
I've been testing various commits. I'm getting hanging behavior with a6781ae, but not the commit before and its pretty consistent The only explanation I can come up with as to why that commit changed what we're seeing is that it made things more efficient by not watching the |
Merging this so that it's easier to diagnose the browser-specific failures exposed by #5058 |
I'm tearing my hair out over some of these. Running
pnpm test
locally consistently fails — some of the tests are flaky, others (likehandles external api
) fail reliably, and I'm struggling to understand why. It feels like a fairly recent phenomenon.The
handles external api
case is particularly weird. The server that serves/server-fetch-request-modified.json
is created twice (once forjavaScriptEnabled: true
and once forfalse
), and listens on the same port in both cases. Adding logging shows that the first server is closed before the second server is started, so it's not weird that the original port is still available, but what is weird is that the first server handles the request intended for the second server even though it's already been closed. Because of that, the request isn't intercepted in the right place.I'm sure someone with a better understanding of all this could fix it in no time.
Sometimes tests fail with
ERR_NETWORK_CHANGED
; I have no idea what's causing those.Flaky tests that fail with more specific errors (may add to this list as I find more):