-
-
Notifications
You must be signed in to change notification settings - Fork 865
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 Code Coverage GitHub Action Test for Values > 0% #3348
Comments
I am thinking once all jest files are converted to vitest, we can re-enable it by remove those all flags introduced Let's put priority on that refactor issues and Prs and close them ASAP |
I'll assign it to you so we don't forget How far away are we from completing the vitest conversion? |
I think only 3-4 issues left |
You should take them over and complete them if there is no response. |
This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue. |
Pull-request.yml has statements related to the simultaneous user of jest and vitest. These will need to be removed |
|
I will look into this |
@varshith257 Have you managed to make any progress? |
Describe the bug
jest
tovitest
for our test suiteTo Reproduce
Here is an example of a failing test suite when the threshold value was set to 89%
Expected behavior
Actual behavior
Screenshots
Merging of the code coverage files from jest and vitest
The failing check section. Failures only happen when not set to zero
Additional details
Add any other context or screenshots about the feature request here.
Potential internship candidates
Please read this if you are planning to apply for a Palisadoes Foundation internship
The text was updated successfully, but these errors were encountered: