Remove integration test and associated CI/Maven scaffolding

The MockWebServer-based integration test added too much complexity
(reflection hacks, test cert setup, extra dependencies) for little
value over the existing unit tests. This reverts commits a481565,
50405c5, 63376d6, and 341b367:

- Delete HttpRateLimitRetryIntegrationIT
- Remove mockwebserver and okhttp-tls test dependencies
- Remove maven-failsafe-plugin configuration
- Revert CI from mvn verify back to mvn test
- Restore testDefaultMaxBackoffIsUsedWhenNotSpecified unit test

Author: jeffreyparker

.github/workflows/java-ci.yml

@@ -33,7 +33,7 @@ jobs:
           --file duo-client/pom.xml
       - name: Test with Maven
         run: >
-          mvn verify
+          mvn test
           --batch-mode
           -file duo-client/pom.xml
       - name: Lint with checkstyle

duo-client/pom.xml

@@ -65,18 +65,6 @@
       <version>3.12.4</version>
       <scope>test</scope>
     </dependency>
-    <dependency>
-      <groupId>com.squareup.okhttp3</groupId>
-      <artifactId>mockwebserver</artifactId>
-      <version>4.12.0</version>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>com.squareup.okhttp3</groupId>
-      <artifactId>okhttp-tls</artifactId>
-      <version>4.12.0</version>
-      <scope>test</scope>
-    </dependency>
   </dependencies>
 
   <build>
@@ -135,19 +123,6 @@
           <parallel>methods</parallel>
         </configuration>
       </plugin>
-      <plugin>
-        <groupId>org.apache.maven.plugins</groupId>
-        <artifactId>maven-failsafe-plugin</artifactId>
-        <version>3.2.5</version>
-        <executions>
-          <execution>
-            <goals>
-              <goal>integration-test</goal>
-              <goal>verify</goal>
-            </goals>
-          </execution>
-        </executions>
-      </plugin>
       <plugin>
         <groupId>org.cyclonedx</groupId>
         <artifactId>cyclonedx-maven-plugin</artifactId>

duo-client/src/test/java/com/duosecurity/client/HttpRateLimitRetryIntegrationIT.java

@@ -209,6 +209,17 @@ public Call answer(InvocationOnMock invocationOnMock) throws Throwable {
         assertEquals(4000L + RANDOM_INT, (long) sleepTimes.get(2));
     }
 
+    @Test
+    public void testDefaultMaxBackoffIsUsedWhenNotSpecified() throws Exception {
+        Http defaultHttp = new Http.HttpBuilder("GET", "example.test", "/foo/bar").build();
+
+        Field maxBackoffField = Http.class.getDeclaredField("maxBackoffMs");
+        maxBackoffField.setAccessible(true);
+        long actualMaxBackoff = (long) maxBackoffField.get(defaultHttp);
+
+        assertEquals(Http.MAX_BACKOFF_MS, actualMaxBackoff);
+    }
+
     @Test(expected = IllegalArgumentException.class)
     public void testMaxBackoffNegativeThrows() {
         new Http.HttpBuilder("GET", "example.test", "/foo/bar")