Restrict latestDepTestLibrary overrides to main test/latestDepTest configurations

The previous PR broadened the resolutionStrategy filter to all 'test*' configurations

(case-insensitive contains) so that custom JvmTestSuite source sets get pinning. However

this also caused latestDepTestLibrary overrides (intended only for the main test/latestDepTest

suites) to be applied to those custom suites, breaking modules that declare a JvmTestSuite

with explicit older versions (e.g. play-mvc-2.4 play24Test, kubernetes-client-7.0 version20Test).

Split the filter: keep overrides scoped to test/latestDepTest, broaden only the pinning logic.

Author: trask

conventions/src/main/kotlin/io.opentelemetry.instrumentation.base.gradle.kts

@@ -173,22 +173,30 @@ configurations {
   if (otelProps.testLatestDeps) {
     // Only apply to test-related configurations, not build tool configurations like Zinc
     // (the Scala compiler). Overriding scala-library in Zinc's configuration breaks compilation.
-    // We match any configuration whose name contains "test" (case-insensitive) so that custom
-    // JvmTestSuite source sets (e.g. `version5TestRuntimeClasspath`) are also covered.
     configureEach {
-      if (isCanBeResolved && name.contains("test", ignoreCase = true)) {
-        resolutionStrategy.eachDependency {
+      if (!isCanBeResolved) return@configureEach
+      // latestDepTestLibrary overrides only apply to the main test/latestDepTest configurations.
+      // Custom JvmTestSuite source sets (e.g. `version20TestRuntimeClasspath`,
+      // `play24TestRuntimeClasspath`) declare their own explicit versions and must not be
+      // affected by overrides intended for the main test suite.
+      val applyOverrides = name.startsWith("test") || name.startsWith("latestDepTest")
+      // Pinning of `latest.release`/`+` versions is applied broadly (case-insensitive "test"
+      // match) so that custom JvmTestSuite source sets are also reproducible.
+      val applyPinning = pinLatestDeps && name.contains("test", ignoreCase = true)
+      if (!applyOverrides && !applyPinning) return@configureEach
+      resolutionStrategy.eachDependency {
+        if (applyOverrides) {
           // latestDepTestLibrary overrides take priority over pinned versions
           val override = latestDepTestLibraryOverrides["${requested.group}:${requested.name}"]
           if (override != null) {
             useVersion(override)
             return@eachDependency
           }
-          if (pinLatestDeps) {
-            val pinnedVersion = lookupPinnedVersion(requested.group, requested.name, requested.version)
-            if (pinnedVersion != null) {
-              useVersion(pinnedVersion)
-            }
+        }
+        if (applyPinning) {
+          val pinnedVersion = lookupPinnedVersion(requested.group, requested.name, requested.version)
+          if (pinnedVersion != null) {
+            useVersion(pinnedVersion)
           }
         }
       }