fix: resolve failing backend tests for auth routes and user model (#278)#286
fix: resolve failing backend tests for auth routes and user model (#278)#286JayRathore10 wants to merge 2 commits into
Conversation
✅ Deploy Preview for github-spy ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughThis PR refactors the User model's password middleware from callback-based ( ChangesUser Password Middleware Modernization
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
🎉 Thank you @JayRathore10 for your contribution. Please make sure your PR follows https://github.com/GitMetricsLab/github_tracker/blob/main/CONTRIBUTING.md#-pull-request-guidelines
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@package.json`:
- Line 46: The listed auth/session libraries (bcryptjs, passport,
passport-local, express-session) are runtime backend deps but currently live in
devDependencies; edit package.json to remove these four entries from
devDependencies and add them under dependencies with their existing version
strings, then run your package manager (npm install or yarn) to update
package-lock.json/yarn.lock so production installs include them; verify no other
code references remain that assume dev-only usage and that the server starts
correctly.
In `@spec/auth.routes.spec.cjs`:
- Line 39: The test suite currently calls
mongoose.connect('mongodb://127.0.0.1:27017/github_tracker_test') which shares a
fixed database with other specs and causes teardown races when dropDatabase() is
called; change the connect call so this suite uses an isolated database name
(e.g., append a unique suffix like process.pid, Date.now(), a UUID, or the test
worker id) so that mongoose.connect(...) creates a per-suite DB and
dropDatabase() only affects that DB; update any teardown that references
dropDatabase() to operate on the same unique DB name (referencing the
mongoose.connect call and the existing dropDatabase() call to ensure they use
the same derived name).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 851efbc1-4313-46a5-a04d-fba04346b05c
📒 Files selected for processing (4)
backend/models/User.jspackage.jsonspec/auth.routes.spec.cjsspec/user.model.spec.cjs
|
@JayRathore10 - pls resolve conflicts |
|
Conflicts have been resolved. Kindly check and review the PR again. |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@package.json`:
- Line 15: The package.json contains a duplicate scripts.test key that
overwrites the earlier entry (the two values are "vitest" and "jasmine"); update
scripts so both test runners are preserved—either rename one (e.g. "test:unit":
"vitest" and "test:legacy": "jasmine") and keep "test" as a harness that runs
both, or make "test" a combined command (using && or a runner like npm-run-all)
that invokes "vitest" and "jasmine"; adjust the scripts object entries
(referencing scripts.test, "vitest", and "jasmine") accordingly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
Conflicts have been resolved. Kindly check and review the PR again. |
2 participants
🔧 Fix: Resolve failing backend test suites for Auth Routes and User Model (#278)
📌 Overview
This PR fixes multiple failing backend test cases related to Auth Routes and the User Model. The root cause was incorrect usage of Mongoose middleware, which caused errors during user creation and authentication flows.
🐛 Issues Identified
TypeError: next is not a functionin User model pre-save hook500instead of201✅ Fixes Implemented
nextusage)comparePasswordmethod for accurate authentication🧪 Test Results
📂 Files Modified
backend/models/User.jsspec/user.model.spec.cjsspec/auth.routes.spec.cjs🏷️ Open Source Context
This PR is submitted as part of GSSoC (GirlScript Summer of Code) contribution.
👤 Contributor Profile
GSSoC Profile: https://gssoc.girlscript.org/profile/e47b2f4a-f3e9-4cb4-97b5-ddd8cb15a1e9
🚀 Impact
This fix ensures:
Screenshot of
npm run test📎 Notes
No breaking changes introduced. Existing functionality remains intact while fixing test stability and backend logic.
Summary by CodeRabbit
Dependencies
Tests
Chores