Skip to content

Commit

Permalink
feat: Include OpenLiberty following JAVA_TOOL_OPTIONS fix
Browse files Browse the repository at this point in the history
See #8031
  • Loading branch information
PerfectSlayer committed Nov 28, 2024
1 parent f397613 commit 1f9b6f6
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 27 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,15 +22,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() {
// Make a copy of the OpenLiberty runnable JAR before injecting JVM configuration
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,15 +22,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() {
// Make a copy of the OpenLiberty runnable JAR before injecting JVM configuration
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,15 +21,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() {
// Make a copy of the OpenLiberty runnable JAR before injecting JVM configuration
Expand Down

0 comments on commit 1f9b6f6

Please sign in to comment.