test: Disable copying annotation on mocks for Java 8 CIs (#13702)

In JSpecify 1.0.0, the `@NullMarked` annotation includes
`ElementType.MODULE` in its `@Target` metadata. Because
`ElementType.MODULE` was introduced in Java 9, reflecting on
`@NullMarked` classes under Java 8 (JDK 1.8) throws
`EnumConstantNotPresentExceptionProxy` wrapped in an
`ArrayStoreException`.

When unit tests run on Java 8, Mockito's default mock generation (via
ByteBuddy) attempts to copy all runtime class annotations from target
classes onto mock subclasses. This triggers the reflection crash when
mocking generated stub and settings classes that are annotated with
`@NullMarked`.

To bypass this Java 8 limitation:
- Replaced default Mockito mock instantiation with `Mockito.mock(Class,
Mockito.withSettings().withoutAnnotations())` across the failing unit
test suites.
- Disabling annotation copying prevents Mockito/ByteBuddy from invoking
`Class.getAnnotations()` on the target class during proxy creation,
allowing tests to run and mock stubs successfully on Java 8 pipelines.

Modified Mockito mock setups in:
- `StreamingSubscriberConnectionTest`
- `SpannerImplTest`
- `SpannerCloudMonitoringExporterTest`
- `BigtableInstanceAdminClientV2Test`
- `BigtableTableAdminClientV2Test`
- `BigtableCloudMonitoringExporterTest`
- `GrpcLongRunningClientTest`

Author: nnicolee

java-bigtable/google-cloud-bigtable/src/test/java/com/google/cloud/bigtable/admin/v2/BigtableTableAdminClientV2Test.java

@@ -31,28 +31,22 @@
 import java.util.concurrent.Executor;
 import java.util.concurrent.atomic.AtomicBoolean;
 import org.junit.Before;
-import org.junit.Rule;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.junit.runners.JUnit4;
-import org.mockito.Mock;
 import org.mockito.Mockito;
-import org.mockito.junit.MockitoJUnit;
-import org.mockito.junit.MockitoRule;
 import org.mockito.stubbing.Answer;
 
 @RunWith(JUnit4.class)
 public class BigtableTableAdminClientV2Test {
-  @Rule public final MockitoRule mockitoRule = MockitoJUnit.rule();
 
   private static final String TABLE_NAME =
       "projects/my-project/instances/my-instance/tables/my-table";
 
   private GrpcBigtableTableAdminStub mockStub;
 
-  @Mock private AwaitConsistencyCallableV2 mockAwaitConsistencyCallable;
+  private AwaitConsistencyCallableV2 mockAwaitConsistencyCallable;
 
-  @Mock
   private OperationCallable<Void, Empty, OptimizeRestoredTableMetadata>
       mockOptimizeRestoredTableCallable;
 
@@ -62,6 +56,10 @@ public class BigtableTableAdminClientV2Test {
   public void setUp() {
     mockStub =
         Mockito.mock(GrpcBigtableTableAdminStub.class, Mockito.withSettings().withoutAnnotations());
+    mockAwaitConsistencyCallable =
+        Mockito.mock(AwaitConsistencyCallableV2.class, Mockito.withSettings().withoutAnnotations());
+    mockOptimizeRestoredTableCallable =
+        Mockito.mock(OperationCallable.class, Mockito.withSettings().withoutAnnotations());
     client =
         new BigtableTableAdminClientV2(
             mockStub, null, false, mockAwaitConsistencyCallable, mockOptimizeRestoredTableCallable);

java-pubsub/google-cloud-pubsub/src/test/java/com/google/cloud/pubsub/v1/StreamingSubscriberConnectionTest.java

@@ -103,7 +103,10 @@ public void setUp() {
     systemExecutor = new FakeScheduledExecutorService();
     executor = new FakeScheduledExecutorService();
     clock = systemExecutor.getClock();
-    mockSubscriberStub = mock(SubscriberStub.class, RETURNS_DEEP_STUBS);
+    mockSubscriberStub =
+        mock(
+            SubscriberStub.class,
+            withSettings().withoutAnnotations().defaultAnswer(RETURNS_DEEP_STUBS));
   }
 
   @After
@@ -128,7 +131,8 @@ public void testRunShutdown_TimeoutMet() throws Exception {
     SubscriberShutdownSettings shutdownSettings =
         SubscriberShutdownSettings.newBuilder().setTimeout(Duration.ofSeconds(10)).build();
     StreamingSubscriberConnection.Builder builder =
-        StreamingSubscriberConnection.newBuilder(mock(MessageReceiverWithAckResponse.class));
+        StreamingSubscriberConnection.newBuilder(
+            mock(MessageReceiverWithAckResponse.class, withSettings().withoutAnnotations()));
     builder.setSubscriberShutdownSettings(shutdownSettings);
     StreamingSubscriberConnection streamingSubscriberConnection =
         getStreamingSubscriberConnectionFromBuilder(builder);
@@ -149,7 +153,8 @@ public void testRunShutdown_TimeoutExceeded() throws Exception {
     SubscriberShutdownSettings shutdownSettings =
         SubscriberShutdownSettings.newBuilder().setTimeout(Duration.ofSeconds(2)).build();
     StreamingSubscriberConnection.Builder builder =
-        StreamingSubscriberConnection.newBuilder(mock(MessageReceiverWithAckResponse.class));
+        StreamingSubscriberConnection.newBuilder(
+            mock(MessageReceiverWithAckResponse.class, withSettings().withoutAnnotations()));
     StreamingSubscriberConnection streamingSubscriberConnection =
         getStreamingSubscriberConnectionFromBuilder(builder, shutdownSettings);
     streamingSubscriberConnection.setExactlyOnceDeliveryEnabled(true);
@@ -199,11 +204,13 @@ public void testAckDuringNackImmediatelyShutdown() throws Exception {
             .setMode(SubscriberShutdownSettings.ShutdownMode.NACK_IMMEDIATELY)
             .build();
 
-    MessageDispatcher mockMessageDispatcher = mock(MessageDispatcher.class);
+    MessageDispatcher mockMessageDispatcher =
+        mock(MessageDispatcher.class, withSettings().withoutAnnotations());
     when(mockMessageDispatcher.getNackImmediatelyShutdownInProgress()).thenReturn(true);
 
     StreamingSubscriberConnection.Builder builder =
-        StreamingSubscriberConnection.newBuilder(mock(MessageReceiverWithAckResponse.class));
+        StreamingSubscriberConnection.newBuilder(
+            mock(MessageReceiverWithAckResponse.class, withSettings().withoutAnnotations()));
     StreamingSubscriberConnection streamingSubscriberConnection =
         getStreamingSubscriberConnectionFromBuilder(builder, shutdownSettings);
 
@@ -687,8 +694,9 @@ public void testMaxPerRequestChanges() {
   @Test
   public void testClientPinger_pingSent() {
     BidiStreamingCallable<StreamingPullRequest, StreamingPullResponse> mockStreamingCallable =
-        mock(BidiStreamingCallable.class);
-    ClientStream<StreamingPullRequest> mockClientStream = mock(ClientStream.class);
+        mock(BidiStreamingCallable.class, withSettings().withoutAnnotations());
+    ClientStream<StreamingPullRequest> mockClientStream =
+        mock(ClientStream.class, withSettings().withoutAnnotations());
     when(mockSubscriberStub.streamingPullCallable()).thenReturn(mockStreamingCallable);
     when(mockStreamingCallable.splitCall(any(ResponseObserver.class), any()))
         .thenReturn(mockClientStream);
@@ -730,8 +738,9 @@ public void testClientPinger_pingSent() {
   @Test
   public void testClientPinger_pingsNotSentWhenDisabled() {
     BidiStreamingCallable<StreamingPullRequest, StreamingPullResponse> mockStreamingCallable =
-        mock(BidiStreamingCallable.class);
-    ClientStream<StreamingPullRequest> mockClientStream = mock(ClientStream.class);
+        mock(BidiStreamingCallable.class, withSettings().withoutAnnotations());
+    ClientStream<StreamingPullRequest> mockClientStream =
+        mock(ClientStream.class, withSettings().withoutAnnotations());
     when(mockSubscriberStub.streamingPullCallable()).thenReturn(mockStreamingCallable);
     when(mockStreamingCallable.splitCall(any(ResponseObserver.class), any()))
         .thenReturn(mockClientStream);
@@ -755,8 +764,9 @@ public void testClientPinger_pingsNotSentWhenDisabled() {
   @Test
   public void testServerMonitor_timesOut() {
     BidiStreamingCallable<StreamingPullRequest, StreamingPullResponse> mockStreamingCallable =
-        mock(BidiStreamingCallable.class);
-    ClientStream<StreamingPullRequest> mockClientStream = mock(ClientStream.class);
+        mock(BidiStreamingCallable.class, withSettings().withoutAnnotations());
+    ClientStream<StreamingPullRequest> mockClientStream =
+        mock(ClientStream.class, withSettings().withoutAnnotations());
     ArgumentCaptor<ResponseObserver<StreamingPullResponse>> observerCaptor =
         ArgumentCaptor.forClass(ResponseObserver.class);
     when(mockSubscriberStub.streamingPullCallable()).thenReturn(mockStreamingCallable);
@@ -786,7 +796,8 @@ public void testServerMonitor_timesOut() {
     streamingSubscriberConnection.awaitRunning();
 
     ResponseObserver<StreamingPullResponse> observer = observerCaptor.getValue();
-    StreamController mockController = mock(StreamController.class);
+    StreamController mockController =
+        mock(StreamController.class, withSettings().withoutAnnotations());
     observer.onStart(mockController);
 
     systemExecutor.advanceTime(CLIENT_PING_INTERVAL);
@@ -803,8 +814,9 @@ public void testServerMonitor_timesOut() {
   @Test
   public void testServerMonitor_doesNotTimeOutIfResponseReceived() {
     BidiStreamingCallable<StreamingPullRequest, StreamingPullResponse> mockStreamingCallable =
-        mock(BidiStreamingCallable.class);
-    ClientStream<StreamingPullRequest> mockClientStream = mock(ClientStream.class);
+        mock(BidiStreamingCallable.class, withSettings().withoutAnnotations());
+    ClientStream<StreamingPullRequest> mockClientStream =
+        mock(ClientStream.class, withSettings().withoutAnnotations());
     ArgumentCaptor<ResponseObserver<StreamingPullResponse>> observerCaptor =
         ArgumentCaptor.forClass(ResponseObserver.class);
     when(mockSubscriberStub.streamingPullCallable()).thenReturn(mockStreamingCallable);
@@ -818,7 +830,8 @@ public void testServerMonitor_doesNotTimeOutIfResponseReceived() {
     streamingSubscriberConnection.awaitRunning();
 
     ResponseObserver<StreamingPullResponse> observer = observerCaptor.getValue();
-    StreamController mockController = mock(StreamController.class);
+    StreamController mockController =
+        mock(StreamController.class, withSettings().withoutAnnotations());
     observer.onStart(mockController);
 
     // t=30s: ping sent.
@@ -837,7 +850,8 @@ private StreamingSubscriberConnection getStreamingSubscriberConnection(
       boolean exactlyOnceDeliveryEnabled) {
     StreamingSubscriberConnection streamingSubscriberConnection =
         getStreamingSubscriberConnectionFromBuilder(
-            StreamingSubscriberConnection.newBuilder(mock(MessageReceiverWithAckResponse.class)));
+            StreamingSubscriberConnection.newBuilder(
+                mock(MessageReceiverWithAckResponse.class, withSettings().withoutAnnotations())));
 
     // This would normally be set from the streaming pull response
     streamingSubscriberConnection.setExactlyOnceDeliveryEnabled(exactlyOnceDeliveryEnabled);
@@ -848,7 +862,8 @@ private StreamingSubscriberConnection getStreamingSubscriberConnection(
   private StreamingSubscriberConnection getKeepaliveStreamingSubscriberConnection() {
     StreamingSubscriberConnection streamingSubscriberConnection =
         getStreamingSubscriberConnectionFromBuilder(
-            StreamingSubscriberConnection.newBuilder(mock(MessageReceiverWithAckResponse.class))
+            StreamingSubscriberConnection.newBuilder(
+                    mock(MessageReceiverWithAckResponse.class, withSettings().withoutAnnotations()))
                 .setProtocolVersion(KEEP_ALIVE_SUPPORT_VERSION));
 
     return streamingSubscriberConnection;
@@ -860,11 +875,12 @@ private StreamingSubscriberConnection getStreamingSubscriberConnectionFromBuilde
         .setSubscription(MOCK_SUBSCRIPTION_NAME)
         .setAckExpirationPadding(ACK_EXPIRATION_PADDING_DEFAULT_DURATION)
         .setMaxAckExtensionPeriod(MAX_ACK_EXTENSION_PERIOD)
-        .setAckLatencyDistribution(mock(Distribution.class))
+        .setAckLatencyDistribution(mock(Distribution.class, withSettings().withoutAnnotations()))
         .setSubscriberStub(mockSubscriberStub)
         .setChannelAffinity(0)
-        .setFlowControlSettings(mock(FlowControlSettings.class))
-        .setFlowController(mock(FlowController.class))
+        .setFlowControlSettings(
+            mock(FlowControlSettings.class, withSettings().withoutAnnotations()))
+        .setFlowController(mock(FlowController.class, withSettings().withoutAnnotations()))
         .setExecutor(executor)
         .setSystemExecutor(systemExecutor)
         .setClock(clock)
@@ -881,11 +897,12 @@ private StreamingSubscriberConnection getStreamingSubscriberConnectionFromBuilde
     return builder
         .setSubscription(MOCK_SUBSCRIPTION_NAME)
         .setAckExpirationPadding(ACK_EXPIRATION_PADDING_DEFAULT_DURATION)
-        .setAckLatencyDistribution(mock(Distribution.class))
+        .setAckLatencyDistribution(mock(Distribution.class, withSettings().withoutAnnotations()))
         .setSubscriberStub(mockSubscriberStub)
         .setChannelAffinity(0)
-        .setFlowControlSettings(mock(FlowControlSettings.class))
-        .setFlowController(mock(FlowController.class))
+        .setFlowControlSettings(
+            mock(FlowControlSettings.class, withSettings().withoutAnnotations()))
+        .setFlowController(mock(FlowController.class, withSettings().withoutAnnotations()))
         .setExecutor(executor)
         .setSystemExecutor(systemExecutor)
         .setClock(clock)

java-spanner/google-cloud-spanner/src/test/java/com/google/cloud/spanner/SpannerCloudMonitoringExporterTest.java

@@ -72,13 +72,9 @@
 import java.util.stream.Collectors;
 import org.junit.After;
 import org.junit.Before;
-import org.junit.Rule;
 import org.junit.Test;
 import org.mockito.ArgumentCaptor;
-import org.mockito.Mock;
 import org.mockito.Mockito;
-import org.mockito.junit.MockitoJUnit;
-import org.mockito.junit.MockitoRule;
 
 public class SpannerCloudMonitoringExporterTest {
 
@@ -91,9 +87,7 @@ public class SpannerCloudMonitoringExporterTest {
   private static final String clientHash = "spanner-test";
   private static final String instanceConfigId = "fake-instance-config-id";
 
-  @Rule public final MockitoRule mockitoRule = MockitoJUnit.rule();
-
-  @Mock private MetricServiceStub mockMetricServiceStub;
+  private MetricServiceStub mockMetricServiceStub;
   private MetricServiceClient fakeMetricServiceClient;
   private SpannerCloudMonitoringExporter exporter;
 
@@ -107,6 +101,8 @@ public class SpannerCloudMonitoringExporterTest {
 
   @Before
   public void setUp() {
+    mockMetricServiceStub =
+        Mockito.mock(MetricServiceStub.class, Mockito.withSettings().withoutAnnotations());
     fakeMetricServiceClient = new FakeMetricServiceClient(mockMetricServiceStub);
     exporter = new SpannerCloudMonitoringExporter(fakeMetricServiceClient);
 

java-spanner/google-cloud-spanner/src/test/java/com/google/cloud/spanner/SpannerImplTest.java

@@ -62,10 +62,10 @@
 public class SpannerImplTest {
   @Mock private SpannerRpc rpc;
   @Mock private SpannerOptions spannerOptions;
-  @Mock private DatabaseAdminStubSettings databaseAdminStubSettings;
-  @Mock private DatabaseAdminStub databaseAdminStub;
-  @Mock private InstanceAdminStubSettings instanceAdminStubSettings;
-  @Mock private InstanceAdminStub instanceAdminStub;
+  private DatabaseAdminStubSettings databaseAdminStubSettings;
+  private DatabaseAdminStub databaseAdminStub;
+  private InstanceAdminStubSettings instanceAdminStubSettings;
+  private InstanceAdminStub instanceAdminStub;
   private SpannerImpl impl;
 
   @Captor ArgumentCaptor<Map<SpannerRpc.Option, Object>> options;
@@ -78,6 +78,15 @@ public static void setupOpenTelemetry() {
 
   @Before
   public void setUp() {
+    databaseAdminStubSettings =
+        Mockito.mock(DatabaseAdminStubSettings.class, Mockito.withSettings().withoutAnnotations());
+    databaseAdminStub =
+        Mockito.mock(DatabaseAdminStub.class, Mockito.withSettings().withoutAnnotations());
+    instanceAdminStubSettings =
+        Mockito.mock(InstanceAdminStubSettings.class, Mockito.withSettings().withoutAnnotations());
+    instanceAdminStub =
+        Mockito.mock(InstanceAdminStub.class, Mockito.withSettings().withoutAnnotations());
+
     MockitoAnnotations.initMocks(this);
     when(spannerOptions.getNumChannels()).thenReturn(4);
     when(spannerOptions.getDatabaseRole()).thenReturn("role");

sdk-platform-java/gax-java/gax-grpc/src/test/java/com/google/api/gax/grpc/GrpcLongRunningClientTest.java

@@ -50,7 +50,8 @@ class GrpcLongRunningClientTest {
 
   @Test
   void get() {
-    OperationsStub operationsStub = mock(OperationsStub.class);
+    OperationsStub operationsStub =
+        mock(OperationsStub.class, org.mockito.Mockito.withSettings().withoutAnnotations());
     when(operationsStub.getOperationCallable())
         .thenReturn(
             new UnaryCallable<GetOperationRequest, Operation>() {
@@ -73,7 +74,8 @@ public ApiFuture<Operation> futureCall(
 
   @Test
   void cancel() {
-    OperationsStub operationsStub = mock(OperationsStub.class);
+    OperationsStub operationsStub =
+        mock(OperationsStub.class, org.mockito.Mockito.withSettings().withoutAnnotations());
     when(operationsStub.cancelOperationCallable())
         .thenReturn(
             new UnaryCallable<CancelOperationRequest, Empty>() {
@@ -94,7 +96,8 @@ public ApiFuture<Empty> futureCall(
 
   @Test
   void delete() {
-    OperationsStub operationsStub = mock(OperationsStub.class);
+    OperationsStub operationsStub =
+        mock(OperationsStub.class, org.mockito.Mockito.withSettings().withoutAnnotations());
     when(operationsStub.deleteOperationCallable())
         .thenReturn(
             new UnaryCallable<DeleteOperationRequest, Empty>() {