chore(deps): bump Mockito to 5.21.0

Update Mockito version and adjust affected tests to use Mockito 5 compatible APIs:
- Replace InvocationMatcher with MatchableInvocation
- Replace getWanted() with getTarget()
- Replace anyObject() with any()/anyString()
- Remove spy() on mock objects

Signed-off-by: Oleksii Orel <oorel@redhat.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

Author: olexii4

core/che-core-api-core/src/test/java/org/eclipse/che/api/core/util/lineconsumer/ConcurrentCompositeLineConsumerTest.java

@@ -32,10 +32,10 @@
 import org.eclipse.che.commons.test.mockito.answer.WaitingAnswer;
 import org.mockito.Mock;
 import org.mockito.exceptions.base.MockitoException;
-import org.mockito.internal.invocation.InvocationMatcher;
 import org.mockito.internal.verification.VerificationModeFactory;
 import org.mockito.internal.verification.api.VerificationData;
 import org.mockito.invocation.Invocation;
+import org.mockito.invocation.MatchableInvocation;
 import org.mockito.testng.MockitoTestNGListener;
 import org.mockito.verification.VerificationMode;
 import org.testng.annotations.AfterMethod;
@@ -270,7 +270,7 @@ public Last() {}
 
     public void verify(VerificationData verificationData) {
       List<Invocation> invocations = verificationData.getAllInvocations();
-      InvocationMatcher invocationMatcher = verificationData.getWanted();
+      MatchableInvocation invocationMatcher = verificationData.getTarget();
 
       if (invocations == null || invocations.isEmpty()) {
         throw new MockitoException(

infrastructures/kubernetes/src/test/java/org/eclipse/che/workspace/infrastructure/kubernetes/docker/auth/UserSpecificDockerRegistryCredentialsProviderTest.java

@@ -11,7 +11,7 @@
  */
 package org.eclipse.che.workspace.infrastructure.kubernetes.docker.auth;
 
-import static org.mockito.ArgumentMatchers.anyObject;
+import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.Mockito.when;
 import static org.testng.Assert.assertEquals;
 import static org.testng.Assert.assertNotNull;
@@ -104,6 +104,6 @@ private void setCredentialsIntoPreferences(String base64encodedCredentials)
     preferences.put(DOCKER_REGISTRY_CREDENTIALS_KEY, base64encodedCredentials);
 
     EnvironmentContext.getCurrent().setSubject(new SubjectImpl("name", "id", "token1234", false));
-    when(preferenceManager.find(anyObject(), anyObject())).thenReturn(preferences);
+    when(preferenceManager.find(anyString(), anyString())).thenReturn(preferences);
   }
 }

infrastructures/kubernetes/src/test/java/org/eclipse/che/workspace/infrastructure/kubernetes/namespace/configurator/CredentialsSecretConfiguratorTest.java

@@ -15,7 +15,6 @@
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.doThrow;
 import static org.mockito.Mockito.never;
-import static org.mockito.Mockito.spy;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 import static org.testng.Assert.assertEquals;
@@ -84,10 +83,8 @@ public void setUp() throws InfrastructureException {
             new KubernetesMixedDispatcher(responses),
             true);
     kubernetesMockServer.init();
-    kubernetesClient = spy(kubernetesMockServer.createClient());
-
-    KubernetesClient client = spy(kubernetesClient);
-    when(cheServerKubernetesClientFactory.create()).thenReturn(client);
+    kubernetesClient = kubernetesMockServer.createClient();
+    when(cheServerKubernetesClientFactory.create()).thenReturn(kubernetesClient);
 
     namespaceResolutionContext =
         new NamespaceResolutionContext(TEST_WORKSPACE_ID, TEST_USER_ID, TEST_USERNAME);

multiuser/permission/che-multiuser-permission-system/src/test/java/org/eclipse/che/multiuser/permission/system/JvmServicePermissionsFilterTest.java

@@ -17,7 +17,7 @@
 import static org.everrest.assured.JettyHttpServer.ADMIN_USER_NAME;
 import static org.everrest.assured.JettyHttpServer.ADMIN_USER_PASSWORD;
 import static org.everrest.assured.JettyHttpServer.SECURE_PATH;
-import static org.mockito.ArgumentMatchers.anyObject;
+import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.doAnswer;
 import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.verify;
@@ -146,7 +146,7 @@ private static void permitSubject(String... allowedActions) throws ForbiddenExce
               return null;
             })
         .when(subject)
-        .checkPermission(anyObject(), anyObject(), anyObject());
+        .checkPermission(any(), any(), any());
   }
 
   private static Set<String> getDeclaredPublicMethods(Class<?> c) {

pom.xml

@@ -151,7 +151,7 @@
         <org.jgroups.version>4.1.9.Final</org.jgroups.version>
         <org.leadpony.justify.version>3.1.0</org.leadpony.justify.version>
         <org.mockito.mockito-testng.version>0.5.4</org.mockito.mockito-testng.version>
-        <org.mockito.version>3.11.2</org.mockito.version>
+        <org.mockito.version>5.21.0</org.mockito.version>
         <org.postgresql.version>42.7.9</org.postgresql.version>
         <org.reflections.version>0.9.9</org.reflections.version>
         <org.slf4j.version>2.0.17</org.slf4j.version>