Skip to content

test: controller specs and auth e2e suite (#218, #219, #220, #221) - #361

Merged
EmmanuelOchaje merged 5 commits into
StellarTickets:mainfrom
precious-akpan:feat/218-219-220-221-controller-and-auth-e2e-tests
Sep 26, 2026
Merged

EmmanuelOchaje merged 5 commits into
StellarTickets:mainfrom
precious-akpan:feat/218-219-220-221-controller-and-auth-e2e-tests

Conversation

@precious-akpan

Copy link
Copy Markdown
Contributor

Summary

Controller specs share test/helpers/controller-app.ts, which boots one controller over HTTP with a mocked service, the app's ValidationPipe settings and a fake JWT guard.

Test evidence

  • npx jest: 67 suites, 538 tests pass.
  • Controller coverage for the four controllers: 100% statements, functions and lines (branch % is decorator metadata).
  • jest --config test/jest-e2e.json test/auth against Postgres 16: 13/13 pass.
  • New files pass eslint and prettier.

Notes

  • StellarService is overridden in the auth e2e (same as events.e2e-spec.ts); auth never touches the chain.
  • package-lock.json is out of sync with package.json on main (npm ci fails), unrelated to this PR and left untouched.
  • CHANGELOG updated.

Closes #218
Closes #219
Closes #220
Closes #221

@netlify

netlify Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for stellarticketsbackend ready!

Name Link
🔨 Latest commit d3e60c7
🔍 Latest deploy log https://app.netlify.com/projects/stellarticketsbackend/deploys/6ab72cc0b95d7100087724e5
😎 Deploy Preview https://deploy-preview-361--stellarticketsbackend.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@drips-wave

drips-wave Bot commented Sep 26, 2026

Copy link
Copy Markdown

@precious-akpan 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

@EmmanuelOchaje
EmmanuelOchaje merged commit 837d94e into StellarTickets:main Sep 26, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants