pr review

Author: zeitlinger

javaagent-tooling/src/test/java/io/opentelemetry/javaagent/tooling/bytebuddy/matcher/ExtendsClassMatcherTest.java

@@ -8,7 +8,6 @@
 import static io.opentelemetry.javaagent.extension.matcher.AgentElementMatchers.extendsClass;
 import static net.bytebuddy.matcher.ElementMatchers.named;
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.assertj.core.api.Assertions.assertThatCode;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
@@ -64,7 +63,7 @@ void testTraversalExceptions() {
     boolean result = extendsClass(named(Object.class.getName())).matches(type);
 
     assertThat(result).isFalse();
-    assertThatCode(() -> extendsClass(named(Object.class.getName())).matches(type))
-        .doesNotThrowAnyException();
+    // should not throw
+    extendsClass(named(Object.class.getName())).matches(type);
   }
 }

javaagent-tooling/src/test/java/io/opentelemetry/javaagent/tooling/bytebuddy/matcher/ThrowOnFirstElement.java

@@ -7,7 +7,7 @@
 
 import java.util.Iterator;
 
-public class ThrowOnFirstElement implements Iterator<Object> {
+class ThrowOnFirstElement implements Iterator<Object> {
   private int current = 0;
 
   @Override