Skip to content

Commit c939cfb

Browse files
committed
Remove redundant reflection-based default maxBackoffMs test
The testRepeatRetryAfterRateLimit test already covers default backoff behavior (7 requests = 32s max) through observable retry count.
1 parent 5cf667b commit c939cfb

1 file changed

Lines changed: 0 additions & 11 deletions

File tree

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

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -209,17 +209,6 @@ public Call answer(InvocationOnMock invocationOnMock) throws Throwable {
209209
assertEquals(4000L + RANDOM_INT, (long) sleepTimes.get(2));
210210
}
211211

212-
@Test
213-
public void testDefaultMaxBackoffIsUsedWhenNotSpecified() throws Exception {
214-
Http defaultHttp = new Http.HttpBuilder("GET", "example.test", "/foo/bar").build();
215-
216-
Field maxBackoffField = Http.class.getDeclaredField("maxBackoffMs");
217-
maxBackoffField.setAccessible(true);
218-
long actualMaxBackoff = (long) maxBackoffField.get(defaultHttp);
219-
220-
assertEquals(Http.MAX_BACKOFF_MS, actualMaxBackoff);
221-
}
222-
223212
@Test(expected = IllegalArgumentException.class)
224213
public void testMaxBackoffNegativeThrows() {
225214
new Http.HttpBuilder("GET", "example.test", "/foo/bar")

0 commit comments

Comments
 (0)