EnableKubernetesMockClient && junit5 - KubernetesMockServer stopped to work correctly from the version 7.0.x

[svagner]

Describe the bug

Example of code:

import io.fabric8.kubernetes.api.model.PodList;
import io.fabric8.kubernetes.api.model.PodListBuilder;
import io.fabric8.kubernetes.client.KubernetesClient;
import io.fabric8.kubernetes.client.server.mock.EnableKubernetesMockClient;
import io.fabric8.kubernetes.client.server.mock.KubernetesMockServer;
import org.junit.jupiter.api.Test;
import static org.junit.jupiter.api.Assertions.assertNotNull;

@EnableKubernetesMockClient
class KubernetesClientTest {
    KubernetesClient client;
    KubernetesMockServer server;

    @Test
    void testPodList() {
        // Mock the server response
        server.expect().get().withPath("/api/v1/namespaces/default/pods")
              .andReturn(200, new PodListBuilder().build())
              .once();

        // Use the client to list pods
        PodList podList = client.pods().inNamespace("default").list();
        assertNotNull(podList);
    }
}

Result:

io.fabric8.kubernetes.client.KubernetesClientException: Connection was closed
	at [email protected]/io.fabric8.kubernetes.client.dsl.internal.OperationSupport.waitForResult(OperationSupport.java:509)
	at [email protected]/io.fabric8.kubernetes.client.dsl.internal.BaseOperation.list(BaseOperation.java:451)
	at [email protected]/io.fabric8.kubernetes.client.dsl.internal.BaseOperation.list(BaseOperation.java:419)
	at [email protected]/io.fabric8.kubernetes.client.dsl.internal.BaseOperation.list(BaseOperation.java:98)
	at my.package@SNAPSHOT/my.package.KubernetesClientTest.testPodList(KubernetesClientTest.java:30)
	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1541)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1541)
Caused by: io.vertx.core.http.HttpClosedException: Connection was closed

Fabric8 Kubernetes Client version

7.0.1

Steps to reproduce

• Run the test from the bug description

Expected behavior

• in version 6.13.4 the same code works without any issues and return results.

Runtime

minikube

Kubernetes API Server version

1.25.3@latest

Environment

macOS

Fabric8 Kubernetes Client Logs

System.setProperty("kubernetes.client.logging.level", "FINE"); 

doesn't provide any details anymore

Additional context

No response

[manusa]

Could you try with 7.1.0 which was released yesterday.
It might be related to a bug with the Vertx client.

(also note that you mention in the issue that you're using 7.0.1, but your stacktrace reports 7.0.0, 7.0.1 should correct that bug too).

[svagner]

7.1.0 has the same problem:(

[manusa]

7.1.0 has the same problem:(

Could you share the new stack trace?

[svagner]

o.fabric8.kubernetes.client.KubernetesClientException: Connection was closed
        at [email protected]/io.fabric8.kubernetes.client.dsl.internal.OperationSupport.waitForResult(OperationSupport.java:509)
        at [email protected]/io.fabric8.kubernetes.client.dsl.internal.BaseOperation.list(BaseOperation.java:451)
        at [email protected]/io.fabric8.kubernetes.client.dsl.internal.BaseOperation.list(BaseOperation.java:419)
        at [email protected]/io.fabric8.kubernetes.client.dsl.internal.BaseOperation.list(BaseOperation.java:98)
        at my.package@SNAPSHOT/my.package.KubernetesClientTest.testPodList(KubernetesClientTest.java:30)
        at java.base/java.lang.reflect.Method.invoke(Method.java:566)
        at java.base/java.util.ArrayList.forEach(ArrayList.java:1541)
        at java.base/java.util.ArrayList.forEach(ArrayList.java:1541)
Caused by: io.vertx.core.http.HttpClosedException: Connection was closed

[manusa]

I can't really tell what's wrong here, there's nothing different from any of our standard test which do work.

For example:

kubernetes-client/kubernetes-tests/src/test/java/io/fabric8/kubernetes/client/mock/PodTest.java

Lines 95 to 131 in 5a216fc

	@Test
	void testList() {
	server.expect().withPath("/api/v1/namespaces/test/pods").andReturn(200, new PodListBuilder().build()).once();
	server.expect()
	.withPath("/api/v1/namespaces/ns1/pods")
	.andReturn(200, new PodListBuilder()
	.addNewItem()
	.and()
	.addNewItem()
	.and()
	.build())
	.once();
	server.expect()
	.withPath("/api/v1/pods")
	.andReturn(200, new PodListBuilder()
	.addNewItem()
	.and()
	.addNewItem()
	.and()
	.addNewItem()
	.and()
	.build())
	.once();
	PodList podList = client.pods().list();
	assertNotNull(podList);
	assertEquals(0, podList.getItems().size());
	podList = client.pods().inNamespace("ns1").list();
	assertNotNull(podList);
	assertEquals(2, podList.getItems().size());
	podList = client.pods().inAnyNamespace().list();
	assertNotNull(podList);
	assertEquals(3, podList.getItems().size());
	}

[manusa]

In your stack trace there is a forEach call, which makes me believe that your test is different to what you're showing in your description.
Please, share your original test. It's possible that you're including a call to client.close in your test loop.

[svagner]

I assume loop is because of Jupiter annotation...