Skip to content

Conversation

@tarun1sisodia
Copy link
Contributor

Backend Test Suite and Coverage

This backend includes an expanded automated test suite with improved coverage compared to the previous baseline on main.

Current branch (chore/test-verification)

  • Test suites: 11 passed / 11 total
  • Tests: 131 total (129 passed, 2 skipped)
  • Coverage (all files):
    • Statements: 40.66%
    • Branches: 30.40%
    • Functions: 36.96%
    • Lines: 40.85%

Baseline (main)

  • Test suites: 5 passed / 5 total
  • Tests: 85 total (83 passed, 2 skipped)
  • Coverage (all files):
    • Statements: 23.73%
    • Branches: 20.64%
    • Functions: 18.95%
    • Lines: 23.89%

Net improvement

  • +6 test suites
  • +46 tests
  • +16.93 pts statements
  • +9.76 pts branches
  • +18.01 pts functions
  • +16.96 pts lines

Notes

  • New tests exercise additional routes (auth, dashboard, learning, health) and unit layers (db, middleware, services).
  • Coverage reports are generated via Jest (npm test -- --coverage).

@crisecheverria
Copy link
Owner

Hi, can you please address the 4 ESLint errors

Screenshot 2025-09-06 at 06 26 47

@tarun1sisodia
Copy link
Contributor Author

GOT it

- Add full-codebase test workflow with parallel backend/frontend testing
- Include pre-checks to detect changes and optimize CI runs
- Add security audits, coverage reporting, and full integration tests
- Support manual triggering with workflow_dispatch
- Provide comprehensive test summary and status reporting
…h and pull_request events; add WARP.md to .gitignore
- Fix rimraf not found error by using npx
- Fix lcov-reporter permission issues with continue-on-error
- Fix React Hook dependency warning using useCallback
- Update path filtering to match backend/frontend workflows
- Improve coverage reporting with better error handling
Copy link
Owner

Choose a reason for hiding this comment

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

Please remove any commit made to add Cursor files/rules

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I am working on it, will push only tests and resolving the 4 ESLint errors.

Copy link
Owner

Choose a reason for hiding this comment

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

Please remove any AI instructions file/document.

Copy link
Owner

Choose a reason for hiding this comment

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

No need to add a full codebase test, want to keep them separate for now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, Got it

Copy link
Owner

Choose a reason for hiding this comment

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

No need to update Readme with tests status

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay.

Copy link
Owner

@crisecheverria crisecheverria left a comment

Choose a reason for hiding this comment

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

Added some comments to the PR

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.

2 participants