talawa-admin
talawa-admin copied to clipboard
Code Coverage: Create tests for OrgPost.tsx
The Talawa-Admin code base needs to be 100% reliable. This means we need to have 100% test code coverage.
Tests need to be written for file src/screens/OrgPost/OrgPost.tsx
- When complete, all methods, classes and/or functions in the file will need to be tested.
- These tests must be placed in a single file with the name
src/screens/OrgPost/OrgPost.test.tsx
. You may need to create the appropriate directory structure to do this.
IMPORTANT:
Please refer to the parent issue on how to implement these tests correctly:
- #241
PR Acceptance Criteria
- When complete this file must show 100% coverage when merged into the code base. This will be clearly visible when you submit your PR.
- The current code coverage for the file can be found here. If the file isn't found in this directory, or there is a 404 error, then tests have not been created.
- Create a code coverage account for your repo's preferred branch to generate the values when you do your commits for that branch to your repo.
-
NOTE: Make sure you select the correct branch when you do the setup, or else the reporting will fail. You must also create a
CODECOV
GitHub secret for your repo as part of the process.
-
NOTE: Make sure you select the correct branch when you do the setup, or else the reporting will fail. You must also create a
- The PR will show a report for the code coverage for the file you have added. You can also use that as a guide.
hi can i work on this issue?
@priyang12 if you are not working on this I would like to take this issue @palisadoes sir you can assign me if he is not working on this one
@SiddheshKukade Sure
@palisadoes sir, I would like to work on this issue
Is this issue still open? It's been a month. If no one is working on this, I would like to work on this. @palisadoes
@palisadoes as far as I remember, the codecov for OrgPost.tsx
wasn't reaching up to 100% because of the error in tags attribute that occurs while organisation creation.
But that was raised as an issue by @VenomFate-619 and his PR - https://github.com/PalisadoesFoundation/talawa-api/pull/875 was also merged recently. Hence I think that this issue is resolved and can be closed.
Currently I have my University's end semester examinations hence I will be able to verify this properly only after a few days, if necessary. Kindly excuse me for the same. Thank you.
Is this issue still open? It's been a month. If no one is working on this, I would like to work on this. @palisadoes
Please check the current coverage of this file and let us know. If its below 100% I'll assign it to you.
Sure, thanks!
I'm reassigning. It's been 4 weeks.
@pinkman7009 Any update ? Its been more that 2 weeks.
@noman2002 currently It has around 70% coverage. I can work on this. please assign
This issue did not get any activity in the past 10 days and will be closed in 365 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue.
Unassigning due to inactivity. We need to give others a chance.
@palisadoes Assign me
@palisadoes Kindly assign me this issue
Sorry for the delay I will submit a PR in before 9PM (IST)