From 0e476b9a6342411a7a8fd80c7d0b2eaab6f0a291 Mon Sep 17 00:00:00 2001 From: Stuart McCulloch Date: Mon, 29 Jun 2026 13:36:37 +0100 Subject: [PATCH 1/2] Exclude patched OkHttp classes and trivial I/O classes from coverage --- communication/build.gradle.kts | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/communication/build.gradle.kts b/communication/build.gradle.kts index 8a4285843b0..afa25ef5a99 100644 --- a/communication/build.gradle.kts +++ b/communication/build.gradle.kts @@ -39,7 +39,12 @@ dependencies { extra["minimumBranchCoverage"] = 0.5 extra["minimumInstructionCoverage"] = 0.8 extra["excludedClassesCoverage"] = listOf( + "okhttp3.internal.PatchUtil", + "okhttp3.internal.PatchUtil.1", + "okhttp3.internal.PatchUtil.2", + "okhttp3.internal.platform.PatchPlatform", "datadog.communication.ddagent.ExternalAgentLauncher", + "datadog.communication.ddagent.NoopFeaturesDiscovery", "datadog.communication.ddagent.ExternalAgentLauncher.NamedPipeHealthCheck", "datadog.communication.ddagent.SharedCommunicationObjects.FixedConfigUrlSupplier", "datadog.communication.ddagent.SharedCommunicationObjects.RetryConfigUrlSupplier", @@ -56,6 +61,7 @@ extra["excludedClassesCoverage"] = listOf( "datadog.communication.IntakeApi", "datadog.communication.util.IOUtils", "datadog.communication.util.IOUtils.1", + "datadog.communication.http.SocketUtils", ) extra["excludedClassesBranchCoverage"] = listOf( "datadog.communication.ddagent.TracerVersion", From f305af9c18464f530d6f475901111863ea2f77bd Mon Sep 17 00:00:00 2001 From: Stuart McCulloch Date: Mon, 29 Jun 2026 13:36:54 +0100 Subject: [PATCH 2/2] Remove unused matches(CacheEntry) method --- .../datadog/communication/serialization/SimpleUtf8Cache.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/communication/src/main/java/datadog/communication/serialization/SimpleUtf8Cache.java b/communication/src/main/java/datadog/communication/serialization/SimpleUtf8Cache.java index 93e6c92bb9b..eb227d1385e 100644 --- a/communication/src/main/java/datadog/communication/serialization/SimpleUtf8Cache.java +++ b/communication/src/main/java/datadog/communication/serialization/SimpleUtf8Cache.java @@ -179,10 +179,6 @@ public CacheEntry(int adjHash, String value) { this.valueUtf8 = utf8(value); } - boolean matches(CacheEntry thatEntry) { - return (this == thatEntry) || this.matches(thatEntry.adjHash, thatEntry.value); - } - boolean matches(int adjHash, String value) { return (this.adjHash == adjHash) && value.equals(this.value); }