-
-
Notifications
You must be signed in to change notification settings - Fork 878
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: src/resolvers/middleware/currentUserExists.ts #1818
Comments
I would like to work on this issue |
@NayOoLwin5 You now have two issues. 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. |
@Cioppolo14 I would like to work on this issue |
Is there a command to run test on specified only instead of all the files? |
I could not write the tests, my apologies. |
Can I be assigned this issue? |
@aarishshahmohsin |
I would like to work on this issue |
@aarishshahmohsin I am going to assign you this issue, I only see that you have one issue. I think we had some confusion, you are allowed a maximum of 2 issues across all repositories with a few exceptions. Thank you for your understanding, we are working hard to keep the flow going! |
The Talawa-API code base needs to be 100% reliable. This means we need to have 100% test code coverage.
src/resolvers/middleware/currentUserExists.ts
tests/
with a.spec.ts
suffix.IMPORTANT:
Please refer to the parent issue on how to implement these tests correctly:
PR Acceptance Criteria
codecov.io
to help with testing.Other
Please coordinate issue assignment and PR reviews with the contributors listed in this issue:
The text was updated successfully, but these errors were encountered: