Skip to content

WIP: Updated getTopicsByDate tests to reflect new changes to it#45

Open
AlexDaGreat555 wants to merge 1 commit into
mainfrom
filter-by-date-tests
Open

WIP: Updated getTopicsByDate tests to reflect new changes to it#45
AlexDaGreat555 wants to merge 1 commit into
mainfrom
filter-by-date-tests

Conversation

@AlexDaGreat555
Copy link
Copy Markdown

@AlexDaGreat555 AlexDaGreat555 commented Oct 11, 2025

Context:
Previously, I made some changes to my backend helper method that would query the database for topics based on the specified category, user id, and date provided. I have now updated the test suite to reflect this

What I changed:

  • I updated the test cases for the getTopicsByDate method in the tests/topics.js file. All tests successfully passed, reflecting accuracy in updated changes

To test, run npx mocha test/topics.js

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 18423269380

Details

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • 8 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.007%) to 78.213%

Files with Coverage Reduction New Missed Lines %
src/controllers/admin/info.js 2 71.76%
src/activitypub/index.js 6 56.44%
Totals Coverage Status
Change from base Build 18422444793: -0.007%
Covered Lines: 25046
Relevant Lines: 30113

💛 - Coveralls

Copy link
Copy Markdown

@dsblack0 dsblack0 left a comment

Choose a reason for hiding this comment

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

Tests have all passed, and the code looks correct so far and with good style.

@dsblack0 dsblack0 changed the title Updated getTopicsByDate tests to reflect new changes to it WIP: Updated getTopicsByDate tests to reflect new changes to it Oct 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants