minor changes

Author: SylvainJuge

javaagent-tooling/src/testExceptionHandler/java/io/opentelemetry/javaagent/tooling/bytebuddy/BadAdvice.java

@@ -12,7 +12,7 @@
 public class BadAdvice {
 
   @Advice.OnMethodExit(suppress = Throwable.class)
-  public static boolean throwAnException(@Advice.Argument(0) AtomicBoolean isInstrumented) {
+  public static void throwAnException(@Advice.Argument(0) AtomicBoolean isInstrumented) {
     // mark that the advice has been executed
     isInstrumented.set(true);
     throw new IllegalStateException("Test Exception");

javaagent-tooling/src/testExceptionHandler/java/io/opentelemetry/javaagent/tooling/bytebuddy/ExceptionHandlerTest.java

@@ -116,6 +116,7 @@ void exceptionOnNondelegatingClassloader() throws Exception {
     AtomicBoolean instrumented = new AtomicBoolean(false);
     someClazz.getMethod("isInstrumented", AtomicBoolean.class).invoke(null, instrumented);
     assertThat(testHandler.getRecords()).hasSize(initLogEvents);
+    assertThat(instrumented.get()).describedAs("method should have been instrumented").isTrue();
   }
 
   @Test