fix(deps): update errorproneversion to v2.39.0 (#7448)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: John Watson <[email protected]>

Author: renovate[bot]

api/testing-internal/src/main/java/io/opentelemetry/api/testing/internal/AbstractDefaultMeterTest.java

@@ -25,7 +25,7 @@
 import org.junit.jupiter.api.Test;
 
 /** Unit tests for No-op {@link Meter}. */
-@SuppressLogger()
+@SuppressLogger
 public abstract class AbstractDefaultMeterTest {
   private final Meter meter = getMeter();
 

dependencyManagement/build.gradle.kts

@@ -28,7 +28,7 @@ val DEPENDENCY_BOMS = listOf(
 )
 
 val autoValueVersion = "1.11.0"
-val errorProneVersion = "2.38.0"
+val errorProneVersion = "2.39.0"
 val jmhVersion = "1.37"
 // Mockito 5.x.x requires Java 11 https://github.com/mockito/mockito/releases/tag/v5.0.0
 val mockitoVersion = "4.11.0"

exporters/common/src/test/java/io/opentelemetry/exporter/internal/metrics/ExporterInstrumentationTest.java

@@ -36,7 +36,7 @@ class ExporterInstrumentationTest {
   Supplier<MeterProvider> meterProviderSupplier = mock(Supplier.class);
 
   @ParameterizedTest
-  @EnumSource()
+  @EnumSource
   void validMeterProvider(InternalTelemetryVersion schemaVersion) {
     when(meterProviderSupplier.get())
         .thenReturn(
@@ -84,7 +84,7 @@ public AggregationTemporality getAggregationTemporality(
   }
 
   @ParameterizedTest
-  @EnumSource()
+  @EnumSource
   void noopMeterProvider(InternalTelemetryVersion schemaVersion) {
 
     when(meterProviderSupplier.get()).thenReturn(MeterProvider.noop());

exporters/sender/okhttp/src/main/java/io/opentelemetry/exporter/sender/okhttp/internal/OkHttpGrpcSender.java

@@ -75,6 +75,7 @@ public final class OkHttpGrpcSender<T extends Marshaler> implements GrpcSender<T
   @Nullable private final Compressor compressor;
 
   /** Creates a new {@link OkHttpGrpcSender}. */
+  @SuppressWarnings("TooManyParameters")
   public OkHttpGrpcSender(
       String endpoint,
       @Nullable Compressor compressor,

opencensus-shim/src/main/java/io/opentelemetry/opencensusshim/OpenTelemetrySpanBuilderImpl.java

@@ -129,12 +129,12 @@ public Span startSpan() {
     Boolean hasRemoteParent = null;
     if (ocRemoteParentSpanContext != null && ocRemoteParentSpanContext.isValid()) {
       ocParentContext = ocRemoteParentSpanContext;
-      hasRemoteParent = Boolean.TRUE;
+      hasRemoteParent = true;
       ocTraceId = ocParentContext.getTraceId();
       ocTracestate = ocParentContext.getTracestate();
     } else if (ocParent != null && ocParent.getContext().isValid()) {
       ocParentContext = ocParent.getContext();
-      hasRemoteParent = Boolean.FALSE;
+      hasRemoteParent = false;
       ocTraceId = ocParentContext.getTraceId();
       ocTracestate = ocParentContext.getTracestate();
     } else {

sdk-extensions/incubator/src/main/java/io/opentelemetry/sdk/extension/incubator/fileconfig/OpenTelemetryConfigurationFactory.java

@@ -39,7 +39,7 @@ public OpenTelemetrySdk create(
     // TODO(jack-berg): log warning if version is not exact match, which may result in unexpected
     // behavior for experimental properties.
 
-    if (Objects.equals(Boolean.TRUE, model.getDisabled())) {
+    if (Objects.equals(true, model.getDisabled())) {
       return builder.build();
     }
 

sdk/metrics/src/test/java/io/opentelemetry/sdk/metrics/internal/data/MutableHistogramPointDataTest.java

@@ -103,7 +103,7 @@ void testSanity() {
     assertThat(anotherPointData.hashCode()).isNotEqualTo(pointData.hashCode());
   }
 
-  @Test()
+  @Test
   void testBoundaries() {
     MutableHistogramPointData pointData = new MutableHistogramPointData(10);
     assertThatThrownBy(