Fix ApacheHttpAsyncClientTest cleanup firing per @nested container

AutoCleanupExtension.deferAfterAll(...) registers callbacks on a static
extension whose afterAll() fires once per @nested class container, so
clients were closed after the first nested class finished, causing all
tests in subsequent nested classes to fail with CancellationException.

Use a plain @afterall method on the outer class instead — it runs only
once, after all nested classes complete.

Author: trask

instrumentation/apache-httpasyncclient-4.1/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/apachehttpasyncclient/v4_1/ApacheHttpAsyncClientTest.java

@@ -5,7 +5,6 @@
 
 package io.opentelemetry.javaagent.instrumentation.apachehttpasyncclient.v4_1;
 
-import io.opentelemetry.instrumentation.testing.internal.AutoCleanupExtension;
 import io.opentelemetry.instrumentation.testing.junit.InstrumentationExtension;
 import io.opentelemetry.instrumentation.testing.junit.http.AbstractHttpClientTest;
 import io.opentelemetry.instrumentation.testing.junit.http.HttpClientInstrumentationExtension;
@@ -24,6 +23,7 @@
 import org.apache.http.impl.nio.client.HttpAsyncClients;
 import org.apache.http.message.BasicHeader;
 import org.apache.http.nio.client.HttpAsyncClient;
+import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.TestInstance;
@@ -35,8 +35,6 @@ class ApacheHttpAsyncClientTest {
   @RegisterExtension
   static final InstrumentationExtension testing = HttpClientInstrumentationExtension.forAgent();
 
-  @RegisterExtension static final AutoCleanupExtension cleanup = AutoCleanupExtension.create();
-
   private final RequestConfig requestConfig =
       RequestConfig.custom()
           .setConnectTimeout((int) AbstractHttpClientTest.CONNECTION_TIMEOUT.toMillis())
@@ -54,12 +52,16 @@ class ApacheHttpAsyncClientTest {
 
   @BeforeAll
   void setUp() {
-    cleanup.deferAfterAll(client);
-    cleanup.deferAfterAll(clientWithReadTimeout);
     client.start();
     clientWithReadTimeout.start();
   }
 
+  @AfterAll
+  void tearDown() throws IOException {
+    client.close();
+    clientWithReadTimeout.close();
+  }
+
   HttpAsyncClient getClient(URI uri) {
     if (uri.toString().contains("/read-timeout")) {
       return clientWithReadTimeout;