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

Test: Create tests for src/helpers/event/deleteEventHelpers/deleteRecurringEvent.ts #2054

Closed
palisadoes opened this issue Mar 24, 2024 · 1 comment
Assignees
Labels
good first issue Good for newcomers test Testing application

Comments

@palisadoes
Copy link
Contributor

We will need unittests done for all methods, classes and functions found in this file.

Any widgets, components, modals referenced in this file must also have unittests done

PR Acceptance Criteria

@palisadoes palisadoes added good first issue Good for newcomers feature request test Testing application labels Mar 24, 2024
@github-actions github-actions bot added the unapproved Unapproved for Pull Request label Mar 24, 2024
@meetulr
Copy link

meetulr commented Apr 4, 2024

I should take this. This needs to work properly for my upcoming PRs in admin. PalisadoesFoundation/talawa-admin#1643

This PR #2032 was supposed to integrate the deleteRecurringEvent functionalities into develop-userTypeFix. But it removed all the tests😅, so the functionalities remained untested.

I'll add those back with the latest changes for appUserProfile. The PR is ready, please assign this issue to me.

Also, this would cover the tests for deleteRecurringEventInstances.ts (issue #2053), as that file is just exporting a function that's being used only in deleteRecurringEvent.ts (this issue). So, we should close that issue.

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 test Testing application
Projects
Archived in project
Development

No branches or pull requests

3 participants