Review tests

[PierreBrisorgueil]

No description provided.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.07%. Comparing base (c71e51c) to head (b22b459).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3127      +/-   ##
==========================================
+ Coverage   79.03%   88.07%   +9.03%     
==========================================
  Files          60       51       -9     
  Lines        1202     1115      -87     
  Branches      236      213      -23     
==========================================
+ Hits          950      982      +32     
+ Misses        221      118     -103     
+ Partials       31       15      -16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull request overview

This pull request significantly improves test coverage across the application by adding comprehensive error handling tests, edge case coverage, and Jest configuration enhancements. The changes introduce systematic testing of error paths using Jest mocking, add missing test scenarios, and establish coverage thresholds to maintain code quality.

Changes:

• Added error handling tests for all major modules (users, auth, tasks, uploads, home) using Jest spies to mock service failures
• Enhanced Jest configuration with coverage collection, thresholds (85% statements/functions/lines, 75% branches), and well-documented exclusions
• Added missing test cases for OAuth flows, password reset, pagination validation, GitHub API mocking, and core service utilities
• Fixed test bugs (object mutation in uploads purge test, removed unnecessary bootstrap in unit tests)

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
jest.config.js	Enabled coverage collection with reasonable thresholds and comprehensive exclusion patterns for non-testable code (models, configs, OAuth strategies)
modules/users/tests/user.admin.integration.tests.js	Added error handling tests for list/update/remove operations, non-existent user lookup, and pagination validation
modules/users/tests/user.account.integration.tests.js	Added comprehensive error tests for account operations (terms, password, avatar, data), mismatched password validation, and bio/position field coverage
modules/uploads/tests/uploads.integration.tests.js	Added error tests for stream/remove operations, middleware error handling, and fixed object mutation bug in purge test
modules/tasks/tests/tasks.integration.tests.js	Added error handling tests for CRUD operations and stats endpoint failures
modules/home/tests/home.integration.tests.js	Mocked GitHub API to avoid network requests, added error tests for team/page services, and tested Authorization header with tokens
modules/core/tests/core.unit.tests.js	Removed unnecessary bootstrap call, added tests for config helpers, Express middleware error handling, mongoose service, and auth service utilities
modules/core/tests/core.integration.tests.js	Added seed service tests with log output verification and single user seeding
modules/auth/tests/auth.integration.tests.js	Added OAuth profile tests (create/find user), password reset flow tests, wrong password validation, and error path coverage

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.