From e429fa78d8c68e57f07564ddbae5d4d86c58436b Mon Sep 17 00:00:00 2001 From: Andrea Marziali Date: Mon, 29 Jun 2026 13:48:05 +0200 Subject: [PATCH] Quickwin: remove unneded strictTraceWrites overrides --- .../mule-4.5/src/test/groovy/mule4/MuleForkedTest.groovy | 6 ------ .../src/test/groovy/mule4/MuleHttpServerForkedTest.groovy | 6 ------ .../src/test/groovy/OpensearchRestClientTest.groovy | 6 ------ .../src/test/groovy/OpensearchNodeClientTest.groovy | 6 ------ 4 files changed, 24 deletions(-) diff --git a/dd-java-agent/instrumentation/mule-4.5/src/test/groovy/mule4/MuleForkedTest.groovy b/dd-java-agent/instrumentation/mule-4.5/src/test/groovy/mule4/MuleForkedTest.groovy index 81c9b9c2a9f..0fc8809a8b6 100644 --- a/dd-java-agent/instrumentation/mule-4.5/src/test/groovy/mule4/MuleForkedTest.groovy +++ b/dd-java-agent/instrumentation/mule-4.5/src/test/groovy/mule4/MuleForkedTest.groovy @@ -22,12 +22,6 @@ import spock.lang.Shared class MuleForkedTest extends WithHttpServer { - // TODO since mule uses reactor core, things sometime propagate to places where they're not closed - @Override - boolean useStrictTraceWrites() { - return false - } - @Override protected void configurePreAgent() { super.configurePreAgent() diff --git a/dd-java-agent/instrumentation/mule-4.5/src/test/groovy/mule4/MuleHttpServerForkedTest.groovy b/dd-java-agent/instrumentation/mule-4.5/src/test/groovy/mule4/MuleHttpServerForkedTest.groovy index c5b060a2f2b..a9cc6ba8e9b 100644 --- a/dd-java-agent/instrumentation/mule-4.5/src/test/groovy/mule4/MuleHttpServerForkedTest.groovy +++ b/dd-java-agent/instrumentation/mule-4.5/src/test/groovy/mule4/MuleHttpServerForkedTest.groovy @@ -9,12 +9,6 @@ import spock.lang.Shared class MuleHttpServerForkedTest extends HttpServerTest { - // TODO since mule uses reactor core, things sometime propagate to places where they're not closed - @Override - boolean useStrictTraceWrites() { - return false - } - @Override boolean testRedirect() { // Dynamic adding of headers to and HttpResponse from inside a mule application seems diff --git a/dd-java-agent/instrumentation/opensearch/opensearch-rest-1.0/src/test/groovy/OpensearchRestClientTest.groovy b/dd-java-agent/instrumentation/opensearch/opensearch-rest-1.0/src/test/groovy/OpensearchRestClientTest.groovy index fe8a6e8d4e3..b6722fbc7e3 100644 --- a/dd-java-agent/instrumentation/opensearch/opensearch-rest-1.0/src/test/groovy/OpensearchRestClientTest.groovy +++ b/dd-java-agent/instrumentation/opensearch/opensearch-rest-1.0/src/test/groovy/OpensearchRestClientTest.groovy @@ -31,12 +31,6 @@ class OpensearchRestClientTest extends InstrumentationSpecification { @Shared RestClient client - @Override - boolean useStrictTraceWrites() { - //FIXME IDM - false - } - def setupSpec() { aosWorkingDir = File.createTempDir("test-aos-working-dir-", "") diff --git a/dd-java-agent/instrumentation/opensearch/opensearch-transport-1.0/src/test/groovy/OpensearchNodeClientTest.groovy b/dd-java-agent/instrumentation/opensearch/opensearch-transport-1.0/src/test/groovy/OpensearchNodeClientTest.groovy index 153a764ac71..97a8f801ea0 100644 --- a/dd-java-agent/instrumentation/opensearch/opensearch-transport-1.0/src/test/groovy/OpensearchNodeClientTest.groovy +++ b/dd-java-agent/instrumentation/opensearch/opensearch-transport-1.0/src/test/groovy/OpensearchNodeClientTest.groovy @@ -28,12 +28,6 @@ class OpensearchNodeClientTest extends InstrumentationSpecification { def client = testNode.client() - @Override - boolean useStrictTraceWrites() { - //FIXME IDM - false - } - @Override protected void configurePreAgent() { super.configurePreAgent()