fix(deps): update junit5 monorepo to v5.11.0 (main) (minor) (#12010)

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

benchmark-overhead/build.gradle.kts

@@ -16,7 +16,7 @@ repositories {
 }
 
 dependencies {
-  implementation(enforcedPlatform("org.junit:junit-bom:5.10.3"))
+  implementation(enforcedPlatform("org.junit:junit-bom:5.11.0"))
 
   testImplementation("org.testcontainers:testcontainers:1.20.1")
   testImplementation("org.testcontainers:postgresql:1.20.1")

conventions/build.gradle.kts

@@ -71,7 +71,7 @@ dependencies {
   implementation("net.ltgt.gradle:gradle-nullaway-plugin:2.0.0")
   implementation("me.champeau.gradle:japicmp-gradle-plugin:0.4.3")
 
-  testImplementation(enforcedPlatform("org.junit:junit-bom:5.10.3"))
+  testImplementation(enforcedPlatform("org.junit:junit-bom:5.11.0"))
   testImplementation("org.junit.jupiter:junit-jupiter-api")
   testRuntimeOnly("org.junit.jupiter:junit-jupiter-engine")
   testImplementation("org.assertj:assertj-core:3.26.3")

dependencyManagement/build.gradle.kts

@@ -33,7 +33,7 @@ val DEPENDENCY_BOMS = listOf(
   "org.apache.groovy:groovy-bom:${groovyVersion}",
   "io.opentelemetry:opentelemetry-bom:${otelSdkVersion}",
   "io.opentelemetry:opentelemetry-bom-alpha:${otelSdkAlphaVersion}",
-  "org.junit:junit-bom:5.10.3",
+  "org.junit:junit-bom:5.11.0",
   "org.testcontainers:testcontainers-bom:1.20.1",
   "org.spockframework:spock-bom:2.4-M4-groovy-4.0"
 )

examples/distro/build.gradle

@@ -34,7 +34,7 @@ subprojects {
       opentelemetryJavaagentAlpha: "2.7.0-alpha-SNAPSHOT",
 
       autoservice                : "1.1.1",
-      junit                      : "5.10.3"
+      junit                      : "5.11.0"
     ]
 
     deps = [

examples/extension/build.gradle

@@ -29,7 +29,7 @@ ext {
     opentelemetryJavaagent     : "2.7.0-SNAPSHOT",
     opentelemetryJavaagentAlpha: "2.7.0-alpha-SNAPSHOT",
 
-    junit                      : "5.10.3"
+    junit                      : "5.11.0"
   ]
 
   deps = [

gradle-plugins/build.gradle.kts

@@ -43,7 +43,7 @@ dependencies {
 
   testImplementation("org.assertj:assertj-core:3.26.3")
 
-  testImplementation(enforcedPlatform("org.junit:junit-bom:5.10.3"))
+  testImplementation(enforcedPlatform("org.junit:junit-bom:5.11.0"))
   testImplementation("org.junit.jupiter:junit-jupiter-api")
   testImplementation("org.junit.jupiter:junit-jupiter-params")
   testRuntimeOnly("org.junit.jupiter:junit-jupiter-engine")

instrumentation/cassandra/cassandra-4.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/cassandra/v4_0/CassandraTest.java

@@ -3,12 +3,14 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
+package io.opentelemetry.javaagent.instrumentation.cassandra.v4_0;
+
 import io.opentelemetry.cassandra.v4.common.AbstractCassandraTest;
 import io.opentelemetry.instrumentation.testing.junit.AgentInstrumentationExtension;
 import io.opentelemetry.instrumentation.testing.junit.InstrumentationExtension;
 import org.junit.jupiter.api.extension.RegisterExtension;
 
-public class CassandraTest extends AbstractCassandraTest {
+class CassandraTest extends AbstractCassandraTest {
 
   @RegisterExtension
   static final InstrumentationExtension testing = AgentInstrumentationExtension.create();

instrumentation/quarkus-resteasy-reactive/quarkus2-testing/build.gradle.kts

@@ -10,6 +10,8 @@ otelJava {
 
 dependencies {
   implementation(enforcedPlatform("io.quarkus.platform:quarkus-bom:2.16.7.Final"))
+  // fails with junit 5.11.+
+  implementation(enforcedPlatform("org.junit:junit-bom:5.10.3"))
   implementation("io.quarkus:quarkus-resteasy-reactive")
 
   testInstrumentation(project(":instrumentation:netty:netty-4.1:javaagent"))

instrumentation/quarkus-resteasy-reactive/quarkus3-testing/build.gradle.kts

@@ -16,6 +16,8 @@ if (findProperty("testLatestDeps") as Boolean) {
 
 dependencies {
   implementation(enforcedPlatform("io.quarkus.platform:quarkus-bom:$quarkusVersion"))
+  // fails with junit 5.11.+
+  implementation(enforcedPlatform("org.junit:junit-bom:5.10.3"))
   implementation("io.quarkus:quarkus-resteasy-reactive")
 
   testInstrumentation(project(":instrumentation:netty:netty-4.1:javaagent"))

instrumentation/spring/spring-data/spring-data-1.8/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/spring/data/v1_8/SpringJpaTest.java

@@ -3,6 +3,9 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
+package io.opentelemetry.javaagent.instrumentation.spring.data.v1_8;
+
+import io.opentelemetry.javaagent.instrumentation.spring.data.AbstractSpringJpaTest;
 import java.util.List;
 import java.util.Optional;
 import org.springframework.context.annotation.AnnotationConfigApplicationContext;
@@ -13,27 +16,27 @@ import spring.jpa.JpaPersistenceConfig;
 public class SpringJpaTest extends AbstractSpringJpaTest<JpaCustomer, JpaCustomerRepository> {
 
   @Override
-  JpaCustomer newCustomer(String firstName, String lastName) {
+  protected JpaCustomer newCustomer(String firstName, String lastName) {
     return new JpaCustomer(firstName, lastName);
   }
 
   @Override
-  Long id(JpaCustomer customer) {
+  protected Long id(JpaCustomer customer) {
     return customer.getId();
   }
 
   @Override
-  void setFirstName(JpaCustomer customer, String firstName) {
+  protected void setFirstName(JpaCustomer customer, String firstName) {
     customer.setFirstName(firstName);
   }
 
   @Override
-  Class<JpaCustomerRepository> repositoryClass() {
+  protected Class<JpaCustomerRepository> repositoryClass() {
     return JpaCustomerRepository.class;
   }
 
   @Override
-  JpaCustomerRepository repository() {
+  protected JpaCustomerRepository repository() {
     AnnotationConfigApplicationContext context =
         new AnnotationConfigApplicationContext(JpaPersistenceConfig.class);
     JpaCustomerRepository repo = context.getBean(JpaCustomerRepository.class);
@@ -45,17 +48,18 @@ public class SpringJpaTest extends AbstractSpringJpaTest<JpaCustomer, JpaCustome
   }
 
   @Override
-  List<JpaCustomer> findByLastName(JpaCustomerRepository repository, String lastName) {
+  protected List<JpaCustomer> findByLastName(JpaCustomerRepository repository, String lastName) {
     return repository.findByLastName(lastName);
   }
 
   @Override
-  List<JpaCustomer> findSpecialCustomers(JpaCustomerRepository repository) {
+  protected List<JpaCustomer> findSpecialCustomers(JpaCustomerRepository repository) {
     return repository.findSpecialCustomers();
   }
 
   @Override
-  Optional<JpaCustomer> findOneByLastName(JpaCustomerRepository repository, String lastName) {
+  protected Optional<JpaCustomer> findOneByLastName(
+      JpaCustomerRepository repository, String lastName) {
     return repository.findOneByLastName(lastName);
   }
 }

instrumentation/spring/spring-data/spring-data-3.0/testing/src/test/java/io/opentelemetry/javaagent/instrumentation/spring/data/v3_0/SpringJpaTest.java

@@ -3,6 +3,9 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
+package io.opentelemetry.javaagent.instrumentation.spring.data.v3_0;
+
+import io.opentelemetry.javaagent.instrumentation.spring.data.AbstractSpringJpaTest;
 import java.util.List;
 import java.util.Optional;
 import org.springframework.context.annotation.AnnotationConfigApplicationContext;
@@ -13,27 +16,27 @@ import spring.jpa.JpaPersistenceConfig;
 public class SpringJpaTest extends AbstractSpringJpaTest<JpaCustomer, JpaCustomerRepository> {
 
   @Override
-  JpaCustomer newCustomer(String firstName, String lastName) {
+  protected JpaCustomer newCustomer(String firstName, String lastName) {
     return new JpaCustomer(firstName, lastName);
   }
 
   @Override
-  Long id(JpaCustomer customer) {
+  protected Long id(JpaCustomer customer) {
     return customer.getId();
   }
 
   @Override
-  void setFirstName(JpaCustomer customer, String firstName) {
+  protected void setFirstName(JpaCustomer customer, String firstName) {
     customer.setFirstName(firstName);
   }
 
   @Override
-  Class<JpaCustomerRepository> repositoryClass() {
+  protected Class<JpaCustomerRepository> repositoryClass() {
     return JpaCustomerRepository.class;
   }
 
   @Override
-  JpaCustomerRepository repository() {
+  protected JpaCustomerRepository repository() {
     AnnotationConfigApplicationContext context =
         new AnnotationConfigApplicationContext(JpaPersistenceConfig.class);
     JpaCustomerRepository repo = context.getBean(JpaCustomerRepository.class);
@@ -45,17 +48,18 @@ public class SpringJpaTest extends AbstractSpringJpaTest<JpaCustomer, JpaCustome
   }
 
   @Override
-  List<JpaCustomer> findByLastName(JpaCustomerRepository repository, String lastName) {
+  protected List<JpaCustomer> findByLastName(JpaCustomerRepository repository, String lastName) {
     return repository.findByLastName(lastName);
   }
 
   @Override
-  List<JpaCustomer> findSpecialCustomers(JpaCustomerRepository repository) {
+  protected List<JpaCustomer> findSpecialCustomers(JpaCustomerRepository repository) {
     return repository.findSpecialCustomers();
   }
 
   @Override
-  Optional<JpaCustomer> findOneByLastName(JpaCustomerRepository repository, String lastName) {
+  protected Optional<JpaCustomer> findOneByLastName(
+      JpaCustomerRepository repository, String lastName) {
     return repository.findOneByLastName(lastName);
   }
 }

instrumentation/spring/spring-data/spring-data-common/testing/src/main/java/io/opentelemetry/javaagent/instrumentation/spring/data/AbstractSpringJpaTest.java

@@ -3,6 +3,8 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
+package io.opentelemetry.javaagent.instrumentation.spring.data;
+
 import static io.opentelemetry.sdk.testing.assertj.OpenTelemetryAssertions.equalTo;
 import static io.opentelemetry.sdk.testing.assertj.OpenTelemetryAssertions.satisfies;
 import static org.assertj.core.api.Assertions.catchThrowableOfType;
@@ -33,23 +35,23 @@ public abstract class AbstractSpringJpaTest<
   @RegisterExtension
   private static final InstrumentationExtension testing = AgentInstrumentationExtension.create();
 
-  abstract ENTITY newCustomer(String firstName, String lastName);
+  protected abstract ENTITY newCustomer(String firstName, String lastName);
 
-  abstract Long id(ENTITY customer);
+  protected abstract Long id(ENTITY customer);
 
-  abstract void setFirstName(ENTITY customer, String firstName);
+  protected abstract void setFirstName(ENTITY customer, String firstName);
 
-  abstract Class<REPOSITORY> repositoryClass();
+  protected abstract Class<REPOSITORY> repositoryClass();
 
-  abstract REPOSITORY repository();
+  protected abstract REPOSITORY repository();
 
-  abstract List<ENTITY> findByLastName(REPOSITORY repository, String lastName);
+  protected abstract List<ENTITY> findByLastName(REPOSITORY repository, String lastName);
 
-  abstract List<ENTITY> findSpecialCustomers(REPOSITORY repository);
+  protected abstract List<ENTITY> findSpecialCustomers(REPOSITORY repository);
 
-  abstract Optional<ENTITY> findOneByLastName(REPOSITORY repository, String lastName);
+  protected abstract Optional<ENTITY> findOneByLastName(REPOSITORY repository, String lastName);
 
-  void clearData() {
+  protected void clearData() {
     testing.clearData();
   }
 

instrumentation/tapestry-5.4/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/tapestry/TapestryTest.java

@@ -3,6 +3,8 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
+package io.opentelemetry.javaagent.instrumentation.tapestry;
+
 import static org.assertj.core.api.Assertions.assertThat;
 
 import io.opentelemetry.api.trace.SpanKind;