Disable css on all okhttp3 tests

amarziali · amarziali · commit 3146c32a6654 · 2025-09-11T15:57:27.000+02:00
diff --git a/dd-java-agent/instrumentation/okhttp-3/src/test/groovy/IastOkHttp3InstrumentationTest.groovy b/dd-java-agent/instrumentation/okhttp-3/src/test/groovy/IastOkHttp3InstrumentationTest.groovy
@@ -18,6 +18,9 @@ class IastOkHttp3InstrumentationTest extends InstrumentationSpecification {
     // HttpUrl gets loaded early so we have to disable the advice transformer
     IastHttpUrlInstrumentation.ENABLE_ADVICE_TRANSFORMER = false
     injectSysConfig('dd.iast.enabled', 'true')
+    // disable tracer metrics because it uses OkHttp and class loading is
+    // not isolated in tests
+    injectSysConfig("trace.stats.computation.enabled", "false")
   }
 
   @Override
diff --git a/dd-java-agent/instrumentation/okhttp-3/src/test/groovy/OkHttp3Test.groovy b/dd-java-agent/instrumentation/okhttp-3/src/test/groovy/OkHttp3Test.groovy
@@ -14,7 +14,7 @@ abstract class OkHttp3Test extends HttpClientTest {
     super.configurePreAgent()
     // disable tracer metrics because it uses OkHttp and class loading is
     // not isolated in tests
-    injectSysConfig("dd.trace.tracer.metrics.enabled", "false")
+    injectSysConfig("trace.stats.computation.enabled", "false")
   }
 
   @Override

Original file line number	Diff line number	Diff line change
`@@ -18,6 +18,9 @@ class IastOkHttp3InstrumentationTest extends InstrumentationSpecification {`
`18`	`18`	`// HttpUrl gets loaded early so we have to disable the advice transformer`
`19`	`19`	`IastHttpUrlInstrumentation.ENABLE_ADVICE_TRANSFORMER = false`
`20`	`20`	`injectSysConfig('dd.iast.enabled', 'true')`
	`21`	`+ // disable tracer metrics because it uses OkHttp and class loading is`
	`22`	`+ // not isolated in tests`
	`23`	`+ injectSysConfig("trace.stats.computation.enabled", "false")`
`21`	`24`	`}`
`22`	`25`
`23`	`26`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ abstract class OkHttp3Test extends HttpClientTest {`
`14`	`14`	`super.configurePreAgent()`
`15`	`15`	`// disable tracer metrics because it uses OkHttp and class loading is`
`16`	`16`	`// not isolated in tests`
`17`		`- injectSysConfig("dd.trace.tracer.metrics.enabled", "false")`
	`17`	`+ injectSysConfig("trace.stats.computation.enabled", "false")`
`18`	`18`	`}`
`19`	`19`
`20`	`20`	`@Override`