From 2f0d8a6a2765c28c710138becf9afac56adc3d3a Mon Sep 17 00:00:00 2001 From: Jose Carranza Date: Thu, 15 Jun 2023 20:19:49 +0200 Subject: [PATCH] parent ff5751750e6166419f78b5b4af295ee5868a80cf author Jose Carranza 1686853189 +0200 committer Jose Carranza 1688565488 +0200 adding some veretx http coverage related to HttpServer, HttpServerRequest, HttpServerResponse,HttpHeaders improve form logic implement ServerWebSocket logic, remove RestEasy annotation not needed here, and use @QuarkusScenario implement ServerWebSocket logic, remove RestEasy annotation not needed here, and use @QuarkusScenario keep 999999-SNAPSHOT version keep 999999-SNAPSHOT version keep pom as in upstream/main keep pom as in upstream/main use of @ApplicationScoped and onStart to deploy HttpVertxReactiveServer, websocket test reformat code style and add HttpClient test fix websocket test redoing tests as quarkus way - just keep vertx HttpClient test Enable OpenShiftQuickstartIT with snapshot Adjust GraphQLTelemetryIT to reflect latest updates Change Quarkus CLI utils to use default streams Remove todo with outdated Zulip conversation Bump maven-checkstyle-plugin from 3.2.1 to 3.3.0 Bumps [maven-checkstyle-plugin](https://github.com/apache/maven-checkstyle-plugin) from 3.2.1 to 3.3.0. - [Commits](https://github.com/apache/maven-checkstyle-plugin/compare/maven-checkstyle-plugin-3.2.1...maven-checkstyle-plugin-3.3.0) --- updated-dependencies: - dependency-name: org.apache.maven.plugins:maven-checkstyle-plugin dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] Remove magical constant in PostgresPoolTest Trailing comma is no longer present in Micrometer Prometheus See https://github.com/quarkusio/quarkus/issues/30343 for details Remove PostgreSQL image workaround Issue https://github.com/quarkus-qe/quarkus-test-framework/issues/641 was solved Bump quarkiverse.pact.version from 1.0.0.Final to 1.0.1.Final Bumps `quarkiverse.pact.version` from 1.0.0.Final to 1.0.1.Final. Updates `quarkus-pact-provider` from 1.0.0.Final to 1.0.1.Final Updates `quarkus-pact-consumer` from 1.0.0.Final to 1.0.1.Final --- updated-dependencies: - dependency-name: io.quarkiverse.pact:quarkus-pact-provider dependency-type: direct:development update-type: version-update:semver-patch - dependency-name: io.quarkiverse.pact:quarkus-pact-consumer dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] Update deprecated usage of RestSseElementType * Updating test app to use RestStreamElementType rather than the deprecated RestSseElementType. This also fixes failure of the Kafka tests running with the test app. Bump quarkus.qe.framework.version from 1.3.0.Beta16 to 1.3.0.Beta17 Bumps `quarkus.qe.framework.version` from 1.3.0.Beta16 to 1.3.0.Beta17. Updates `quarkus-test-parent` from 1.3.0.Beta16 to 1.3.0.Beta17 - [Release notes](https://github.com/quarkus-qe/quarkus-test-framework/releases) - [Changelog](https://github.com/quarkus-qe/quarkus-test-framework/blob/main/RELEASE.md) - [Commits](https://github.com/quarkus-qe/quarkus-test-framework/compare/1.3.0.Beta16...1.3.0.Beta17) Updates `quarkus-test-containers` from 1.3.0.Beta16 to 1.3.0.Beta17 - [Release notes](https://github.com/quarkus-qe/quarkus-test-framework/releases) - [Changelog](https://github.com/quarkus-qe/quarkus-test-framework/blob/main/RELEASE.md) - [Commits](https://github.com/quarkus-qe/quarkus-test-framework/compare/1.3.0.Beta16...1.3.0.Beta17) Updates `quarkus-test-openshift` from 1.3.0.Beta16 to 1.3.0.Beta17 - [Release notes](https://github.com/quarkus-qe/quarkus-test-framework/releases) - [Changelog](https://github.com/quarkus-qe/quarkus-test-framework/blob/main/RELEASE.md) - [Commits](https://github.com/quarkus-qe/quarkus-test-framework/compare/1.3.0.Beta16...1.3.0.Beta17) --- updated-dependencies: - dependency-name: io.quarkus.qe:quarkus-test-parent dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: io.quarkus.qe:quarkus-test-containers dependency-type: direct:development update-type: version-update:semver-patch - dependency-name: io.quarkus.qe:quarkus-test-openshift dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] Bump tj-actions/changed-files from 35.9.2 to 36.0.8 Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 35.9.2 to 36.0.8. - [Release notes](https://github.com/tj-actions/changed-files/releases) - [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md) - [Commits](https://github.com/tj-actions/changed-files/compare/v35.9.2...v36.0.8) --- updated-dependencies: - dependency-name: tj-actions/changed-files dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] Add option to debug Quarkus Test Suite Rename PostgreSQL image instances Extends https://github.com/quarkus-qe/quarkus-test-suite/pull/1240 Bump checkstyle from 10.11.0 to 10.12.0 Bumps [checkstyle](https://github.com/checkstyle/checkstyle) from 10.11.0 to 10.12.0. - [Release notes](https://github.com/checkstyle/checkstyle/releases) - [Commits](https://github.com/checkstyle/checkstyle/compare/checkstyle-10.11.0...checkstyle-10.12.0) --- updated-dependencies: - dependency-name: com.puppycrawl.tools:checkstyle dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] Change PostgresqlService instances to use default properties Related issue https://github.com/quarkus-qe/quarkus-test-framework/issues/455 was fixed Bump tj-actions/changed-files from 36.0.8 to 36.0.10 Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 36.0.8 to 36.0.10. - [Release notes](https://github.com/tj-actions/changed-files/releases) - [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md) - [Commits](https://github.com/tj-actions/changed-files/compare/v36.0.8...v36.0.10) --- updated-dependencies: - dependency-name: tj-actions/changed-files dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] Use only major version tj-actions/changed-files Fix failing HTTP DEV mode log checks Disable DomainSocketIT due to upstream bug Bump formatter-maven-plugin from 2.21.0 to 2.23.0 Bumps [formatter-maven-plugin](https://github.com/revelc/formatter-maven-plugin) from 2.21.0 to 2.23.0. - [Changelog](https://github.com/revelc/formatter-maven-plugin/blob/main/CHANGELOG.md) - [Commits](https://github.com/revelc/formatter-maven-plugin/compare/formatter-maven-plugin-2.21.0...formatter-maven-plugin-2.23.0) --- updated-dependencies: - dependency-name: net.revelc.code.formatter:formatter-maven-plugin dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] Verify OIDC service app dont generate enc. key Revert "Fix failing HTTP DEV mode log checks" This reverts commit 82bba48064fec875c436c781d2ceba8a1f62ca9e. Verify REST Client Reactive support truststore Replace OpenTracing with OTel in javaee-like-getting-started The goal is to have a module which combines OTel with other extensions, namely `smallrye-jwt` and `smallrye-openapi`. --- .../ts/http/advanced/DomainSocketIT.java | 2 ++ .../io/quarkus/ts/vertx/AbstractVertxIT.java | 23 +++++++++++++++++++ 2 files changed, 25 insertions(+) diff --git a/http/http-advanced/src/test/java/io/quarkus/ts/http/advanced/DomainSocketIT.java b/http/http-advanced/src/test/java/io/quarkus/ts/http/advanced/DomainSocketIT.java index 6bcf8d73c4..1343db35a7 100644 --- a/http/http-advanced/src/test/java/io/quarkus/ts/http/advanced/DomainSocketIT.java +++ b/http/http-advanced/src/test/java/io/quarkus/ts/http/advanced/DomainSocketIT.java @@ -7,6 +7,7 @@ import java.util.concurrent.TimeUnit; import java.util.concurrent.atomic.AtomicReference; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Tag; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.condition.DisabledIfSystemProperty; @@ -25,6 +26,7 @@ import io.vertx.ext.web.client.predicate.ResponsePredicate; import io.vertx.ext.web.codec.BodyCodec; +@Disabled("https://github.com/quarkusio/quarkus/issues/33679") @QuarkusScenario @Tag("https://github.com/quarkusio/quarkus/issues/24739") @DisabledOnOs(value = OS.WINDOWS, disabledReason = "Netty Native Transport not supported on Windows, see https://quarkus.io/guides/vertx-reference#native-transport") diff --git a/http/vertx/src/test/java/io/quarkus/ts/vertx/AbstractVertxIT.java b/http/vertx/src/test/java/io/quarkus/ts/vertx/AbstractVertxIT.java index 7bc567a2fa..01f241cd67 100644 --- a/http/vertx/src/test/java/io/quarkus/ts/vertx/AbstractVertxIT.java +++ b/http/vertx/src/test/java/io/quarkus/ts/vertx/AbstractVertxIT.java @@ -12,7 +12,13 @@ import java.util.Map; import java.util.Objects; +import io.vertx.core.Vertx; +import io.vertx.core.http.HttpClient; +import io.vertx.core.http.HttpClientRequest; +import io.vertx.core.http.HttpClientResponse; +import io.vertx.core.http.HttpMethod; import org.apache.http.HttpStatus; +import org.hamcrest.MatcherAssert; import org.junit.jupiter.api.Test; import io.restassured.response.Response; @@ -41,6 +47,23 @@ public void httpServerParsing() { requests().get("/hello?name=you").then().statusCode(HttpStatus.SC_OK).body("content", is("Hello, you!")); } + @Test + public void httpClient() { + HttpClient httpClient = Vertx.vertx().createHttpClient(); + httpClient.request(HttpMethod.GET, "/hello") + .onComplete(httpClientRequestAsyncResult -> { + if (httpClientRequestAsyncResult.succeeded()) { + HttpClientRequest httpClientRequest = httpClientRequestAsyncResult.result(); + httpClientRequest.send() + .onComplete(httpClientResponseAsyncResult -> { + HttpClientResponse httpClientResponse = httpClientResponseAsyncResult.result(); + assertEquals(200, httpClientResponse.statusCode()); + MatcherAssert.assertThat(httpClientResponse.body(), is("Hello World!")); + }); + } + }); + } + public abstract RequestSpecification requests(); private Map parseMetrics(String body) {