Skip to content

chore: Fix GDCH tests referencing obsolete GDCH audience methods#12771

Merged
blakeli0 merged 5 commits intomainfrom
fix-gdch-issue
Apr 13, 2026
Merged

chore: Fix GDCH tests referencing obsolete GDCH audience methods#12771
blakeli0 merged 5 commits intomainfrom
fix-gdch-issue

Conversation

@lqiu96
Copy link
Copy Markdown
Member

@lqiu96 lqiu96 commented Apr 13, 2026

No description provided.

@lqiu96 lqiu96 requested a review from diegomarquezp April 13, 2026 17:48
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the GdchCredentials API usage to pass audience values as strings instead of URI objects, with corresponding updates to the test suite's mocks and verifications. A review comment identifies that the gdchAudienceUri variable in ClientContext.java is now unused and suggests re-evaluating the necessity of the URI validation block if the API change was intended to support non-URI audience strings.

@lqiu96 lqiu96 marked this pull request as ready for review April 13, 2026 19:49
@lqiu96 lqiu96 requested a review from a team as a code owner April 13, 2026 19:49
@blakeli0 blakeli0 merged commit facb2f6 into main Apr 13, 2026
106 of 122 checks passed
@blakeli0 blakeli0 deleted the fix-gdch-issue branch April 13, 2026 20:30
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.

3 participants