-
-
Notifications
You must be signed in to change notification settings - Fork 813
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
Refactor: src/setupTests.ts
from Jest to Vitest
#2753
Comments
Hy @palisadoes , can you assign this issue to me ? |
If no one is currently working on this issue, I would like to take it on. Please assign it to me. |
@Cioppolo14 If there is no activity on this issue, I would be happy to work on it. |
@amaan-aly246 I just assigned you 2 issues, so you now have reached your limit. Our policy is to assign no more than two issues to each contributor across all repositories. This way everyone gets a chance to participate in the projects. We sometimes give exceptions for more urgent cases and sometimes we lose track, but the policy stands. You have reached your limit, please wait until your existing issues are closed before requesting more issues. You could unassign yourself from one of the other issues too. |
After refactoring the file, all Jest test cases will fail, while all Vitest test cases will pass successfully. |
@palisadoes , can I be assigned this issue , i want to work on this one |
Should this be closed now? |
Yes @palisadoes |
Is your feature request related to a problem? Please describe.
This file is currently tested using Jest. As part of our migration to Vitest, we need to refactor this file's test cases to align with Vitest's syntax and features.
Acceptance Criteria
.test.*
suffix to a.spec.*
suffixnpm run test:vitest
Additional context
https://github.com/PalisadoesFoundation/talawa-admin/pull/2457/files#diff-9a5a35cf8d7f02b2bba90905714b762560579c4789e9eb4cebb4607980aa3d93
https://vitest.dev/
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: