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

Refactor: src/screens/OrganizationPeople from Jest to Vitest #2562

Closed
VanshikaSabharwal opened this issue Dec 1, 2024 · 4 comments
Closed
Assignees
Labels
good first issue Good for newcomers refactor Refactor the code to improve its quality test Testing application

Comments

@VanshikaSabharwal
Copy link
Contributor

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

  1. Replace Jest-specific functions and mocks with Vitest equivalents
  2. Ensure all tests in src/screens/OrganizationPeople pass after migration using npm run test:vitest
  3. Maintain the test coverage for the file as 100% after migration
  4. Upload a video or photo for this specific file coverage is 100% in the PR description

Additional context

Potential internship candidates

Please read this if you are planning to apply for a Palisadoes Foundation internship

@abbi4code
Copy link
Contributor

Can i work on this issue ?

@palisadoes palisadoes assigned abbi4code and unassigned abbi4code Dec 11, 2024
@palisadoes
Copy link
Contributor

Unassigning. You already have 2 issues assigned

@abbi4code
Copy link
Contributor

abbi4code commented Dec 11, 2024

I’m really sorry, I wasn’t aware of this. I’ll wait for my existing two PRs to get merged first before asking to be assigned more issues.

@shivasankaran18
Copy link
Contributor

@palisadoes can you assign this to me?..Fixed this locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers refactor Refactor the code to improve its quality test Testing application
Projects
Status: Done
Archived in project
Development

No branches or pull requests

6 participants