fixup! STF-322: Add tests for transport-failure retry

Author: oschwald

src/test/java/com/maxmind/geoip2/WebServiceClientTest.java

@@ -15,11 +15,13 @@
 import static org.junit.jupiter.api.Assertions.assertNull;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.fail;
 
 import com.github.tomakehurst.wiremock.http.Fault;
 import com.github.tomakehurst.wiremock.junit5.WireMockExtension;
 import com.github.tomakehurst.wiremock.junit5.WireMockTest;
 import com.github.tomakehurst.wiremock.stubbing.Scenario;
+import com.github.tomakehurst.wiremock.verification.LoggedRequest;
 import com.maxmind.geoip2.exception.AddressNotFoundException;
 import com.maxmind.geoip2.exception.AuthenticationException;
 import com.maxmind.geoip2.exception.GeoIp2Exception;
@@ -665,7 +667,19 @@ public void testMaxRetriesZeroDisablesRetry() {
         assertThrows(Exception.class,
             () -> client.insights(InetAddress.getByName("1.2.3.4")));
 
-        wireMock.verify(1, getRequestedFor(urlEqualTo(url)));
+        // DIAGNOSTIC: dump full request details on mismatch so we can
+        // distinguish JDK-internal retry from h2c upgrade fallback. To be
+        // reverted once the windows-21 wire-count behavior is understood.
+        List<LoggedRequest> requests = wireMock.findAll(getRequestedFor(urlEqualTo(url)));
+        if (requests.size() != 1) {
+            StringBuilder sb = new StringBuilder("Expected 1 request, got ")
+                .append(requests.size()).append(":\n");
+            for (int i = 0; i < requests.size(); i++) {
+                sb.append("--- Request ").append(i + 1).append(" ---\n")
+                    .append(requests.get(i).toString()).append("\n");
+            }
+            fail(sb.toString());
+        }
     }
 
     @Test