test: update tests to support mockito 5.x

Author: olavloite

pom.xml

@@ -252,8 +252,8 @@
     </dependency>
     <dependency>
       <groupId>org.mockito</groupId>
-      <artifactId>mockito-core</artifactId>
-      <version>5.0.0</version>
+      <artifactId>mockito-subclass</artifactId>
+      <version>5.14.2</version>
       <scope>test</scope>
     </dependency>
   </dependencies>

src/test/java/com/google/cloud/spanner/pgadapter/ConnectionHandlerTest.java

@@ -908,7 +908,7 @@ public void testListDatabasesOrInstances() {
     Spanner spanner = mock(Spanner.class);
     DatabaseAdminClient adminClient = mock(DatabaseAdminClient.class);
     when(spanner.getDatabaseAdminClient()).thenReturn(adminClient);
-    when(adminClient.listDatabases(eq("my-instance"), any())).thenReturn(Pages.empty());
+    when(adminClient.listDatabases("my-instance")).thenReturn(Pages.empty());
     StatusRuntimeException exception =
         newStatusResourceNotFoundException(
             "test-database",

src/test/java/com/google/cloud/spanner/pgadapter/JdbcMockServerTest.java

@@ -4552,7 +4552,8 @@ public void testSetTimeZoneEST() throws SQLException {
     try (Connection connection = DriverManager.getConnection(createUrl())) {
       // Java considers 'EST' to always be '-05:00'. That is; it is never DST.
       connection.createStatement().execute("set time zone 'EST'");
-      verifySettingValue(connection, "timezone", "-05:00");
+      verifySettingValue(
+          connection, "timezone", java.time.ZoneId.SHORT_IDS.getOrDefault("EST", "-05:00"));
       // 'EST5EDT' is the ID for the timezone that will change with DST.
       connection.createStatement().execute("set time zone 'EST5EDT'");
       verifySettingValue(connection, "timezone", "EST5EDT");

src/test/java/com/google/cloud/spanner/pgadapter/utils/MutationWriterTest.java

@@ -170,7 +170,7 @@ public void testWriteMutations() throws Exception {
                 .set("name")
                 .to("Two")
                 .build());
-    verify(databaseClient).writeWithOptions(eq(expectedMutations), any());
+    verify(databaseClient).writeWithOptions(eq(expectedMutations), any(com.google.cloud.spanner.Options.TransactionOption[].class));
   }
 
   @Test
@@ -270,7 +270,7 @@ public void testWriteMutations_NonAtomic_SucceedsForLargeBatch() throws Exceptio
       StatementResult updateCount = mutationWriter.call();
 
       assertEquals(5L, updateCount.getUpdateCount().longValue());
-      verify(databaseClient, times(3)).writeWithOptions(anyIterable(), any());
+      verify(databaseClient, times(3)).writeWithOptions(anyIterable(), any(com.google.cloud.spanner.Options.TransactionOption[].class));
     } finally {
       System.getProperties().remove("copy_in_mutation_limit");
     }
@@ -372,7 +372,7 @@ public void testWriteMutations_NonAtomic_SucceedsForLargeCommit() throws Excepti
       // 4. The second batch contains 28 bytes. (3 - 'Three')
       // 5. The third batch contains 24 bytes. (4 - 'Four')
       // 6. the fourth batch contains 24 bytes. (5 - 'Five')
-      verify(databaseClient, times(4)).writeWithOptions(anyIterable(), any());
+      verify(databaseClient, times(4)).writeWithOptions(anyIterable(), any(com.google.cloud.spanner.Options.TransactionOption[].class));
     } finally {
       System.getProperties().remove("copy_in_commit_limit");
     }
@@ -441,7 +441,7 @@ public void testWritePartials() throws Exception {
                 .set("name")
                 .to("Five")
                 .build());
-    verify(databaseClient).writeWithOptions(eq(expectedMutations), any());
+    verify(databaseClient).writeWithOptions(eq(expectedMutations), any(com.google.cloud.spanner.Options.TransactionOption[].class));
 
     executor.shutdown();
   }