Skip to content
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

[code-infra] Explain how to install the browsers #40474

Merged

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Jan 7, 2024

It wasn't so obvious to me how to debug:

Screenshot 2024-01-07 at 19 57 03

Applies to MUI X, Material UI, etc.

This is required now because of the change made in https://github.com/microsoft/playwright/releases/tag/v1.38.0. A much better solution would be to catch the error and log the right error message but I didn't have the time to look into this.

@oliviertassinari oliviertassinari added test scope: code-infra Specific to the core-infra product labels Jan 7, 2024
@mui-bot
Copy link

mui-bot commented Jan 7, 2024

Netlify deploy preview

https://deploy-preview-40474--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against d9340ab

- Headless Firefox
- Chrome, Safari, and Edge thanks to [BrowserStack](https://www.browserstack.com)

In development mode, if `pnpm test:karma` fails with this error "Cannot start ChromeHeadless. Can not find the binary", you can solve it by installing the missing headless browsers: `pnpm playwright install --with-deps`.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copies:

command: pnpm playwright install --with-deps

@oliviertassinari oliviertassinari changed the title [test] Explain how to install the browsers [code-infra] Explain how to install the browsers Jan 7, 2024
@oliviertassinari oliviertassinari merged commit fa0c44e into mui:master Jan 22, 2024
21 checks passed
@oliviertassinari oliviertassinari deleted the document-how-to-fix-karma-test branch January 22, 2024 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: code-infra Specific to the core-infra product test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants