Skip to content

test(bigtable): fix JSpecify CI issues on Java 8#13809

Merged
lqiu96 merged 1 commit into
mainfrom
fix-jspecify-ci-bigtable-mockito
Jul 17, 2026
Merged

test(bigtable): fix JSpecify CI issues on Java 8#13809
lqiu96 merged 1 commit into
mainfrom
fix-jspecify-ci-bigtable-mockito

Conversation

@lqiu96

@lqiu96 lqiu96 commented Jul 17, 2026

Copy link
Copy Markdown
Member

Disable annotations in Mockito mocks for Gax callables and providers to avoid ArrayStoreException on Java 8 when reflecting on JSpecify annotations.

Disable annotations in Mockito mocks for Gax callables and providers
to avoid ArrayStoreException on Java 8 when reflecting on JSpecify
annotations.

TAG=agy
CONV=de3ef09d-a26c-4190-b578-4c993f807bdc

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

Copy link
Copy Markdown
Contributor

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 refactors several test files to replace the use of Mockito's @Mock annotations with manual mock initialization using Mockito.mock(..., Mockito.withSettings().withoutAnnotations()). There are no review comments to address, and I have no additional feedback to provide.

@lqiu96
lqiu96 requested a review from mutianf July 17, 2026 19:41
@lqiu96
lqiu96 marked this pull request as ready for review July 17, 2026 19:42
@lqiu96
lqiu96 requested review from a team as code owners July 17, 2026 19:42
@lqiu96
lqiu96 merged commit f1bd09e into main Jul 17, 2026
211 of 212 checks passed
@lqiu96
lqiu96 deleted the fix-jspecify-ci-bigtable-mockito branch July 17, 2026 19:58
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