Skip to content

feat: add supertest+testcontainers integration tests for /api/auth - #717

Open
ololadedavidvictor-bit wants to merge 1 commit into
Predictify-org:mainfrom
ololadedavidvictor-bit:feat/auth-integration-tests-573
Open

feat: add supertest+testcontainers integration tests for /api/auth#717
ololadedavidvictor-bit wants to merge 1 commit into
Predictify-org:mainfrom
ololadedavidvictor-bit:feat/auth-integration-tests-573

Conversation

@ololadedavidvictor-bit

@ololadedavidvictor-bit ololadedavidvictor-bit commented Jul 28, 2026

Copy link
Copy Markdown

Closes #573

Summary

  • Replace mocked auth integration tests with real Testcontainers Postgres end-to-end coverage
  • Add focused tests for POST /api/auth/challenge, /verify, /refresh, /logout, /wallet/logout
  • Validate standardized error envelopes, correlation IDs, and DB persistence
  • Fix missing logger import in rateLimit middleware (bug fix)
  • Fix missing imports in tags router and duplicate declarations in markets router (bug fix)

Test Plan

  • 42 integration tests passing against Testcontainers Postgres
  • Input validation at the boundary; standardized error envelope
  • Structured logging with correlation IDs
  • Clear documentation and inline comments

- Replace mocked auth integration tests with real Testcontainers Postgres
- Add end-to-end coverage for challenge, verify, refresh, logout, wallet/logout
- Validate standardized error envelopes, correlation IDs, and DB persistence
- Fix missing logger import in rateLimit middleware
- Fix missing imports in tags router and duplicate declarations in markets router
- Closes Predictify-org#573
@drips-wave

drips-wave Bot commented Jul 28, 2026

Copy link
Copy Markdown

@ololadedavidvictor-bit Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

Add integration test for /api/auth [b#004]

1 participant