Skip to content

test: add unit tests for OAuth2CredentialRefresher#462

Closed
nuthalapativarun wants to merge 1 commit into
google:mainfrom
nuthalapativarun:test/oauth2-credential-refresher
Closed

test: add unit tests for OAuth2CredentialRefresher#462
nuthalapativarun wants to merge 1 commit into
google:mainfrom
nuthalapativarun:test/oauth2-credential-refresher

Conversation

@nuthalapativarun

Copy link
Copy Markdown
Contributor

Problem

OAuth2CredentialRefresher in core/src/auth/oauth2/oauth2_credential_refresher.ts has no unit test coverage.

Solution

Add unit tests covering isRefreshNeeded and refresh methods, including happy paths, no-op conditions (missing oauth2/refreshToken fields), and error handling.

Testing

  • All new tests pass: npm test
  • No existing tests broken

Checklist

  • I have performed a self-review of my code
  • Tests added and passing
  • Manual E2E verified (N/A — test-only change)

@nuthalapativarun

Copy link
Copy Markdown
Contributor Author

Superseded by #464 which combines all three test files into a single PR.

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.

1 participant