Skip to content

Commit 6cca46e

Browse files
nikita-tkachenko-datadognikita-tkachenko-datadog
authored
Fix Test Optimization to work with JDK 24 (#9114)
1 parent 041214b commit 6cca46e

File tree

6 files changed

+26
-33
lines changed

6 files changed

+26
-33
lines changed

‎dd-smoke-tests/gradle/src/test/groovy/datadog/smoketest/AbstractGradleTest.groovy‎

Lines changed: 18 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import com.fasterxml.jackson.databind.JsonNode
44
import com.fasterxml.jackson.databind.ObjectMapper
55
import datadog.environment.JavaVirtualMachine
66
import datadog.trace.civisibility.CiVisibilitySmokeTest
7+
import datadog.trace.util.ComparableVersion
78
import okhttp3.OkHttpClient
89
import okhttp3.Request
910
import okhttp3.Response
@@ -84,42 +85,42 @@ class AbstractGradleTest extends CiVisibilitySmokeTest {
8485
}
8586

8687
protected void givenGradleVersionIsCompatibleWithCurrentJvm(String gradleVersion) {
87-
Assumptions.assumeTrue(isSupported(gradleVersion),
88+
Assumptions.assumeTrue(isSupported(new ComparableVersion(gradleVersion)),
8889
"Current JVM " + Jvm.current.javaVersion + " does not support Gradle version " + gradleVersion)
8990
}
9091

91-
private static boolean isSupported(String gradleVersion) {
92+
private static boolean isSupported(ComparableVersion gradleVersion) {
9293
// https://docs.gradle.org/current/userguide/compatibility.html
9394
if (Jvm.current.isJavaVersionCompatible(24)) {
94-
return gradleVersion >= "8.14"
95+
return gradleVersion.compareTo(new ComparableVersion("8.14")) >= 0
9596
} else if (Jvm.current.java21Compatible) {
96-
return gradleVersion >= "8.4"
97+
return gradleVersion.compareTo(new ComparableVersion("8.4")) >= 0
9798
} else if (Jvm.current.java20) {
98-
return gradleVersion >= "8.1"
99+
return gradleVersion.compareTo(new ComparableVersion("8.1")) >= 0
99100
} else if (Jvm.current.java19) {
100-
return gradleVersion >= "7.6"
101+
return gradleVersion.compareTo(new ComparableVersion("7.6")) >= 0
101102
} else if (Jvm.current.java18) {
102-
return gradleVersion >= "7.5"
103+
return gradleVersion.compareTo(new ComparableVersion("7.5")) >= 0
103104
} else if (Jvm.current.java17) {
104-
return gradleVersion >= "7.3"
105+
return gradleVersion.compareTo(new ComparableVersion("7.3")) >= 0
105106
} else if (Jvm.current.java16) {
106-
return gradleVersion >= "7.0"
107+
return gradleVersion.compareTo(new ComparableVersion("7.0")) >= 0
107108
} else if (Jvm.current.java15) {
108-
return gradleVersion >= "6.7"
109+
return gradleVersion.compareTo(new ComparableVersion("6.7")) >= 0
109110
} else if (Jvm.current.java14) {
110-
return gradleVersion >= "6.3"
111+
return gradleVersion.compareTo(new ComparableVersion("6.3")) >= 0
111112
} else if (Jvm.current.java13) {
112-
return gradleVersion >= "6.0"
113+
return gradleVersion.compareTo(new ComparableVersion("6.0")) >= 0
113114
} else if (Jvm.current.java12) {
114-
return gradleVersion >= "5.4"
115+
return gradleVersion.compareTo(new ComparableVersion("5.4")) >= 0
115116
} else if (Jvm.current.java11) {
116-
return gradleVersion >= "5.0"
117+
return gradleVersion.compareTo(new ComparableVersion("5.0")) >= 0
117118
} else if (Jvm.current.java10) {
118-
return gradleVersion >= "4.7"
119+
return gradleVersion.compareTo(new ComparableVersion("4.7")) >= 0
119120
} else if (Jvm.current.java9) {
120-
return gradleVersion >= "4.3"
121+
return gradleVersion.compareTo(new ComparableVersion("4.3")) >= 0
121122
} else if (Jvm.current.java8) {
122-
return gradleVersion >= "2.0"
123+
return gradleVersion.compareTo(new ComparableVersion("2.0")) >= 0
123124
}
124125
return false
125126
}

‎dd-smoke-tests/gradle/src/test/groovy/datadog/smoketest/GradleDaemonSmokeTest.groovy‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,6 @@ class GradleDaemonSmokeTest extends AbstractGradleTest {
6060
"7.6.4" | "test-corrupted-config-legacy-instrumentation" | false | 1 | 0
6161
}
6262

63-
@IgnoreIf(reason = "Failing on Java 24. Skip until we have a fix.", value = {
64-
JavaVirtualMachine.isJavaVersionAtLeast(24)
65-
})
6663
def "test #projectName, v#gradleVersion, configCache: #configurationCache"() {
6764
runGradleTest(gradleVersion, projectName, configurationCache, successExpected, flakyRetries, expectedTraces, expectedCoverages)
6865

‎dd-smoke-tests/maven/src/test/groovy/datadog/smoketest/MavenSmokeTest.groovy‎

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package datadog.smoketest
22

3-
import datadog.environment.JavaVirtualMachine
43
import datadog.trace.api.Config
54
import datadog.trace.api.civisibility.CIConstants
65
import datadog.trace.api.config.CiVisibilityConfig
@@ -27,14 +26,10 @@ import org.slf4j.LoggerFactory
2726
import org.w3c.dom.Document
2827
import org.w3c.dom.NodeList
2928
import spock.lang.AutoCleanup
30-
import spock.lang.IgnoreIf
3129
import spock.lang.Shared
3230
import spock.lang.TempDir
3331
import spock.util.environment.Jvm
3432

35-
@IgnoreIf(reason = "Failing on Java 24. Skip until we have a fix.", value = {
36-
JavaVirtualMachine.isJavaVersionAtLeast(24)
37-
})
3833
class MavenSmokeTest extends CiVisibilitySmokeTest {
3934

4035
private static final Logger LOGGER = LoggerFactory.getLogger(MavenSmokeTest.class)

‎dd-smoke-tests/maven/src/test/resources/test_successful_maven_run_junit_platform_runner/events.ftl‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
"test.code_coverage.enabled" : "true",
2323
"test.command" : "mvn -B test",
2424
"test.framework" : "spock",
25-
"test.framework_version" : "2.4.0-M2-groovy-4.0",
25+
"test.framework_version" : "2.4.0-M6-groovy-4.0",
2626
"test.status" : "pass",
2727
"test.toolchain" : ${content_meta_test_toolchain},
2828
"test.type" : "test",
@@ -66,7 +66,7 @@
6666
"test.command" : "mvn -B test",
6767
"test.execution" : "maven-surefire-plugin:test:default-test",
6868
"test.framework" : "spock",
69-
"test.framework_version" : "2.4.0-M2-groovy-4.0",
69+
"test.framework_version" : "2.4.0-M6-groovy-4.0",
7070
"test.module" : "Maven Smoke Tests Project maven-surefire-plugin default-test",
7171
"test.status" : "pass",
7272
"test.type" : "test",
@@ -298,7 +298,7 @@
298298
"runtime.version" : ${content_meta_runtime_version},
299299
"span.kind" : "test_suite_end",
300300
"test.framework" : "spock",
301-
"test.framework_version" : "2.4.0-M2-groovy-4.0",
301+
"test.framework_version" : "2.4.0-M6-groovy-4.0",
302302
"test.module" : "Maven Smoke Tests Project maven-surefire-plugin default-test",
303303
"test.status" : "pass",
304304
"test.suite" : "test_successful_maven_run_junit_platform_runner.src.test.groovy.SampleSpockTest",
@@ -350,7 +350,7 @@
350350
"runtime.version" : ${content_meta_runtime_version},
351351
"span.kind" : "test",
352352
"test.framework" : "spock",
353-
"test.framework_version" : "2.4.0-M2-groovy-4.0",
353+
"test.framework_version" : "2.4.0-M6-groovy-4.0",
354354
"test.module" : "Maven Smoke Tests Project maven-surefire-plugin default-test",
355355
"test.name" : "test should pass",
356356
"test.source.method" : "test should pass()V",
@@ -378,4 +378,4 @@
378378
},
379379
"type" : "test",
380380
"version" : 2
381-
} ]
381+
} ]

‎dd-smoke-tests/maven/src/test/resources/test_successful_maven_run_junit_platform_runner/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@
5050
<dependency>
5151
<groupId>org.spockframework</groupId>
5252
<artifactId>spock-bom</artifactId>
53-
<version>2.4-M2-groovy-4.0</version>
53+
<version>2.4-M6-groovy-4.0</version>
5454
<type>pom</type>
5555
<scope>import</scope>
5656
</dependency>

‎dd-trace-api/src/main/java/datadog/trace/api/ConfigDefaults.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -160,8 +160,8 @@ public final class ConfigDefaults {
160160
static final boolean DEFAULT_CIVISIBILITY_BUILD_INSTRUMENTATION_ENABLED = true;
161161
static final boolean DEFAULT_CIVISIBILITY_AUTO_CONFIGURATION_ENABLED = true;
162162
static final boolean DEFAULT_CIVISIBILITY_COMPILER_PLUGIN_AUTO_CONFIGURATION_ENABLED = true;
163-
static final String DEFAULT_CIVISIBILITY_COMPILER_PLUGIN_VERSION = "0.2.2";
164-
static final String DEFAULT_CIVISIBILITY_JACOCO_PLUGIN_VERSION = "0.8.12";
163+
static final String DEFAULT_CIVISIBILITY_COMPILER_PLUGIN_VERSION = "0.2.4";
164+
static final String DEFAULT_CIVISIBILITY_JACOCO_PLUGIN_VERSION = "0.8.13";
165165
static final String DEFAULT_CIVISIBILITY_JACOCO_PLUGIN_EXCLUDES =
166166
"datadog.trace.*:org.apache.commons.*:org.mockito.*";
167167
static final boolean DEFAULT_CIVISIBILITY_GIT_UPLOAD_ENABLED = true;

0 commit comments

Comments
 (0)