From bf54895fb08470aea320eb3997f9f600efdf852f Mon Sep 17 00:00:00 2001 From: Alexey Kuznetsov Date: Mon, 23 Feb 2026 11:20:33 -0500 Subject: [PATCH] Refactored smoke tests to start agent before smock application started to reduce potential flakiness. --- .../src/main/groovy/datadog/smoketest/AbstractSmokeTest.groovy | 3 ++- .../src/main/groovy/datadog/smoketest/ProcessManager.groovy | 3 +++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/dd-smoke-tests/src/main/groovy/datadog/smoketest/AbstractSmokeTest.groovy b/dd-smoke-tests/src/main/groovy/datadog/smoketest/AbstractSmokeTest.groovy index 8a246ed0c03..b256cc4fc4a 100644 --- a/dd-smoke-tests/src/main/groovy/datadog/smoketest/AbstractSmokeTest.groovy +++ b/dd-smoke-tests/src/main/groovy/datadog/smoketest/AbstractSmokeTest.groovy @@ -287,7 +287,8 @@ abstract class AbstractSmokeTest extends ProcessManager { } } - def setupSpec() { + @Override + protected void setupTracesConsumer() { startServer() } diff --git a/dd-smoke-tests/src/main/groovy/datadog/smoketest/ProcessManager.groovy b/dd-smoke-tests/src/main/groovy/datadog/smoketest/ProcessManager.groovy index 5f23eafb04f..5c06793dd15 100644 --- a/dd-smoke-tests/src/main/groovy/datadog/smoketest/ProcessManager.groovy +++ b/dd-smoke-tests/src/main/groovy/datadog/smoketest/ProcessManager.groovy @@ -71,6 +71,7 @@ abstract class ProcessManager extends Specification { assert Files.isDirectory(Paths.get(buildDirectory)) assert Files.isRegularFile(Paths.get(shadowJarPath)) + setupTracesConsumer() beforeProcessBuilders() (0.. @@ -237,6 +238,8 @@ abstract class ProcessManager extends Specification { outputThreads.processTestLogLines { return checker(it) } } + protected abstract void setupTracesConsumer() + protected void beforeProcessBuilders() {} protected ProcessBuilder createProcessBuilder() {