open-telemetry
/
opentelemetry-java-instrumentation
mirror of https://github.com/open-telemetry/opentelemetry-java-instrumentation.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Java 7 doesn't have Process.waitFor(time) so use Spock's timeout

This commit is contained in:
Laplie Anderson 2019-07-18 16:28:35 -04:00
parent 8909e7d4e1
commit 74099735ed
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
dd-smoke-tests/cli/src/test/groovy/datadog/smoketest/CliApplicationSmokeTest.groovy
View File

@ -1,11 +1,12 @@
package datadog.smoketest package datadog.smoketest
import spock.lang.Timeout
import java.util.concurrent.TimeUnit import java.util.concurrent.TimeUnit
class CliApplicationSmokeTest extends AbstractSmokeTest { class CliApplicationSmokeTest extends AbstractSmokeTest {
// Estimate for the amount of time instrumentation, plus request, plus some extra // Estimate for the amount of time instrumentation, plus request, plus some extra
private static final long TIMEOUT_SECS = 10 private static final int TIMEOUT_SECS = 10
@Override @Override
ProcessBuilder createProcessBuilder() { ProcessBuilder createProcessBuilder() {
@ -19,10 +20,9 @@ class CliApplicationSmokeTest extends AbstractSmokeTest {
processBuilder.directory(new File(buildDirectory)) processBuilder.directory(new File(buildDirectory))
} }
@Timeout(value = TIMEOUT_SECS, unit = TimeUnit.SECONDS)
def "Cli application process ends before timeout"() { def "Cli application process ends before timeout"() {
expect: expect:
assert serverProcess.waitFor(TIMEOUT_SECS, TimeUnit.SECONDS) assert serverProcess.waitFor() == 0
assert serverProcess.exitValue() == 0
} }
} }