-
Notifications
You must be signed in to change notification settings - Fork 38
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
Don't work with Playwright 1.38 #327
Comments
Thanks for reporting, we will take a look into this one! If you would have any other questions regarding lost pixel - feel free to ping me anytime 🙌🏼 |
Hi @d-ivashchuk! Any news here? |
Same problem here with playwright 1.40.0 In my case the folder name is |
Seems like this is caused by listing |
I am sorry for not providing enough attention to this! will try to reproduce it once more. @greenskybluephish as you already got some notion on what's going on, do you want to give it a run as a PR fix? |
Experiencing the same, reproducable with:
Results in error |
@chbiel if you wanna give fix a go - very much welcome. Otherwise we will tackle it over the weekend! |
@d-ivashchuk I tried, but following the contribution guide does also not work right now :/ |
would love to assist you with any troubles(and improve the docs for contribution guide at the same time :D) what challenges did you face? |
I'm very sorry for the late reply. I've created a tiny beta npm version to test that theory. This is the version:
Of course, this requires you to also install your own |
Hey guys, had same issue, found a solution I think. Basically when you run And in the latest version it has different browsers.json file with default install version of chrome. So the most sustainable solution is to run Otherwise you can modify your command to use I guess alternatively you could configure your own browsers.json I'm not sure, first time using playwright |
Yes, @Maxim-Mazurok, this solution works too, but there's a drawback: Of course, some projects use Playwright just via Lost Pixel and that's fine, but if a project wants to run Playwright in a new version (to e.g. create integration tests) it would become a problem. That's why I moved Maybe you could give |
Ah I see, makes sense, didn't read thread in detail. Sorry, don't have bandwidth for now, also not using playwright for anything other than lost-pixel for now, so happy with my setup, will be happy to try if anything changes and will report back, cheers! |
It seems like there is still an issue here, or am I doing something wrong? In my
When running
When checking the browsers directory, I see a newer |
Bug description
In my project I'm using
"@playwright/browser-chromium": "^1.39.0",
"@playwright/browser-firefox": "^1.38.0",
"@playwright/browser-webkit": "^1.38.0",
"@playwright/test": "^1.38.0"
When I tried to run Lost Pixel I received
❌ browserType.launch: Executable doesn't exist at. ../ms-playwright/chromium-1076/chrome-mac/Chromium.app/Contents/MacOS/Chromium
Suggested
npx playwright install
didn't help, but I was able to fix it with workaroundin the folder
/ms-playwright/
I copied existing/ms-playwright/chromium-1080/
, paste it and rename it to chromium-1076I bet upgrading Playwright to 1.38 on you package.json should solve this globally.
How to reproduce
Expected behavior
No response
Lost Pixel information
lost-pixel logs from CI
No response
The text was updated successfully, but these errors were encountered: