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 6ec98b2 commit 634c68a
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 @@ -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<String> command = new ArrayList<>()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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<String> command = new ArrayList<>()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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<String> command = new ArrayList<>()
Expand Down

0 comments on commit 634c68a

Please sign in to comment.