Skip to content

test: cover transcript authorization errors - #2625

Open
Chris0Jeky wants to merge 1 commit into
mainfrom
issue-1871/auth-error-coverage
Open

test: cover transcript authorization errors#2625
Chris0Jeky wants to merge 1 commit into
mainfrom
issue-1871/auth-error-coverage

Conversation

@Chris0Jeky

Copy link
Copy Markdown
Owner

Refs #1871

Summary

  • Add coverage for both HTTP 401 and 403 transcript authorization failures using the real transport error shape.
  • Assert the English unauthorized catalog value and the Spanish translation after a live locale switch.
  • Restore the prior locale even if the test fails.

Verification

  • TranscriptEvidenceViewer.spec.ts: 11/11
  • Frontend typecheck: passed
  • File-scoped ESLint: passed
  • Mutation check: removing 403 from the production mapping made the HTTP 403 test receive the generic error and fail; production code was restored.
  • git diff --check: passed

Two coordinator review passes completed for coverage correctness and scope/regression quality.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pending

Development

Successfully merging this pull request may close these issues.

1 participant