fix(deps): update errorproneversion to v2.37.0 (minor) (#13552)

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

Author: renovate[bot]

conventions/src/main/kotlin/otel.errorprone-conventions.gradle.kts

@@ -10,6 +10,7 @@ dependencies {
 }
 
 val disableErrorProne = properties["disableErrorProne"]?.toString()?.toBoolean() ?: false
+val testLatestDeps = gradle.startParameter.projectProperties["testLatestDeps"] == "true"
 
 tasks {
   withType<JavaCompile>().configureEach {
@@ -123,6 +124,16 @@ tasks {
 
         disable("NonFinalStaticField")
 
+        // We get this warning in modules that compile for old java versions
+        disable("StringConcatToTextBlock")
+
+        if (testLatestDeps) {
+          // Some latest dep tests are compiled for java 17 although the base version uses an older
+          // version. Disable rules that suggest using new language features.
+          disable("StatementSwitchToExpressionSwitch")
+          disable("PatternMatchingInstanceof")
+        }
+
         if (name.contains("Jmh") || name.contains("Test")) {
           // Allow underscore in test-type method names
           disable("MemberName")

custom-checks/src/test/java/io/opentelemetry/javaagent/customchecks/OtelInternalJavadocTest.java

@@ -12,13 +12,47 @@ class OtelInternalJavadocTest {
 
   @Test
   void test() {
-    doTest("internal/InternalJavadocPositiveCases.java");
-    doTest("internal/InternalJavadocNegativeCases.java");
+    doTest(
+        "internal/InternalJavadocPositiveCases.java",
+        """
+package io.opentelemetry.javaagent.customchecks.internal;
+
+// BUG: Diagnostic contains: doesn't end with any of the applicable javadoc disclaimers
+public class InternalJavadocPositiveCases {
+
+  // BUG: Diagnostic contains: doesn't end with any of the applicable javadoc disclaimers
+  public static class One {}
+
+  /** Doesn't have the disclaimer. */
+  // BUG: Diagnostic contains: doesn't end with any of the applicable javadoc disclaimers
+  public static class Two {}
+}
+""");
+    doTest(
+        "internal/InternalJavadocNegativeCases.java",
+        """
+package io.opentelemetry.javaagent.customchecks.internal;
+
+/**
+ * This class is internal and is hence not for public use. Its APIs are unstable and can change at
+ * any time.
+ */
+public class InternalJavadocNegativeCases {
+
+  /**
+   * This class is internal and is hence not for public use. Its APIs are unstable and can change at
+   * any time.
+   */
+  public static class One {}
+
+  static class Two {}
+}
+""");
   }
 
-  private static void doTest(String path) {
+  private static void doTest(String path, String... lines) {
     CompilationTestHelper.newInstance(OtelInternalJavadoc.class, OtelInternalJavadocTest.class)
-        .addSourceFile(path)
+        .addSourceLines(path, lines)
         .doTest();
   }
 }

custom-checks/src/test/resources/io/opentelemetry/javaagent/customchecks/internal/InternalJavadocNegativeCases.java

@@ -38,7 +38,7 @@ val DEPENDENCY_BOMS = listOf(
 
 val autoServiceVersion = "1.1.1"
 val autoValueVersion = "1.11.0"
-val errorProneVersion = "2.36.0"
+val errorProneVersion = "2.37.0"
 val byteBuddyVersion = "1.17.4"
 val asmVersion = "9.7.1"
 val jmhVersion = "1.37"

custom-checks/src/test/resources/io/opentelemetry/javaagent/customchecks/internal/InternalJavadocPositiveCases.java

@@ -74,6 +74,7 @@ public String getUrlQuery(HttpRequest request) {
   }
 
   @Override
+  @SuppressWarnings("UnnecessaryDefaultInEnumSwitch")
   public String getNetworkProtocolName(HttpRequest request, @Nullable HttpResponse httpResponse) {
     return switch (request.getVersion()) {
       case HTTP_0_9, HTTP_1_0, HTTP_1_1, HTTP_2_0 -> "http";
@@ -82,6 +83,7 @@ public String getNetworkProtocolName(HttpRequest request, @Nullable HttpResponse
   }
 
   @Override
+  @SuppressWarnings("UnnecessaryDefaultInEnumSwitch")
   public String getNetworkProtocolVersion(
       HttpRequest request, @Nullable HttpResponse httpResponse) {
     return switch (request.getVersion()) {

dependencyManagement/build.gradle.kts

@@ -18,6 +18,7 @@ public Node newNode(Settings settings) {
         InternalSettingsPreparer.prepareEnvironment(settings, null),
         Collections.singleton(Netty4Plugin.class)) {
 
+      @SuppressWarnings({"UnusedMethod", "UnusedVariable"})
       protected void registerDerivedNodeNameWithLogger(String s) {}
     };
   }

instrumentation/activej-http-6.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/activejhttp/ActivejHttpServerAttributesGetter.java

@@ -65,10 +65,9 @@ public boolean equals(@Nullable Object obj) {
     if (obj == this) {
       return true;
     }
-    if (!(obj instanceof Customer)) {
+    if (!(obj instanceof Customer other)) {
       return false;
     }
-    Customer other = (Customer) obj;
     return Objects.equals(id, other.id)
         && Objects.equals(firstName, other.firstName)
         && Objects.equals(lastName, other.lastName);

instrumentation/elasticsearch/elasticsearch-transport-6.0/javaagent/src/elasticsearch6Test/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/transport/v6_0/Elasticsearch6NodeFactory.java

@@ -86,14 +86,14 @@ public String getNetworkProtocolVersion(Request request, @Nullable Response unus
   public InetSocketAddress getNetworkPeerInetSocketAddress(
       Request request, @Nullable Response unused) {
     SocketAddress address = request.getConnectionMetaData().getRemoteSocketAddress();
-    return address instanceof InetSocketAddress ? (InetSocketAddress) address : null;
+    return address instanceof InetSocketAddress inetSocketAddress ? inetSocketAddress : null;
   }
 
   @Nullable
   @Override
   public InetSocketAddress getNetworkLocalInetSocketAddress(
       Request request, @Nullable Response unused) {
     SocketAddress address = request.getConnectionMetaData().getLocalSocketAddress();
-    return address instanceof InetSocketAddress ? (InetSocketAddress) address : null;
+    return address instanceof InetSocketAddress inetSocketAddress ? inetSocketAddress : null;
   }
 }

instrumentation/hibernate/hibernate-6.0/spring-testing/src/test/java/spring/jpa/Customer.java

@@ -19,8 +19,7 @@ public final class ThreadGrouper {
   @Nullable
   public String groupedName(RecordedEvent ev) {
     Object thisField = ev.getValue("eventThread");
-    if (thisField instanceof RecordedThread) {
-      RecordedThread thread = (RecordedThread) thisField;
+    if (thisField instanceof RecordedThread thread) {
       return thread.getJavaName();
     }
     return null;