Skip to content

Cover missing auth header rejection - #193

Merged
chizzy192 merged 1 commit into
StellarState:devfrom
louiss72:agent/missing-auth-header-test-171
Jul 29, 2026
Merged

Cover missing auth header rejection#193
chizzy192 merged 1 commit into
StellarState:devfrom
louiss72:agent/missing-auth-header-test-171

Conversation

@louiss72

Copy link
Copy Markdown
Contributor

What changed

  • Added integration coverage for protected invoice requests sent without an Authorization header.
  • Verifies the existing 401 response shape and confirms invoice service logic is not invoked when auth is missing.

Note

The issue text references /api/invoices and Authentication header missing; this project currently exposes invoices under /api/v1/invoices and uses the existing error message Authorization token is required.. This PR tests the current API behavior without changing the response contract.

Validation

  • npm test -- --runTestsByPath tests/integration/auth-jwt-validation.test.ts
  • npm run type-check
  • npm run lint

Closes #171

@chizzy192 chizzy192 closed this Jul 29, 2026
@chizzy192 chizzy192 reopened this Jul 29, 2026
@chizzy192
chizzy192 merged commit e5c25e5 into StellarState:dev Jul 29, 2026
3 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

Development

Successfully merging this pull request may close these issues.

[TESTING] Add integration test for missing Authorization header rejection

2 participants