Skip to content

Conversation

@stevej2608
Copy link
Contributor

@stevej2608 stevej2608 commented Dec 10, 2025

Description

Fix: tests are failing when 'playwright install-deps' has already been done as part of the host/container provisioning.

Checklist

Please update this checklist as you complete each item:

  • Tests have been developed for bug fixes or new functionality.
  • The changelog has been updated, if necessary.
  • Documentation has been updated, if necessary.
  • GitHub Issues closed by this PR have been linked.

By submitting this pull request I agree that all contributions comply with this project's open source license(s).

@Archmonger Archmonger changed the title Fix: tests failing when 'playwright install-deps' has already been do… Fix: tests fail when 'playwright install-deps' has already run Dec 10, 2025
@Archmonger Archmonger merged commit 87ec4a7 into reactive-python:main Dec 10, 2025
20 checks passed
@Archmonger
Copy link
Contributor

I am refactoring the build process in #1317 so I just want to ask for some additional context. In what scenario/environment is install-deps causing issues? I haven't been able to replicate any problems re-running it multiple times on unix/windows/macos.

@stevej2608
Copy link
Contributor Author

I'm developing & testing in a container hosted on WSL2/Ubuntu. playwright is installed and 'playwright install-deps' is run in the Dockerfile.

@Archmonger
Copy link
Contributor

Thanks. I've managed to generate an error with a fresh WSL2 (Ubuntu) instance.

Will presume this is the same issue you had:

The following packages were automatically installed and are no longer required:
  libdrm-nouveau2 libdrm-radeon1 libgl1-amber-dri libglapi-mesa libllvm17t64 libxcb-dri2-0
Use 'sudo apt autoremove' to remove them.

@stevej2608
Copy link
Contributor Author

That message doesn't look familiar.

I'm away for the weekend, I'll reproduce the error and get back to you on Monday. Cheers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants