Fix org.csanchez.jenkins.plugins.kubernetes.pipeline.KubernetesPipelineTest#runInPodWithRetention

Author: Vlatombe

src/test/java/org/csanchez/jenkins/plugins/kubernetes/KubernetesTestUtil.java

@@ -239,10 +239,7 @@ public static boolean deletePods(KubernetesClient client, Map<String, String> la
             PodList list = client.pods().withLabels(labels).list();
             if (!list.getItems().isEmpty()) {
                 LOGGER.log(WARNING, "Deleting leftover pods: {0}", print(list));
-                if (Boolean.TRUE.equals(client.pods().withLabels(labels).delete())) {
-                    return true;
-                }
-
+                return client.pods().withLabels(labels).delete().size() > 0;
             }
         }
         return false;

src/test/java/org/csanchez/jenkins/plugins/kubernetes/pipeline/KubernetesPipelineTest.java

@@ -42,8 +42,9 @@
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assume.*;
 
+import io.fabric8.kubernetes.api.model.StatusDetails;
+import io.fabric8.kubernetes.client.KubernetesClient;
 import java.util.Arrays;
-import java.util.Collections;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
@@ -460,7 +461,10 @@ public void runWithActiveDeadlineSeconds() throws Exception {
     @Test
     public void runInPodWithRetention() throws Exception {
         r.assertBuildStatusSuccess(r.waitForCompletion(b));
-        assertTrue(deletePods(cloud.connect(), getLabels(this, name), true));
+        System.out.println("Deleting leftover pods");
+        KubernetesClient client = cloud.connect();
+        Map<String, String> labels = getLabels(this, name);
+        assertTrue(client.pods().withLabels(labels).delete().size() > 0);
     }
 
     @Issue("JENKINS-49707")