diff --git a/dd-smoke-tests/cli/src/test/groovy/datadog/smoketest/CliApplicationSmokeTest.groovy b/dd-smoke-tests/cli/src/test/groovy/datadog/smoketest/CliApplicationSmokeTest.groovy index 425fb1095d..44c44b0e9f 100644 --- a/dd-smoke-tests/cli/src/test/groovy/datadog/smoketest/CliApplicationSmokeTest.groovy +++ b/dd-smoke-tests/cli/src/test/groovy/datadog/smoketest/CliApplicationSmokeTest.groovy @@ -1,11 +1,12 @@ package datadog.smoketest +import spock.lang.Timeout import java.util.concurrent.TimeUnit class CliApplicationSmokeTest extends AbstractSmokeTest { // 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 ProcessBuilder createProcessBuilder() { @@ -19,10 +20,9 @@ class CliApplicationSmokeTest extends AbstractSmokeTest { processBuilder.directory(new File(buildDirectory)) } + @Timeout(value = TIMEOUT_SECS, unit = TimeUnit.SECONDS) def "Cli application process ends before timeout"() { expect: - assert serverProcess.waitFor(TIMEOUT_SECS, TimeUnit.SECONDS) - - assert serverProcess.exitValue() == 0 + assert serverProcess.waitFor() == 0 } }