-
-
Notifications
You must be signed in to change notification settings - Fork 126
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] Upgrade mui material try to fix pb with jest #2496
[FIX] Upgrade mui material try to fix pb with jest #2496
Conversation
✅ Deploy Preview for eclipsefdn-adoptium ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2496 +/- ##
=======================================
Coverage 99.20% 99.20%
=======================================
Files 87 87
Lines 6541 6541
Branches 552 552
=======================================
Hits 6489 6489
Misses 52 52 ☔ View full report in Codecov by Sentry. |
Ok this is not concluant - same error ... |
Ok may be it's 👍 ReferenceError: jest is not defined from code : |
The error is coming from vitest-canvas-mock (which in turn depends on jest-canvas-mock. There is an open issue upstream to hopefully fix this |
What I don't understand is why it works perfectly on my computer and it fails everytime in actions... |
Description of change
Checklist
npm test
passes