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
fix: also kill Firefox when temporary profile is used #8233
Conversation
There are quite some failures visible on Windows so we most likely have to delay landing of this PR. |
@OrKoN as we discussed we would still like to get this workaround removed given that the underlying issue in Firefox is really fixed. Failures in the launcher spec tests are most likely caused by some combination of running these in sequence. As such I would propose (for now) to just separate out the |
LGTM thanks! |
@OrKoN I just noticed that we have issues locally with our expected results JSON file which is keyed by test name. With having the same test name twice now (once for Chrome and one more for Firefox) we cannot explicitly mark the expected data due to duplicated keys. I would suggest that we slightly rename the Firefox test so that the name is different if that's ok. |
This particular check had been added because of issues with killing the Firefox process on Windows. But that was fixed in Firefox 96 and we should be able to remove the workaround.
Lets see if CI is happy with it.