-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[BUG] nix and playwright: install-deps is missing required dependencies #16432
Comments
I tried reproducing, but was unable to at this time:
What directory are you running |
i do have a package.json and package-lock.json. file struct looks something like this
running |
🤔 Can you paste the parts of your package.json and package-lock.jons that reference Playwright here? And also include the output of:
as well as:
and
Thanks! |
oops my apologies, i'm still on 1.24.0.
one thing that i noticed is that the tests succeed if I don't run them in nix-shell. I assume the deps are being installed on my machine, but maybe they also need to be avail in nix? |
This comment was marked as outdated.
This comment was marked as outdated.
Hid my last comment, as it's no longer relevant.
I'm not a nix user, but it sounds like some interaction with nix and Playwright is not happy. For now, I recommend running outside of nix-shell since that works 😄 . If you sort out the nix issue, please file a new issue and we can address, otherwise we'll have to continue recommending running outside nix since I don't have the expertise to troubleshoot nix. Thanks! |
yep it seems like i'm running into an issue similar to this: #5501 |
👍 Yeah, I'm not a nix user, but looks like the info in #5806 will help (either the code or some community user comments in that PR). If you're open to it, feel free to contribute a PR doc page of how to use PW and nix! |
Context:
Describe the bug
Tests are failing with the following error
yet when running
npx playwright install-deps
, it's telling me everything's up-to-date and installedHere's my
playwright.config.ts
Ideally
install-deps
should be installing those missing deps, and I don't need to manually sudo apt install every single one.The text was updated successfully, but these errors were encountered: