Fix a race condition in the HttpClientModule tests. This seems to h…

…appen when an exception is raised over the SSL socket: the socket is then closed but the `mockWebServer` still tries to access it when `shutdown()` is called.

PiperOrigin-RevId: 587722530
Change-Id: I37ba7f9013e68f751e47a8520f23c2bddb0ebb04

common/src/test/java/com/google/tsunami/common/net/http/HttpClientModuleTest.java

@@ -119,7 +119,9 @@ public void setTrustAllCertificates_whenFalseAndCertIsInvalid_throws()
         SSLHandshakeException.class,
         () -> httpClient.send(get(mockWebServer.url("/")).withEmptyHeaders().build()));
 
-    mockWebServer.shutdown();
+    // Note: b/314642696 - After this point, the socket in mockWebServer was closed when the
+    //                     exception was raised. So working with the mockWebServer would be
+    //                     hazardous.
   }
 
   @Test
@@ -137,7 +139,9 @@ public void setTrustAllCertificates_whenBothCliAndConfigValuesAreSet_cliValueTak
         SSLHandshakeException.class,
         () -> httpClient.send(get(mockWebServer.url("/")).withEmptyHeaders().build()));
 
-    mockWebServer.shutdown();
+    // Note: b/314642696 - After this point, the socket in mockWebServer was closed when the
+    //                     exception was raised. So working with the mockWebServer would be
+    //                     hazardous.
   }
 
   @Test