Update mockito-core to 4.3.1

Author: injectives

driver/src/test/java/org/neo4j/driver/internal/InternalMapAccessorWithDefaultValueTest.java

@@ -26,6 +26,9 @@
 import java.util.List;
 import java.util.Map;
 
+import org.neo4j.driver.Record;
+import org.neo4j.driver.Value;
+import org.neo4j.driver.Values;
 import org.neo4j.driver.internal.value.BooleanValue;
 import org.neo4j.driver.internal.value.FloatValue;
 import org.neo4j.driver.internal.value.IntegerValue;
@@ -36,9 +39,6 @@
 import org.neo4j.driver.internal.value.PathValue;
 import org.neo4j.driver.internal.value.RelationshipValue;
 import org.neo4j.driver.internal.value.StringValue;
-import org.neo4j.driver.Record;
-import org.neo4j.driver.Value;
-import org.neo4j.driver.Values;
 import org.neo4j.driver.types.Entity;
 import org.neo4j.driver.types.Node;
 import org.neo4j.driver.types.Path;
@@ -115,8 +115,8 @@ void shouldGetObjectFromRecord()
         Record record = createRecord();
 
         // IntegerValue.asObject returns Long
-        assertThat( record.get( "IntegerValue", (Object) 3 ), equalTo( (Object) new Long( 11 ) ) );
-        assertThat( record.get( wrongKey, (Object) 3 ), equalTo( (Object) new Integer( 3 ) ) );
+        assertThat( record.get( "IntegerValue", (Object) 3 ), equalTo( 11L ) );
+        assertThat( record.get( wrongKey, (Object) 3 ), equalTo( 3 ) );
     }
 
     @Test
@@ -125,8 +125,8 @@ void shouldGetNumberFromRecord()
         Record record = createRecord();
 
         // IntegerValue.asNumber returns Long
-        assertThat( record.get( "IntegerValue", (Number) 3 ), equalTo( (Object) new Long( 11 ) ) );
-        assertThat( record.get( wrongKey, (Number) 3 ), equalTo( (Object) new Integer( 3 ) ) );
+        assertThat( record.get( "IntegerValue", (Number) 3 ), equalTo( (Object) 11L ) );
+        assertThat( record.get( wrongKey, (Number) 3 ), equalTo( (Object) 3 ) );
     }
 
     @Test

driver/src/test/java/org/neo4j/driver/internal/async/pool/ConnectionPoolImplTest.java

@@ -36,7 +36,7 @@
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.verifyNoInteractions;
 import static org.mockito.Mockito.when;
 import static org.neo4j.driver.internal.BoltServerAddress.LOCAL_DEFAULT;
 import static org.neo4j.driver.internal.async.connection.ChannelAttributes.authorizationStateListener;
@@ -57,7 +57,7 @@ void shouldDoNothingWhenRetainOnEmptyPool()
 
         pool.retainAll( singleton( LOCAL_DEFAULT ) );
 
-        verifyZeroInteractions( nettyChannelTracker );
+        verifyNoInteractions( nettyChannelTracker );
     }
 
     @Test

driver/src/test/java/org/neo4j/driver/internal/cluster/loadbalancing/LeastConnectedLoadBalancingStrategyTest.java

@@ -38,7 +38,7 @@
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
-import static org.mockito.MockitoAnnotations.initMocks;
+import static org.mockito.MockitoAnnotations.openMocks;
 import static org.neo4j.driver.internal.logging.DevNullLogging.DEV_NULL_LOGGING;
 import static org.neo4j.driver.internal.util.ClusterCompositionUtil.A;
 
@@ -51,7 +51,7 @@ class LeastConnectedLoadBalancingStrategyTest
     @BeforeEach
     void setUp()
     {
-        initMocks( this );
+        openMocks( this );
         strategy = new LeastConnectedLoadBalancingStrategy( connectionPool, DEV_NULL_LOGGING );
     }
 

driver/src/test/java/org/neo4j/driver/internal/handlers/RoutingResponseHandlerTest.java

@@ -23,20 +23,20 @@
 
 import java.util.concurrent.CompletionException;
 
-import org.neo4j.driver.internal.RoutingErrorHandler;
-import org.neo4j.driver.internal.spi.ResponseHandler;
 import org.neo4j.driver.AccessMode;
 import org.neo4j.driver.exceptions.ClientException;
 import org.neo4j.driver.exceptions.ServiceUnavailableException;
 import org.neo4j.driver.exceptions.SessionExpiredException;
 import org.neo4j.driver.exceptions.TransientException;
+import org.neo4j.driver.internal.RoutingErrorHandler;
+import org.neo4j.driver.internal.spi.ResponseHandler;
 
 import static org.hamcrest.Matchers.instanceOf;
 import static org.hamcrest.junit.MatcherAssert.assertThat;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.verifyNoInteractions;
 import static org.neo4j.driver.internal.BoltServerAddress.LOCAL_DEFAULT;
 
 class RoutingResponseHandlerTest
@@ -50,7 +50,7 @@ void shouldUnwrapCompletionException()
         Throwable handledError = handle( new CompletionException( error ), errorHandler );
 
         assertEquals( error, handledError );
-        verifyZeroInteractions( errorHandler );
+        verifyNoInteractions( errorHandler );
     }
 
     @Test
@@ -86,7 +86,7 @@ void shouldHandleTransientException()
         Throwable handledError = handle( error, errorHandler );
 
         assertEquals( error, handledError );
-        verifyZeroInteractions( errorHandler );
+        verifyNoInteractions( errorHandler );
     }
 
     @Test
@@ -122,7 +122,7 @@ void shouldHandleClientException()
         Throwable handledError = handle( error, errorHandler, AccessMode.READ );
 
         assertEquals( error, handledError );
-        verifyZeroInteractions( errorHandler );
+        verifyNoInteractions( errorHandler );
     }
 
     @Test
@@ -158,7 +158,7 @@ private void testWriteFailureWithReadAccessMode( String code )
 
         assertThat( handledError, instanceOf( ClientException.class ) );
         assertEquals( "Write queries cannot be performed in READ access mode.", handledError.getMessage() );
-        verifyZeroInteractions( errorHandler );
+        verifyNoInteractions( errorHandler );
     }
 
     private void testWriteFailureWithWriteAccessMode( String code )

driver/src/test/java/org/neo4j/driver/internal/retry/ExponentialBackoffRetryLogicTest.java

@@ -75,7 +75,7 @@
 import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.verifyNoInteractions;
 import static org.mockito.Mockito.when;
 import static org.neo4j.driver.internal.logging.DevNullLogging.DEV_NULL_LOGGING;
 import static org.neo4j.driver.internal.util.Futures.failedFuture;
@@ -1090,7 +1090,7 @@ void nothingIsLoggedOnFatalFailure()
                     throw new RuntimeException( "Fatal blocking" );
                 } ) );
         assertEquals( "Fatal blocking", error.getMessage() );
-        verifyZeroInteractions( logger );
+        verifyNoInteractions( logger );
     }
 
     @Test
@@ -1106,7 +1106,7 @@ void nothingIsLoggedOnFatalFailureAsync()
                 await( logic.retryAsync( () -> failedFuture( new RuntimeException( "Fatal async" ) ) ) ) );
 
         assertEquals( "Fatal async", error.getMessage() );
-        verifyZeroInteractions( logger );
+        verifyNoInteractions( logger );
     }
 
     @Test
@@ -1122,7 +1122,7 @@ void nothingIsLoggedOnFatalFailureRx()
         RuntimeException error = assertThrows( RuntimeException.class, () -> await( retryRx ) );
 
         assertEquals( "Fatal rx", error.getMessage() );
-        verifyZeroInteractions( logger );
+        verifyNoInteractions( logger );
     }
 
     @Test

pom.xml

@@ -42,7 +42,7 @@
     <rxjava.version>2.2.21</rxjava.version>
     <slf4j-api.version>1.7.35</slf4j-api.version>
     <hamcrest-junit.version>2.0.0.0</hamcrest-junit.version>
-    <mockito-core.version>2.28.2</mockito-core.version>
+    <mockito-core.version>4.3.1</mockito-core.version>
     <junit.version>5.8.2</junit.version>
     <jarchivelib.version>1.2.0</jarchivelib.version>
     <bouncycastle-jdk15on.version>1.70</bouncycastle-jdk15on.version>