From 4e795042e9d486cdb01ccb1cd1addc77e3cdd9f4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Rapha=C3=ABl=20Vandon?= Date: Mon, 6 Jan 2025 11:10:31 +0100 Subject: [PATCH] Remove restriction to not run vertx4 latest tests on java 17 (#8133) * does this pass ? * Revert "does this pass ?" This reverts commit 57dc9941485065ff31015a840983dc7e7080dbfa. * use java 17 only for latest dep tests --- dd-java-agent/instrumentation/vertx-web-4.0/build.gradle | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/dd-java-agent/instrumentation/vertx-web-4.0/build.gradle b/dd-java-agent/instrumentation/vertx-web-4.0/build.gradle index ff0f792de7d..1f3deaca347 100644 --- a/dd-java-agent/instrumentation/vertx-web-4.0/build.gradle +++ b/dd-java-agent/instrumentation/vertx-web-4.0/build.gradle @@ -1,7 +1,8 @@ // Set properties before any plugins get loaded ext { - // TODO Java 17: This version of vertx-web doesn't support Java 17 + // vertx-web doesn't support Java 17 until v4.2 maxJavaVersionForTests = JavaVersion.VERSION_15 + latestDepTestMaxJavaVersionForTests = JavaVersion.VERSION_17 } apply from: "$rootDir/gradle/java.gradle" @@ -29,7 +30,7 @@ artifacts { dependencies { api project(':dd-java-agent:instrumentation:netty-4.1-shared') - compileOnly group: 'io.vertx', name: 'vertx-web', version: '4.2.7' + compileOnly group: 'io.vertx', name: 'vertx-web', version: '4.0.0' testImplementation project(':dd-java-agent:instrumentation:netty-4.1') testImplementation project(':dd-java-agent:instrumentation:trace-annotation') @@ -44,6 +45,7 @@ dependencies { testRuntimeOnly project(':dd-java-agent:instrumentation:jackson-core') testRuntimeOnly project(':dd-java-agent:instrumentation:netty-buffer-4') + // TODO support v>=4.5 latestDepTestImplementation group: 'io.vertx', name: 'vertx-web', version: '4.4.+' latestDepTestImplementation group: 'io.vertx', name: 'vertx-web-client', version: '4.4.+' }