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

Fix strict and strictEffects not enabling Strict Mode in React 19 #45630

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

bernardobelchior
Copy link
Member

@bernardobelchior bernardobelchior commented Mar 20, 2025

Fix strict and strictEffects not enabling Strict Mode in React 19.

Failing test before the fix: https://app.circleci.com/pipelines/github/mui/material-ui/150321/workflows/b726b7ae-d5cf-4759-bd41-b025f7c04dec/jobs/807845

Bug reproduction opened in React Testing Library: testing-library/react-testing-library#1388

Many tests were failing, so I had to fix them. On most of them, I've disabled strict mode. That solution didn't work for the two Autocomplete.test.tsx tests, so I changed the assertion.

@bernardobelchior bernardobelchior added bug 🐛 Something doesn't work test labels Mar 20, 2025
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Mar 20, 2025
@bernardobelchior bernardobelchior force-pushed the fix-test-utils-strict-mode-react-19 branch from fdf1f5e to f853929 Compare March 20, 2025 07:39
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Mar 20, 2025
@bernardobelchior bernardobelchior force-pushed the fix-test-utils-strict-mode-react-19 branch from f853929 to e4923ef Compare March 20, 2025 07:40
@mui-bot
Copy link

mui-bot commented Mar 20, 2025

Netlify deploy preview

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

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 72b627a

@bernardobelchior bernardobelchior force-pushed the fix-test-utils-strict-mode-react-19 branch from fcc7e44 to 8fcf935 Compare March 21, 2025 09:10
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Mar 27, 2025
@bernardobelchior bernardobelchior force-pushed the fix-test-utils-strict-mode-react-19 branch from e6cea70 to 7f5f581 Compare March 27, 2025 14:53
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Mar 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants