Skip to content

[test] Setup tests on all browsers #1592

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

Merged
merged 13 commits into from
Apr 30, 2025

Conversation

michaldudak
Copy link
Member

@michaldudak michaldudak commented Mar 21, 2025

Prepare npm scripts for running the test in each supported browser (Chromium, Firefox and WebKit) separately or in all at once.

I initially intended to run all browsers on CI, but they are quite unstable.

Copy link

netlify bot commented Mar 21, 2025

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit 024ed1a
🔍 Latest deploy log https://app.netlify.com/sites/base-ui/deploys/6809eafde02d240008cfe6b4
😎 Deploy Preview https://deploy-preview-1592--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@michaldudak
Copy link
Member Author

Firefox connection timeouts look like vitest-dev/vitest#7377

@michaldudak michaldudak marked this pull request as ready for review March 21, 2025 14:06
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 1, 2025
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 24, 2025
@@ -43,11 +43,16 @@
"test:regressions:dev": "vite --config test/regressions/vite.config.mjs",
"test:regressions:run": "cross-env VITEST_ENV=chromium vitest --project regressions",
"test:regressions:server": "serve test/regressions -p 5173",
"test:jsdom": "cross-env NODE_ENV=test VITEST_ENV=jsdom vitest --project @base-ui-components/react --project docs",
"test:_unit": "cross-env NODE_ENV=test vitest --project @base-ui-components/react --project docs",
Copy link
Member

Choose a reason for hiding this comment

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

Is the underscore in test:_unit intentional?

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, to indicate it's a subtask, not meant to be used directly

@michaldudak michaldudak merged commit eb15fcf into mui:master Apr 30, 2025
22 checks passed
@michaldudak michaldudak deleted the setup-tests-on-all-browsers branch April 30, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants