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

Refactor: tests/resolvers/GroupChatMessage #969

Closed
anshgoyalevil opened this issue Feb 1, 2023 · 2 comments · Fixed by #977
Closed

Refactor: tests/resolvers/GroupChatMessage #969

anshgoyalevil opened this issue Feb 1, 2023 · 2 comments · Fixed by #977
Assignees
Labels
good first issue Good for newcomers unapproved Unapproved for Pull Request

Comments

@anshgoyalevil
Copy link
Contributor

anshgoyalevil commented Feb 1, 2023

This is a child issue to the issue listed here: #965

  • Code need to be refactored for folder tests/resolvers/GroupChatMessage
  • We will need the API to be refactored for all methods, classes and/or functions found in this file for testing to be correctly executed.
  • When complete, all all methods, classes and/or functions in the refactored file will need to be tested.
  • Refer to the parent issue on getting an insight on how this issue need to be resolved.

IMPORTANT:

Please refer to the parent issue on how to refactor the code correctly.

Parent Issue

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 by visting this page. Login using your GitHub credentials.
  • Create your own codecov.io to help with testing.
  • The PR you create will show a report for the code coverage for the file you have added. You can also use that as a guide.
@github-actions github-actions bot added parent Parent issue unapproved Unapproved for Pull Request labels Feb 1, 2023
@anshgoyalevil
Copy link
Contributor Author

@EshaanAgg You can take on this

@EshaanAgg
Copy link
Contributor

I would like to work on this 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 unapproved Unapproved for Pull Request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants