Do not double close in AbstractHttpServerTransportTests.testForceClosesOpenChannels (#96635)

Also reduce all grace periods below 100ms
Fixes: #96617

Author: stu-elastic

server/src/test/java/org/elasticsearch/http/AbstractHttpServerTransportTests.java

@@ -1054,7 +1054,7 @@ public void dispatchBadRequest(RestChannel channel, ThreadContext threadContext,
     }
 
     public void testStopClosesChannelAfterRequest() {
-        try (TestHttpServerTransport transport = new TestHttpServerTransport(gracePeriod(1_000))) {
+        try (TestHttpServerTransport transport = new TestHttpServerTransport(gracePeriod(100))) {
             transport.bindServer();
 
             TestHttpChannel httpChannel = new TestHttpChannel();
@@ -1101,9 +1101,8 @@ public void testStopClosesChannelAfterRequest() {
         }
     }
 
-    @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/96632")
     public void testForceClosesOpenChannels() {
-        try (TestHttpServerTransport transport = new TestHttpServerTransport(gracePeriod(1_000))) {
+        try (TestHttpServerTransport transport = new TestHttpServerTransport(gracePeriod(100))) {
             transport.bindServer();
 
             TestHttpChannel httpChannel = new TestHttpChannel(true);
@@ -1292,7 +1291,9 @@ public void close() {
                 }
                 if (waitForever) {
                     try {
-                        closeLatch.await(1, TimeUnit.SECONDS);
+                        if (closeLatch.await(1, TimeUnit.SECONDS) == false) {
+                            return;
+                        }
                     } catch (InterruptedException ie) {
                         throw new RuntimeException(ie);
                     }