From 634c68a936e2748e0548b93e8683dbe1f9a2c21c Mon Sep 17 00:00:00 2001 From: Bruce Bujon Date: Thu, 28 Nov 2024 15:57:26 +0100 Subject: [PATCH] feat: Include OpenLiberty following JAVA_TOOL_OPTIONS fix See #8031 --- .../smoketest/SpringBootOpenLibertySmokeTest.groovy | 9 --------- .../smoketest/SpringBootOpenLibertySmokeTest.groovy | 9 --------- .../SpringBootOpenLibertySmokeVulnerabilityTest.groovy | 9 --------- 3 files changed, 27 deletions(-) diff --git a/dd-smoke-tests/springboot-openliberty-20/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeTest.groovy b/dd-smoke-tests/springboot-openliberty-20/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeTest.groovy index b8b220c3c9c..83a44b2f6b0 100644 --- a/dd-smoke-tests/springboot-openliberty-20/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeTest.groovy +++ b/dd-smoke-tests/springboot-openliberty-20/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeTest.groovy @@ -20,15 +20,6 @@ class SpringBootOpenLibertySmokeTest extends AbstractServerSmokeTest { @Shared String openLibertyShadowJar = System.getProperty("datadog.smoketest.openliberty.jar.path") - @Override - def testCrashTracking() { - // DQH - 2024 Nov - // Concatenating into a single string to use JAVA_TOOL_OPTIONS doesn't play nice - // with the quoting currently used by crash tracking smoke testing in AbstractSmokeTest, - // so skipping for this test. - return false - } - @Override ProcessBuilder createProcessBuilder() { List command = new ArrayList<>() diff --git a/dd-smoke-tests/springboot-openliberty-23/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeTest.groovy b/dd-smoke-tests/springboot-openliberty-23/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeTest.groovy index 8f17443ed4e..0ed802f9f04 100644 --- a/dd-smoke-tests/springboot-openliberty-23/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeTest.groovy +++ b/dd-smoke-tests/springboot-openliberty-23/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeTest.groovy @@ -20,15 +20,6 @@ class SpringBootOpenLibertySmokeTest extends AbstractServerSmokeTest { @Shared String openLibertyShadowJar = System.getProperty("datadog.smoketest.openliberty.jar.path") - @Override - def testCrashTracking() { - // DQH - 2024 Nov - // Concatenating into a single string to use JAVA_TOOL_OPTIONS doesn't play nice - // with the quoting currently used by crash tracking smoke testing in AbstractSmokeTest, - // so skipping for this test. - return false - } - @Override ProcessBuilder createProcessBuilder() { List command = new ArrayList<>() diff --git a/dd-smoke-tests/springboot-openliberty-23/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeVulnerabilityTest.groovy b/dd-smoke-tests/springboot-openliberty-23/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeVulnerabilityTest.groovy index 62eea3883de..3ed868282c0 100644 --- a/dd-smoke-tests/springboot-openliberty-23/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeVulnerabilityTest.groovy +++ b/dd-smoke-tests/springboot-openliberty-23/src/test/groovy/datadog/smoketest/SpringBootOpenLibertySmokeVulnerabilityTest.groovy @@ -19,15 +19,6 @@ class SpringBootOpenLibertySmokeVulnerabilityTest extends AbstractServerSmokeTes @Shared String openLibertyShadowJar = System.getProperty("datadog.smoketest.openliberty.jar.path") - @Override - def testCrashTracking() { - // DQH - 2024 Nov - // Concatenating into a single string to use JAVA_TOOL_OPTIONS doesn't play nice - // with the quoting currently used by crash tracking smoke testing in AbstractSmokeTest, - // so skipping for this test. - return false - } - @Override ProcessBuilder createProcessBuilder() { List command = new ArrayList<>()