Skip to content

fix(oauth): correct typo in refresh token test description#20800

Merged
dschom merged 1 commit into
mozilla:mainfrom
sreecharan-desu:fix/oauth-test-typo-separately
Jul 10, 2026
Merged

fix(oauth): correct typo in refresh token test description#20800
dschom merged 1 commit into
mozilla:mainfrom
sreecharan-desu:fix/oauth-test-typo-separately

Conversation

@sreecharan-desu

Copy link
Copy Markdown
Contributor

Because

  • An OAuth integration test description misspelled "separately" as "seperately".
  • No linked issue; this is a one-character test string correction with no runtime impact.

This pull request

  • Corrects the test description in oauth_api.in.spec.ts with no other changes.

Issue that this pull request solves

Closes: (none — test typo only)

Checklist

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally. — N/A: test string only.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate). — N/A
  • I have manually reviewed all AI generated code.

How to review (Optional)

  • Key files/areas to focus on: packages/fxa-auth-server/test/remote/oauth_api.in.spec.ts (single-line change)
  • Suggested review order: verify spelling change only
  • Risky or complex parts: none

Test plan

  • grep -r seperately packages/fxa-auth-server/test/remote/oauth_api.in.spec.ts returns no matches

Because:

* The integration test name misspelled "separately" as "seperately".

This commit:

* Corrects the test description string with no other changes.
@sreecharan-desu

Copy link
Copy Markdown
Contributor Author

Split from #19940 — typo fix only, no other changes.

@dschom dschom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@dschom
dschom merged commit 3106338 into mozilla:main Jul 10, 2026
7 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.

2 participants