Skip to content

Add Tests Cases to View All Category Integration #1

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.

Sign up for GitHub

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

Jump to bottom
Open
jarchenn wants to merge 85 commits into
base: main
Choose a base branch
from
Open

Add Tests Cases to View All Category Integration #1

jarchenn wants to merge 85 commits into from

Conversation

@jarchenn
Copy link
Owner

@jarchenn jarchenn commented Oct 4, 2025 •
edited
Loading

Name of PR: Added Test Cases to ViewAll Category Integration ---

1. Issue

2. Changes

What changes did you make to resolve the issue?

  • updated test/controllers.js to exercise the newly added synthetic “View All” category

Back end:

  • ensure /api/categories exposes viewAllCategory/viewAllList with aggregated topic & post counts that increase when new content is created
  • verify /api/category/all renders through the recent controller with the expected metadata/template payload

Does your change introduce new dependencies to this project? If so, list here.

  • none

fatbiscuit247 and others added 30 commits September 17, 2025 21:52
@fatbiscuit247
found where new topics are created
cf9506c
@fatbiscuit247
add default unresolved status
f1972fd
@fatbiscuit247
find all topic posts and add unresolved label to them
c53df71
@aliciach1
refactored filterTids
3df9d46
@qianxuege
refactor: break isReady into smaller functions to reduce complexity
61e9dde
Adding lab1 changes to file
d575d80
@joon0429
joon p1 changes
ac9f4ad
Added lab1 changes to public/src/api/posts.js
7c381e9
@jarchenn
Merge branch 'main' into feature-part1
97ea57f
@aliciach1
Merge pull request #6 from CMU-313/refactor/is-ready
c4cfcd4 
Refactor (src/user/posts.js): reduce complexity in isReady()
@joon0429
syntax
a9ade86
@joon0429
Merge pull request #5 from aliciach1/main
36d4a43 
refactor: wrapped if-elseif-else statement in a function in filterTids
@fatbiscuit247
add unresolved status
df0ff25
@jarchenn
Merge pull request #8 from CMU-313/joonp1
0aa72df 
Refactor (public/src/modules/accounts/delete.js): reducing parameters in executeAction
@fatbiscuit247
Merge pull request #14 from jarchenn/feature-part1
ad61028 
Refactor src/api/posts.js to simplify Complex binary expression in canSeeVotes()
@qianxuege
Merge pull request #20 from CMU-313/add-unread-feature
79b4b52 
Add unread feature
@qianxuege
fix: revert changes from src/api/posts.js
370c5e2
@qianxuege
fix: revert back to original src/api/posts.js
33c05a6
@qianxuege
fix: cleaned export folder
8fe2a72
@qianxuege
fix: removed public folder from build folder
78f0919
@qianxuege
fix: added jerry's changes back
764474c
@aliciach1
added resolved counts on homepage and category pages on frontend
bf7b01d
@fatbiscuit247
refactor too many params smell
7e13d61
@qianxuege
moved secrets into env file
2c9cbdf
@qianxuege
Merge pull request #25 from CMU-313/refactortoomanyparams
19696ee 
Refactor (src/topics/posts.js:336): Function with many parameters (count = 4): incrementFieldAndUpdateSortedSet
@joon0429
Merge pull request #24 from CMU-313/resolvedCountFrontend
01f37bb 
feat: add unresolved count on homepage and categories page frontend only
@qianxuege
added dependencies
cf1487c
@qianxuege
updated gitignore to exclude build/public folder
9253703
@qianxuege
added cache file in gitignore and added console log test message in e…
46060e5 
…mailer.js
@qianxuege
able to send email via mailgun api, working on the emailer.js test
8253f5e
qianxuege and others added 26 commits September 25, 2025 18:11
@qianxuege
made changes to docker file to remove duplicates
90ae808
@qianxuege
use user's package.json instead of the default
8757425
@qianxuege
reordered copying of plugin, and start new branch to clear any cache
dca17b0
@qianxuege
use the user's package.json, not the default
8ab2fd3
@qianxuege
change test.yaml to avoid it overwriting package.json
60258c8
@qianxuege
insert test database only in CI
5a5e04b
@qianxuege
corrected test db value
e1b42a2
@qianxuege
unconditionally add test db
038748e
@jarchenn
debugged tests and schema
f6e2b91
@qianxuege
added pr template for more detailed documentation
34eb9b3
@qianxuege
Merge pull request #44 from CMU-313/pr-template
f80009e 
Pull Request Template
@aliciach1
Merge pull request #43 from CMU-313/feat/email-docker
5b727b7 
Feat/email docker
@jarchenn
Remove build directory from version control
d86fc89
@jarchenn
Restore .gitignore for build folders and remove tracked build files
75fb24a
@fatbiscuit247
feat: initialize back end resolved data status
b87d0e1
@jarchenn
Merge branch 'main' into feature-part1
8843505
@fatbiscuit247
added resolved status backend and basic utility file for resolved fun…
a8ec9f2 
…ctions
@fatbiscuit247
added api endpoint to get resolved status and added resolved field to…
0920219 
… openapi schemafile so all topics have resolved field
@fatbiscuit247
fix openapi test schema validation error
9f8ea8b
@fatbiscuit247
feat: api fix api scheme validation error running build again
59f4b69
@fatbiscuit247
feat: add put endpoint to update resolved status
5758d6a
@fatbiscuit247
feat: add front end only admin can resolve and delete resolved.js file
9623253
@fatbiscuit247
test: add testcases for backend resolved status feature
4b445bc
@fatbiscuit247
Merge pull request #46 from CMU-313/feature-part1
8b26e39 
Adding View All Category to NodeBB Homepage
@qianxuege
Merge pull request #57 from CMU-313/add-onlycoursestaffresolves
9984182 
 feat: initialize back end resolved data status
@jarchenn
test: add comprehensive test coverage for View All category and resol…
8c82bad 
…ved status features
@jarchenn jarchenn changed the title Add test view all Add Tests Cases to View All Category Integration Oct 4, 2025
@jarchenn jarchenn self-assigned this Oct 4, 2025
@jarchenn jarchenn added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jarchenn jarchenn

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jarchenn @fatbiscuit247 @aliciach1 @qianxuege @joon0429