From d4146a9ed8ef3d9c8fc64ab9a6163da979ef810c Mon Sep 17 00:00:00 2001 From: Jose Date: Thu, 1 Jun 2023 15:04:05 +0200 Subject: [PATCH] Fix failing tests after removing the HTTPS port After https://github.com/quarkusio/quarkus/pull/33696 being merged, the port https is no longer bound by default. The problem is that I merged https://github.com/quarkusio/quarkus/pull/33694 without rebasing and these two tests needed to be updated. --- .../kubernetes/KubernetesExposingManagementInterfaceTest.java | 3 +-- .../KubernetesWithHealthUsingManagementInterfaceTest.java | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/integration-tests/kubernetes/quarkus-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesExposingManagementInterfaceTest.java b/integration-tests/kubernetes/quarkus-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesExposingManagementInterfaceTest.java index 593a10524a9f0..55b2a7e9f3ecd 100644 --- a/integration-tests/kubernetes/quarkus-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesExposingManagementInterfaceTest.java +++ b/integration-tests/kubernetes/quarkus-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesExposingManagementInterfaceTest.java @@ -47,9 +47,8 @@ public void assertGeneratedResources() throws IOException { }); assertThat(service.getSpec()).satisfies(spec -> { - assertThat(spec.getPorts()).hasSize(3); + assertThat(spec.getPorts()).hasSize(2); assertThat(spec.getPorts()).satisfiesOnlyOnce(port -> assertThat(port.getName()).isEqualTo("http")); - assertThat(spec.getPorts()).satisfiesOnlyOnce(port -> assertThat(port.getName()).isEqualTo("https")); assertThat(spec.getPorts()).satisfiesOnlyOnce(port -> assertThat(port.getName()).isEqualTo("management")); }); } diff --git a/integration-tests/kubernetes/quarkus-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesWithHealthUsingManagementInterfaceTest.java b/integration-tests/kubernetes/quarkus-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesWithHealthUsingManagementInterfaceTest.java index 92bba8d176c47..0ed7912b667e5 100644 --- a/integration-tests/kubernetes/quarkus-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesWithHealthUsingManagementInterfaceTest.java +++ b/integration-tests/kubernetes/quarkus-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesWithHealthUsingManagementInterfaceTest.java @@ -102,9 +102,8 @@ public void assertGeneratedResources() throws IOException { }); assertThat(s.getSpec()).satisfies(spec -> { - assertThat(spec.getPorts()).hasSize(2); + assertThat(spec.getPorts()).hasSize(1); assertThat(spec.getPorts()).satisfiesOnlyOnce(port -> assertThat(port.getName()).isEqualTo("http")); - assertThat(spec.getPorts()).satisfiesOnlyOnce(port -> assertThat(port.getName()).isEqualTo("https")); }); }); }