diff --git a/httpclient5-cache/src/test/java/org/apache/hc/client5/http/impl/cache/ConsumableInputStream.java b/httpclient5-cache/src/test/java/org/apache/hc/client5/http/impl/cache/ConsumableInputStream.java index 1cd8c34902..e7969f2e5d 100644 --- a/httpclient5-cache/src/test/java/org/apache/hc/client5/http/impl/cache/ConsumableInputStream.java +++ b/httpclient5-cache/src/test/java/org/apache/hc/client5/http/impl/cache/ConsumableInputStream.java @@ -27,7 +27,6 @@ package org.apache.hc.client5.http.impl.cache; import java.io.ByteArrayInputStream; -import java.io.IOException; import java.io.InputStream; import org.apache.hc.core5.io.Closer; diff --git a/httpclient5-cache/src/test/java/org/apache/hc/client5/http/impl/cache/TestRFC5861Compliance.java b/httpclient5-cache/src/test/java/org/apache/hc/client5/http/impl/cache/TestRFC5861Compliance.java index 1d43834cab..cb5c34835c 100644 --- a/httpclient5-cache/src/test/java/org/apache/hc/client5/http/impl/cache/TestRFC5861Compliance.java +++ b/httpclient5-cache/src/test/java/org/apache/hc/client5/http/impl/cache/TestRFC5861Compliance.java @@ -28,6 +28,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNotEquals; import static org.junit.jupiter.api.Assertions.assertTrue; import java.io.ByteArrayInputStream; @@ -170,7 +171,7 @@ void testStaleIfErrorInResponseYieldsToMustRevalidate() final ClassicHttpResponse result = execute(req2); - assertTrue(HttpStatus.SC_OK != result.getCode()); + assertNotEquals(HttpStatus.SC_OK, result.getCode()); } @Test @@ -193,7 +194,7 @@ void testStaleIfErrorInResponseYieldsToProxyRevalidateForSharedCache() final ClassicHttpResponse result = execute(req2); - assertTrue(HttpStatus.SC_OK != result.getCode()); + assertNotEquals(HttpStatus.SC_OK, result.getCode()); } @Test @@ -216,7 +217,7 @@ void testStaleIfErrorInResponseYieldsToExplicitFreshnessRequest() final ClassicHttpResponse result = execute(req2); - assertTrue(HttpStatus.SC_OK != result.getCode()); + assertNotEquals(HttpStatus.SC_OK, result.getCode()); } @Test diff --git a/httpclient5/src/test/java/org/apache/hc/client5/http/TestHttpRoute.java b/httpclient5/src/test/java/org/apache/hc/client5/http/TestHttpRoute.java index c1ba0b5ebf..6614f4236a 100644 --- a/httpclient5/src/test/java/org/apache/hc/client5/http/TestHttpRoute.java +++ b/httpclient5/src/test/java/org/apache/hc/client5/http/TestHttpRoute.java @@ -359,7 +359,7 @@ void testHopping() { final HttpHost[] proxies3 = new HttpHost[]{ PROXY3, PROXY1, PROXY2 }; final HttpRoute route3 = new HttpRoute(TARGET1, LOCAL42, proxies3, false, TunnelType.PLAIN, LayerType.LAYERED); - Assertions.assertEquals(route3.getHopCount(), 4, "C: hop count"); + Assertions.assertEquals(4, route3.getHopCount(), "C: hop count"); Assertions.assertEquals(PROXY3 , route3.getHopTarget(0), "C: hop 0"); Assertions.assertEquals(PROXY1 , route3.getHopTarget(1), "C: hop 1"); Assertions.assertEquals(PROXY2 , route3.getHopTarget(2), "C: hop 2"); diff --git a/httpclient5/src/test/java/org/apache/hc/client5/http/classic/methods/TestHttpRequestBase.java b/httpclient5/src/test/java/org/apache/hc/client5/http/classic/methods/TestHttpRequestBase.java index 7103294ae6..7854d8acf2 100644 --- a/httpclient5/src/test/java/org/apache/hc/client5/http/classic/methods/TestHttpRequestBase.java +++ b/httpclient5/src/test/java/org/apache/hc/client5/http/classic/methods/TestHttpRequestBase.java @@ -176,7 +176,7 @@ void testOptionMethodGetAllowedMethods() { final Set methods = httpOptions.getAllowedMethods(response); assertAll("Must all pass", () -> assertFalse(methods.isEmpty()), - () -> assertEquals(methods.size(), 3), + () -> assertEquals(3, methods.size()), () -> assertTrue(methods.containsAll(Stream.of("HEAD", "DELETE", "GET") .collect(Collectors.toCollection(HashSet::new)))) ); diff --git a/httpclient5/src/test/java/org/apache/hc/client5/http/impl/classic/TestRedirectExec.java b/httpclient5/src/test/java/org/apache/hc/client5/http/impl/classic/TestRedirectExec.java index 4a21390a11..2bbedebe6b 100644 --- a/httpclient5/src/test/java/org/apache/hc/client5/http/impl/classic/TestRedirectExec.java +++ b/httpclient5/src/test/java/org/apache/hc/client5/http/impl/classic/TestRedirectExec.java @@ -404,9 +404,9 @@ void testPutSeeOtherRedirect() throws Exception { final ClassicHttpRequest request2 = allValues.get(1); final ClassicHttpRequest request3 = allValues.get(2); Assertions.assertSame(request, request1); - Assertions.assertEquals(request1.getMethod(), "PUT"); - Assertions.assertEquals(request2.getMethod(), "GET"); - Assertions.assertEquals(request3.getMethod(), "GET"); + Assertions.assertEquals("PUT", request1.getMethod()); + Assertions.assertEquals("GET", request2.getMethod()); + Assertions.assertEquals("GET", request3.getMethod()); } private static class HttpRequestMatcher implements ArgumentMatcher {