Code Coverage for DeleteOrg.tsx added test #1219
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Initial member page made * Changed query added Org and Events section * Homescreen ready * initial translation added for Profile * initial translation added for Profile * Added translations * Added translations * Revised translation * Fixed failing tests for components * Link added to OrgAdminListCard * Linting fixed for OrgPeopleListCard * Added tests, removed logs * Uniform design
* Fix:routing issue for orgdash page * fix:failing test of SuperDashListCard
* Feature Request: Position of the search bar PalisadoesFoundation#614 resolved * Feature Request: Position of the search bar PalisadoesFoundation#614 resolved
…esFoundation#552) * added test for the calendar * updates mocks
…undation#638) * Update Router Function * Empty commit
Signed-off-by: Ekene Nwobodo <[email protected]>
PalisadoesFoundation#539) * Implememnted manage functinality for admin/superadmin after login * Update AdminDashListCard.module.css * Some formatting issues with module file * resolved workflow issues
* added folding feature to posts * added fold feature to posts * brought code under code coverage * added tests * added tests for remainging uncovered lines * brought code coverage to 100% for the changes
…sadoesFoundation#530] (PalisadoesFoundation#657) * fixed updateuser logic * update translations * fix lint * lint fix 2 * fix tests
…on#655) * fixed:move orgLocation & remove duplicate orgName * Dummy commit
… styles (PalisadoesFoundation#603) * fixed: PalisadoesFoundation#585 - Password error showing. Tested * fix: PalisadoesFoundation#590 - fixed navbar hover issues * fix: PalisadoesFoundation#590 - fixed navbar hover issues
* fixed errors and removed window.replace * written test case for ensure window.location.replace * written test case for ensure window.location.replace * added useHistory to keep track of routing behind the scenes * fixed merge conflicts and errors * updated comments for file changes * Added warning comments for routing --------- Co-authored-by: Adepeju <adepeju.kayode@gmail>
Signed-off-by: Ekene Nwobodo <[email protected]>
…in Calendar (PalisadoesFoundation#527) * adds tests for organization events * removes the unrequired branch from OrganizationEvents.tsx * fixes failing tests * fixes failing lint tests
* [fix]- fixed the password error warning * [fix]- fixed the password error warning * [fix]- fixed the password error warning * [fix]-fixed the password error warning * [fix]-fixed the password error warning * [fix]-fixed the password error warning * [fix]-fixed the password error warning * [feat]-added test cases for password error warning * [feat]- added test cases for password error warning * [feat]- added test cases for password error warning * [fix]- fixed the linting
* Improve test coverage for calendar.tsx * Restore previous tests * remove --coverage
…sFoundation#1163) * requests tests completed 100% * resolved warning
* Feat: Cache npm dependencies * Add required changes * Replace with npm ci * Replace with npm install * Remove continue_on_error
…#1154) * feat: updated position of the organization name * Fix: Prevent content of the org card from being cropped when the organization name is long
…ertisementRegister/AdvertisementRegister.tsx (PalisadoesFoundation#1150)
…undation#1082) * requests sorting * added missing test * added missing test * increased test coverage for requests screen * added the reviewed changes and improved tests
* fixed admin card navigation in dashboard * removed the use of any
* Fixed overflow of event registrants * Removed scrollbar
* Update pull-requests.yml * Update pull-requests.yml
…alisadoesFoundation#1165) * fix: improve code coverage for src/screens/OrgPost/OrgPost.test.tsx * fix: improved code coverage for src/components/OrgPostCard/OrgPostCard.tsx
* Added toast on delete action * Achieved 100% test coverage * add confirm model and updated the test * a small commit * added other language
…dation#1191) This reverts commit 15fff09.
…#1166) * Single search modal * fixes failing test * updated the test that were failing * small commit * fixes test
* Added tests for MemberDetail.tsx Signed-off-by: bupd <[email protected]> * Added tests for MemberDetail.tsx Signed-off-by: bupd <[email protected]> --------- Signed-off-by: bupd <[email protected]>
* feat: search on enter press * fixed: test cases for search on enter * fixed test cases & removed debounce.ts
* Imporved events page * Minor retriggering commit * Updated event list design in date * Added test for changed lines * Removed unnecessary code and package * Fixed import
* UI changes in user update page * Fixed some UI bugs. * Fix linting checks
Our Pull Request Approval ProcessWe have these basic policies to make the approval process smoother for our volunteer team. Testing Your CodePlease make sure your code passes all tests. Our test code coverage system will fail if these conditions occur:
The process helps maintain the overall reliability of the code base and is a prerequisite for getting your PR approved. Assigned reviewers regularly review the PR queue and tend to focus on PRs that are passing. ReviewersWhen your PR has been assigned reviewers contact them to get your code reviewed and approved via:
Reviewing Your CodeYour reviewer(s) will have the following roles:
CONTRIBUTING.mdRead our CONTRIBUTING.md file. Most importantly:
Other
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What kind of change does this PR introduce?
Issue Number:
Fixes #1066
Did you add tests for your changes?
yes
Snapshots/Videos:
If relevant, did you update the documentation?
Summary
Handling Deletion Failure
Closing the Delete Organization Modal
Open Delete Organization Modal
Render Delete Button
Does this PR introduce a breaking change?
Other information
Have you read the contributing guide?
yes