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

Test: src/resolvers/User/posts.ts #1928

Closed
palisadoes opened this issue Mar 1, 2024 · 13 comments
Closed

Test: src/resolvers/User/posts.ts #1928

palisadoes opened this issue Mar 1, 2024 · 13 comments
Assignees
Labels
feature request good first issue Good for newcomers test Testing application

Comments

@palisadoes
Copy link
Contributor

The Talawa-API code base needs to be 100% reliable. This means we need to have 100% test code coverage.

  1. Tests need to be written for file src/resolvers/User/posts.ts
  2. 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.
  3. When complete, all all methods, classes and/or functions in the refactored file will need to be tested.
  4. These tests must be placed in a single file with the same path as the tested file, but under the tests/ with a .spec.ts suffix.
  5. 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:

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.

Other

Please coordinate issue assignment and PR reviews with the contributors listed in this issue:

@github-actions github-actions bot added parent Parent issue test Testing application unapproved Unapproved for Pull Request labels Mar 1, 2024
@palisadoes palisadoes added good first issue Good for newcomers and removed parent Parent issue labels Mar 1, 2024
@samiali12
Copy link

I want to work on this issue.

@its-VinayKumar
Copy link

@palisadoes Please Assign me this issue.

@Cioppolo14 Cioppolo14 removed the unapproved Unapproved for Pull Request label Mar 2, 2024
@samiali12
Copy link

There is already test cases is written by someone.

@palisadoes
Copy link
Contributor Author

We need to get the coverage up to 100%

@Abhishek0943
Copy link

I want to work on this issue.

@jyotiv737
Copy link

this issue is already assigned to someone.

Copy link

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.

@github-actions github-actions bot added the no-issue-activity No issue activity label Mar 19, 2024
@Cioppolo14
Copy link
Contributor

Unassigning due to no activity or open PR

Copy link

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.

@github-actions github-actions bot added the no-issue-activity No issue activity label Mar 30, 2024
@Anvita0305
Copy link

Hello, can you please assign this issue to me? thanks!

@github-actions github-actions bot removed the no-issue-activity No issue activity label Apr 15, 2024
@Anvita0305
Copy link

Unassigning myself as won't be able to work due to exams! Sorry for inconvenience

@Anvita0305 Anvita0305 removed their assignment Apr 24, 2024
Copy link

github-actions bot commented May 5, 2024

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.

@github-actions github-actions bot added the no-issue-activity No issue activity label May 5, 2024
@Aaradhy-Sharma
Copy link

Greetings, I'd like to try and solve this issue. Could you please assign it to me
Regards

@varshith257 varshith257 removed the no-issue-activity No issue activity label Jul 26, 2024
Aaradhy-Sharma added a commit to Aaradhy-Sharma/talawa-api that referenced this issue Jul 26, 2024
- Implemented comprehensive test cases for `posts.ts` to ensure 100% code coverage.
- Added tests for various pagination arguments (`first`, `last`, `before`, `after`).
- Included edge cases for no posts and invalid filters.
- Verified error handling for invalid arguments and cursor values.

Fixes PalisadoesFoundation#1928
Aaradhy-Sharma added a commit to Aaradhy-Sharma/talawa-api that referenced this issue Jul 26, 2024
- Implemented comprehensive test cases for `posts.ts` to ensure 100% code coverage.
- Added tests for various pagination arguments (`first`, `last`, `before`, `after`).
- Included edge cases for no posts and invalid filters.
- Verified error handling for invalid arguments and cursor values.

Fixes PalisadoesFoundation#1928
palisadoes pushed a commit that referenced this issue Jul 27, 2024
* fix: issue #2382 by upgrading graphql version

- Updated the `graphql` dependency in `package.json` from version `^16.8.1` to `^16.9.0`
- Regenerated `package-lock.json` to reflect the changes in dependencies

This change addresses issue #2382 by ensuring compatibility with the latest features and bug fixes in the `graphql` package.

Closes #2382

* test: Add tests for 100% coverage of src/resolvers/User/posts.ts

- Implemented comprehensive test cases for `posts.ts` to ensure 100% code coverage.
- Added tests for various pagination arguments (`first`, `last`, `before`, `after`).
- Included edge cases for no posts and invalid filters.
- Verified error handling for invalid arguments and cursor values.

Fixes #1928

* test: Add tests for 100% coverage of src/resolvers/User/posts.ts update-2

* Fix and improve User post resolver tests

- Update 'handles valid cursor value' test to be more flexible
- Ensure test passes with variable post order
- Maintain test coverage and functionality verification
@github-project-automation github-project-automation bot moved this from Backlog to Done in Talawa-API (Tests) Jul 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request good first issue Good for newcomers test Testing application
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

9 participants