-
-
Notifications
You must be signed in to change notification settings - Fork 813
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
Code Coverage: Create tests for src/components/CheckIn/CheckInWrapper.tsx #1031
Comments
hey @palisadoes , can you assign me this issue i wanna work on it |
@duplixx You already have 2 issues in your name. Please get the PRs merged first, then I'll assign more. As per the rules only 2 issues can be assigned at once. |
hey @noman2002 , you can assign me this as my previous PR is merged now!! |
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. |
Please assign this to me. |
Our policy is to assign no more than two issues to each contributor across all repositories. This way everyone gets a chance to participate in the projects. We sometimes give exceptions for more urgent cases and sometimes we lose track, but the policy stands. You have reached your limit, please wait until your existing issues are closed before requesting more issues. You could unassign yourself from one of the other issues too. |
Hey @noman2002 I want to work on this issue! |
Hey ,I want to work on this, please assign me. |
@adi790uu You already have 2 issues assigned to you. |
Het @noman2002, I just found this comment in the test file. If the memory leak is fixed I can uncomment it. Suggestions? |
I tried uncommenting it and the memory leak is still there. I will try to fix that as well. |
@noman2002 Hey, I have been waiting for your input. |
@aashimawadhwa @EshaanAgg Please answer his queries. |
@punitkr03 I had left the comment to show what the prospective test must look like after the memory leak in broken. The test is not full proof as it hasn't been tested in the wild due to the aforementioned leak.
|
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. |
Unassigning due to inactivity and no pull request |
@noman2002 I have read the issue. I can solve it. Could you please assign this to me? |
Hello @noman2002 @palisadoes talawa.mp4but the main branch is working fine |
@DIWAKARKASHYAP You have to run talawa-api either in docker container or manually. So fork |
Did u set your |
yes all variable are setup , the main branch is working properly but develop branch give me this problem |
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. |
Removing assignment due to inactivity |
@palisadoes I'd like to work on this :D |
|
@ashutuptiwari Please fix the memory leak as part of this issue. |
Alright, can you please describe the leak @Cioppolo14 |
If you have discovered that it's still there, you should be able to find it and fix it. |
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. |
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. |
Hello, @noman2002 @Cioppolo14 I would like to work on this issue. Can you please assign it to me? Thanks! |
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
The text was updated successfully, but these errors were encountered: