pr review

zeitlinger · zeitlinger · commit afefdf739a82 · 2025-10-10T13:12:21.000+02:00
diff --git a/muzzle/src/test/java/io/opentelemetry/javaagent/tooling/muzzle/ReferenceMatcherTest.java b/muzzle/src/test/java/io/opentelemetry/javaagent/tooling/muzzle/ReferenceMatcherTest.java
@@ -86,12 +86,9 @@ void matchSafeClasspaths() {
   }
 
   @Test
-  void matchingDoesNotHoldStrongReferenceToClassloaders() {
-    try {
-      assertThat(MuzzleWeakReferenceTestUtil.classLoaderRefIsGarbageCollected()).isTrue();
-    } catch (InterruptedException | TimeoutException e) {
-      throw new RuntimeException(e);
-    }
+  void matchingDoesNotHoldStrongReferenceToClassloaders()
+      throws InterruptedException, TimeoutException {
+    assertThat(MuzzleWeakReferenceTestUtil.classLoaderRefIsGarbageCollected()).isTrue();
   }
 
   private static class CountingClassLoader extends URLClassLoader {

Original file line number	Diff line number	Diff line change
`@@ -86,12 +86,9 @@ void matchSafeClasspaths() {`
`86`	`86`	`}`
`87`	`87`
`88`	`88`	`@Test`
`89`		`- void matchingDoesNotHoldStrongReferenceToClassloaders() {`
`90`		`- try {`
`91`		`- assertThat(MuzzleWeakReferenceTestUtil.classLoaderRefIsGarbageCollected()).isTrue();`
`92`		`- } catch (InterruptedException \| TimeoutException e) {`
`93`		`- throw new RuntimeException(e);`
`94`		`- }`
	`89`	`+ void matchingDoesNotHoldStrongReferenceToClassloaders()`
	`90`	`+ throws InterruptedException, TimeoutException {`
	`91`	`+ assertThat(MuzzleWeakReferenceTestUtil.classLoaderRefIsGarbageCollected()).isTrue();`
`95`	`92`	`}`
`96`	`93`
`97`	`94`	`private static class CountingClassLoader extends URLClassLoader {`