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

Added tests and wrote documentation for post endorsement feature #40

Merged
merged 4 commits into from
Feb 27, 2025

Conversation

joonhoswe
Copy link
Contributor

@joonhoswe joonhoswe commented Feb 27, 2025

I wrote tests for the new post endorsement feature and also wrote documentation in UserGuide.md to inform the users on what new features were added, how they're being tested, and where to find the tests.

test/flags.js
Overview: Contains numerous tests for the various types of flags available
My Additions: Added tests for endorsing a post, checking its status to verify it was endorsed, removing an endorsement, and verifying its able to be re-endorsed again.

UserGuide.md
Overview: Contains all the features we've added over the last couple of sprints, how we tested them, and where users can find those tests
My Additions: Added user documentation for the purposes mentioned above.

CoverAlls Result after adding new tests:
Screenshot 2025-02-26 at 10 04 07 PM

@joonhoswe joonhoswe changed the title test: added tests for post endorsement feature Added tests and wrote documentation for post endorsement feature Feb 27, 2025
@joonhoswe joonhoswe self-assigned this Feb 27, 2025
@joonhoswe joonhoswe added the documentation Improvements or additions to documentation label Feb 27, 2025
@joonhoswe joonhoswe added this to the Sprint 2 milestone Feb 27, 2025
@coveralls
Copy link

Pull Request Test Coverage Report for Build 13557950926

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 82.639%

Totals Coverage Status
Change from base Build 13557881240: 0.0%
Covered Lines: 22418
Relevant Lines: 25705

💛 - Coveralls

Copy link
Contributor

@savannahxunc savannahxunc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Your tests and notes in the 'UserGuide.md' are very comprehensive.

@savannahxunc savannahxunc merged commit b59ce94 into main Feb 27, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants