build: fix auth library native tests and flaky test#12881
Merged
Conversation
The google-auth-library-java migration to the monorepo inadvertently enabled GraalVM native testing for the credentials module because the generic graalvm-single job type runs tests on all modules in the subdirectory. In the original repository, this job was carefully scoped to only test the oauth2_http module. This change adds <skipNativeTests>true</skipNativeTests> to the credentials module POM to skip native tests, resolving the failure when the plugin cannot find test configuration files. Fixes #12859
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
blakeli0
approved these changes
Apr 21, 2026
The google-auth-library-oauth2-http module fails to compile in the GraalVM job because the slf4j package is not found. In the original repository, this job was executed with the -Pslf4j2x profile activated, which adds the dependency. This change adds -Pslf4j2x to INTEGRATION_TEST_ARGS in the job configuration file. Fixes #12889
The refreshCredentialsIfRequired_asyncMultiThread test was failing because the mock clock was returning a fixed stale time, causing multiple threads to trigger redundant refreshes if they didn't overlap perfectly. This change modifies the mock clock to return a stale time on the first call (to trigger the async refresh) and a fresh time on subsequent calls (to skip redundant refreshes). This fix is isolated to only the ASYNC test case to avoid breaking BLOCKING tests.
chore: fix formatting in cab token generator tests to comply with project style rules.
The google-auth-library-appengine module was failing the native test phase because it lacked the necessary GraalVM test configuration file. This change skips native tests for this module, consistent with what we did for the credentials module.
chingor13
approved these changes
Apr 22, 2026
meltsufin
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
google-auth-library-javamigration to the monorepo inadvertently enabled GraalVM native testing for the credentials module because the genericgraalvm-singlejob type runs tests on all modules in the subdirectory. In the original repository, this job was scoped to only test theoauth2_httpmodule.This change adds
<skipNativeTests>true</skipNativeTests>to the credentials and appengine module POMs to skip native tests, resolving the failure when the plugin cannot find test configuration files. It also stabilizes a flaky concurrent test in cab-token-generator.Fixes #12859