kafka icon indicating copy to clipboard operation
kafka copied to clipboard

KAFKA-19308: Add unit tests for OAuthBearerRefreshingLogin

Open skrcode opened this issue 4 months ago • 4 comments

Jira : https://issues.apache.org/jira/browse/KAFKA-19308

This PR adds unit tests for OAuthBearerRefreshingLogin.

  • Increased coverage from 0% → 95% (line), 91% (branch), and 100% (method).
  • Covers edge cases for token refresh, expiry handling, and exception scenarios.
  • Verified locally with ./gradlew test.

This is a test-only change (no production code modifications).

skrcode avatar Sep 01 '25 19:09 skrcode

Reviewers: @kirktrue

skrcode avatar Sep 02 '25 04:09 skrcode

A label of 'needs-attention' was automatically added to this PR in order to raise the attention of the committers. Once this issue has been triaged, the triage label should be removed to prevent this automation from happening again.

github-actions[bot] avatar Sep 09 '25 03:09 github-actions[bot]

This PR is being marked as stale since it has not had any activity in 90 days. If you would like to keep this PR alive, please leave a comment asking for a review. If the PR has merge conflicts, update it with the latest from the base branch.

If you are having difficulty finding a reviewer, please reach out on the [mailing list](https://kafka.apache.org/contact).

If this PR is no longer valid or desired, please feel free to close it. If no activity occurs in the next 30 days, it will be automatically closed.

github-actions[bot] avatar Dec 10 '25 03:12 github-actions[bot]

Somehow I missed/lost track of this PR...

@skrcode: are you able to fix up the linting errors? You can run ./gradlew check -x test to perform the check locally.

Thanks!

kirktrue avatar Dec 11 '25 00:12 kirktrue