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.

Sign up for GitHub

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

Jump to bottom

Refactor: src/screens/OrganizationEvents.tsx from Jest to Vitest #2559

Closed
VanshikaSabharwal opened this issue Dec 1, 2024 · 6 comments
Closed

Refactor: src/screens/OrganizationEvents.tsx from Jest to Vitest #2559

VanshikaSabharwal opened this issue Dec 1, 2024 · 6 comments
Assignees
@Ramneet04
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/OrganizationEvents.tsx 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
@github-actions github-actions bot added documentation Improvements or additions to documentation refactor Refactor the code to improve its quality test Testing application unapproved labels Dec 1, 2024
@palisadoes palisadoes added good first issue Good for newcomers and removed documentation Improvements or additions to documentation feature request labels Dec 3, 2024
This was referenced Dec 9, 2024
Merged
Closed
Closed
Merged
@github-actions GitHub Actions
Copy link

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.

@github-actions github-actions bot added the no-issue-activity No issue activity label Dec 14, 2024
@it1b2313201
Copy link

I want to get the code

@it1b2313201
Copy link

I want to work with you.

This was referenced Dec 14, 2024
Merged
Merged
Merged
@github-actions github-actions bot removed the no-issue-activity No issue activity label Dec 15, 2024
This was referenced Dec 18, 2024
Merged
Merged
Closed
Merged
@coderabbitai coderabbitai bot mentioned this issue Dec 25, 2024
Merged
@Ramneet04
Copy link
Contributor

Is this issue resolved, if not so can I work on this, assign it to me.

@palisadoes
Copy link
Contributor

@it1b2313201

Unassigning due to inactivity and/or no submitted PR. Others need to get a chance. Please let us know whether you want to be reassigned before someone else requests assignment or select another issue.

@palisadoes palisadoes assigned Ramneet04 and unassigned it1b2313201 Dec 26, 2024
@Ramneet04
Copy link
Contributor

hi! Just a quick update, getting some test fail trying to figure out just need time, will raise pr soon

@Ramneet04 Ramneet04 mentioned this issue Dec 28, 2024
Merged
palisadoes pushed a commit that referenced this issue Dec 29, 2024
@Ramneet04
Refactored src/screens/OrganizationEvents.tsx from Jest to Vitest #2559…
1bc3c5e 
… (#3013)

* file name changed

* mocking window object

* migration success

* fixing datetimepicker error

* datepicker error fixed

* fixed the datepickerError
@github-project-automation github-project-automation bot moved this from Backlog to Done in Talawa Admin - Refactoring Dec 29, 2024
@github-project-automation github-project-automation bot moved this from Backlog to Done in Talawa-Admin (Tests) Dec 29, 2024
Dhiren-Mhatre pushed a commit to Dhiren-Mhatre/talawa-admin that referenced this issue Dec 30, 2024
@Ramneet04 @Dhiren-Mhatre
Refactored src/screens/OrganizationEvents.tsx from Jest to Vitest Pal…
52a2ad5 
…isadoesFoundation#2559 (PalisadoesFoundation#3013)

* file name changed

* mocking window object

* migration success

* fixing datetimepicker error

* datepicker error fixed

* fixed the datepickerError
This was referenced Dec 31, 2024
Merged
Merged
@coderabbitai coderabbitai bot mentioned this issue Jan 10, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ramneet04 Ramneet04

Labels
good first issue Good for newcomers refactor Refactor the code to improve its quality test Testing application
Projects
Talawa Admin - Refactoring
Status: Done
Talawa-Admin (Tests)
Archived in project
Milestone
No milestone
Development

No branches or pull requests
4 participants
@palisadoes @VanshikaSabharwal @Ramneet04 @it1b2313201