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

UnVerify API Tests #32

Merged
merged 1 commit into from
Oct 10, 2024
Merged

UnVerify API Tests #32

merged 1 commit into from
Oct 10, 2024

Conversation

KesterTan
Copy link

@KesterTan KesterTan commented Oct 10, 2024

Description

This PR adds on to the verify message component that was previously merged in by #20

Previously, verify was tested but unverify was not tested. This PR makes it so that unverify is also tested and resolved #31

This PR tests the un-verify API call and make some UI changes so that unverify-message appears on the UI.

Testing

This was tested by running npm run test and npm run lint and making sure that all tests cases passed as expected.

@KesterTan KesterTan added the Message verified function These issues relate to the ability to mark messages as verified. label Oct 10, 2024
@KesterTan KesterTan added this to the Sprint 2 milestone Oct 10, 2024
@KesterTan KesterTan self-assigned this Oct 10, 2024
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11279938200

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 82.637%

Totals Coverage Status
Change from base Build 11260059113: 0.02%
Covered Lines: 22368
Relevant Lines: 25645

💛 - Coveralls

Copy link

@philliparaujo philliparaujo left a comment

Choose a reason for hiding this comment

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

Lgtm! Good job

@KesterTan KesterTan merged commit 72ab338 into f24 Oct 10, 2024
1 check passed
@KesterTan KesterTan deleted the verified-tests branch October 10, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Message verified function These issues relate to the ability to mark messages as verified.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add unit tests to test unverify functionality
3 participants