diff --git a/pom.xml b/pom.xml index d471d67..cc30146 100644 --- a/pom.xml +++ b/pom.xml @@ -4,7 +4,7 @@ de.stklcode.jvault connector - 0.7.1 + 0.7.2-SNAPSHOT jar @@ -29,7 +29,7 @@ org.apache.maven.plugins maven-compiler-plugin - 3.7.0 + 3.8.0 1.8 1.8 @@ -41,17 +41,17 @@ org.apache.maven.plugins maven-clean-plugin - 3.0.0 + 3.1.0 org.apache.maven.plugins maven-resources-plugin - 3.0.2 + 3.1.0 org.apache.maven.plugins maven-jar-plugin - 3.0.2 + 3.1.0 @@ -68,14 +68,7 @@ org.apache.maven.plugins maven-surefire-plugin - 2.19.1 - - - org.junit.platform - junit-platform-surefire-provider - 1.1.0 - - + 2.22.0 @@ -106,13 +99,13 @@ org.junit.jupiter junit-jupiter-engine - 5.1.0 + 5.3.0 test org.junit.jupiter junit-jupiter-migrationsupport - 5.1.0 + 5.3.0 org.hamcrest @@ -123,19 +116,19 @@ com.github.stefanbirkner system-rules - 1.17.1 + 1.17.2 test org.mockito mockito-core - 2.15.0 + 2.21.0 test org.mockito mockito-inline - 2.15.0 + 2.21.0 test diff --git a/src/test/java/de/stklcode/jvault/connector/HTTPVaultConnectorOfflineTest.java b/src/test/java/de/stklcode/jvault/connector/HTTPVaultConnectorOfflineTest.java index 1d09826..adc81c7 100644 --- a/src/test/java/de/stklcode/jvault/connector/HTTPVaultConnectorOfflineTest.java +++ b/src/test/java/de/stklcode/jvault/connector/HTTPVaultConnectorOfflineTest.java @@ -60,7 +60,6 @@ import static org.mockito.Mockito.*; public class HTTPVaultConnectorOfflineTest { private static final String INVALID_URL = "foo:/\\1nv4l1d_UrL"; - private static HttpClientBuilder httpMockBuilder = mock(HttpClientBuilder.class); private static CloseableHttpClient httpMock = mock(CloseableHttpClient.class); private CloseableHttpResponse responseMock = mock(CloseableHttpResponse.class); @@ -76,7 +75,7 @@ public class HTTPVaultConnectorOfflineTest { * @return Mocked HTTP client builder. */ public static HttpClientBuilder create() { - return httpMockBuilder; + return new MockedHttpClientBuilder(); } @BeforeEach @@ -88,14 +87,8 @@ public class HTTPVaultConnectorOfflineTest { .make() .load(HttpClientBuilder.class.getClassLoader(), ClassReloadingStrategy.fromInstalledAgent()); - // Ignore SSL context settings. - when(httpMockBuilder.setSSLContext(null)).thenReturn(httpMockBuilder); - // Re-initialize HTTP mock to ensure fresh (empty) results. httpMock = mock(CloseableHttpClient.class); - - // Mock actual client creation. - when(httpMockBuilder.build()).thenReturn(httpMock); } /** @@ -466,7 +459,7 @@ public class HTTPVaultConnectorOfflineTest { private void setPrivate(Object target, String fieldName, Object value) { try { Field field = target.getClass().getDeclaredField(fieldName); - boolean accessible =field.isAccessible(); + boolean accessible = field.isAccessible(); field.setAccessible(true); field.set(target, value); field.setAccessible(accessible); @@ -480,4 +473,15 @@ public class HTTPVaultConnectorOfflineTest { when(responseMock.getStatusLine()).thenReturn(new BasicStatusLine(new ProtocolVersion("HTTP", 1, 1), status, "")); when(responseMock.getEntity()).thenReturn(new StringEntity(body, type)); } + + /** + * Mocked {@link HttpClientBuilder} that always returns the mocked client. + */ + private static class MockedHttpClientBuilder extends HttpClientBuilder { + @Override + public CloseableHttpClient build() { + return httpMock; + } + } + }