Skip to content

test(auth): enforce strict once call counts on token caching tests#580

Merged
quartzmo merged 1 commit into
googleapis:mainfrom
westarle:fix-strict-call-counts
Jun 22, 2026
Merged

test(auth): enforce strict once call counts on token caching tests#580
quartzmo merged 1 commit into
googleapis:mainfrom
westarle:fix-strict-call-counts

Conversation

@westarle

Copy link
Copy Markdown
Contributor

The Ruby specs relied on loose have_been_requested expectations which only check if a request happened, not how many times. Other languages (like Python, Swift, and Java) verify caching logic by asserting that exactly one outbound HTTP request is made regardless of the number of credential calls.

The Ruby specs relied on loose have_been_requested expectations which only check if a request happened, not how many times. Other languages (like Python, Swift, and Java) verify caching logic by asserting that exactly one outbound HTTP request is made regardless of the number of credential calls.
@westarle westarle requested a review from a team as a code owner June 12, 2026 16:26

@quartzmo quartzmo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for catching these!

@quartzmo quartzmo closed this Jun 22, 2026
@quartzmo quartzmo reopened this Jun 22, 2026
@quartzmo

Copy link
Copy Markdown
Member

Trying to get the remaining GH Action presubmits to run.

@quartzmo quartzmo merged commit 78bd92f into googleapis:main Jun 22, 2026
35 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