diff options
author | wsliwka <wojciech.sliwka@nokia.com> | 2019-09-26 10:55:00 +0200 |
---|---|---|
committer | wsliwka <wojciech.sliwka@nokia.com> | 2019-09-26 12:04:33 +0200 |
commit | 398e1db85eca9645b65422eb7c9d5f6956244276 (patch) | |
tree | 5f4872769b7d59b937c88ecd3ab929b9b7b29a2b /vid-app-common | |
parent | 16b52c71ed49b9cd6ffa2cd1383d1b6c7724e17f (diff) |
Try to fix failing tests
Keystore instantiation is failing on jenkins.
Add mocked keystore
Issue-ID: VID-653
Signed-off-by: wsliwka <wojciech.sliwka@nokia.com>
Change-Id: I428db5b14d90c63de6b11ad5d366e839efbf33ed
Diffstat (limited to 'vid-app-common')
-rw-r--r-- | vid-app-common/src/test/java/org/onap/vid/client/SyncRestClientForHttpsServerTest.java | 44 |
1 files changed, 36 insertions, 8 deletions
diff --git a/vid-app-common/src/test/java/org/onap/vid/client/SyncRestClientForHttpsServerTest.java b/vid-app-common/src/test/java/org/onap/vid/client/SyncRestClientForHttpsServerTest.java index f4692c564..645b5eac0 100644 --- a/vid-app-common/src/test/java/org/onap/vid/client/SyncRestClientForHttpsServerTest.java +++ b/vid-app-common/src/test/java/org/onap/vid/client/SyncRestClientForHttpsServerTest.java @@ -8,9 +8,9 @@ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. * You may obtain a copy of the License at - * + * * http://www.apache.org/licenses/LICENSE-2.0 - * + * * Unless required by applicable law or agreed to in writing, software * distributed under the License is distributed on an "AS IS" BASIS, * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. @@ -29,6 +29,7 @@ import static com.xebialabs.restito.semantics.Action.stringContent; import static org.apache.http.client.config.RequestConfig.custom; import static org.mockito.ArgumentMatchers.any; import static org.mockito.ArgumentMatchers.eq; +import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.verify; import static org.testng.Assert.assertEquals; @@ -41,18 +42,25 @@ import com.xebialabs.restito.semantics.Condition; import com.xebialabs.restito.server.StubServer; import io.joshworks.restclient.http.HttpResponse; import io.joshworks.restclient.http.JsonNode; + +import java.io.IOException; import java.security.GeneralSecurityException; +import java.security.KeyStore; +import java.security.KeyStoreException; import java.util.Collections; +import java.util.Enumeration; import javax.net.ssl.SSLContext; import org.apache.http.config.Registry; import org.apache.http.config.RegistryBuilder; import org.apache.http.conn.socket.ConnectionSocketFactory; +import org.apache.http.conn.ssl.NoopHostnameVerifier; import org.apache.http.conn.ssl.SSLConnectionSocketFactory; -import org.apache.http.conn.ssl.SSLContextBuilder; + import org.apache.http.conn.ssl.TrustSelfSignedStrategy; import org.apache.http.impl.client.CloseableHttpClient; import org.apache.http.impl.client.HttpClientBuilder; import org.apache.http.impl.conn.PoolingHttpClientConnectionManager; +import org.apache.http.ssl.SSLContextBuilder; import org.glassfish.grizzly.http.Method; import org.onap.vid.utils.Logging; import org.springframework.http.HttpMethod; @@ -71,7 +79,7 @@ public class SyncRestClientForHttpsServerTest { private Logging mockLoggingService; @BeforeMethod - public void setUp() throws GeneralSecurityException { + public void setUp() throws GeneralSecurityException{ stubServer = new StubServer(); stubServer.secured().run(); mockLoggingService = mock(Logging.class); @@ -132,12 +140,16 @@ public class SyncRestClientForHttpsServerTest { .then(ok(), jsonContent(), contentType("application/json")); } - private CloseableHttpClient createNaiveHttpClient() throws GeneralSecurityException { - final SSLContext context = new SSLContextBuilder().loadTrustMaterial(null, new TrustSelfSignedStrategy()) - .build(); + private CloseableHttpClient createNaiveHttpClient() throws GeneralSecurityException{ + KeyStore mock = getMockedKeyStore(); + + final SSLContext context = SSLContextBuilder + .create() + .loadTrustMaterial(mock, new TrustSelfSignedStrategy()) + .build(); final SSLConnectionSocketFactory socketFactory = new SSLConnectionSocketFactory(context, SUPPORTED_PROTOCOLS, - null, SSLConnectionSocketFactory.ALLOW_ALL_HOSTNAME_VERIFIER); + null, NoopHostnameVerifier.INSTANCE); Registry<ConnectionSocketFactory> registry = RegistryBuilder.<ConnectionSocketFactory>create() .register("https", socketFactory) .build(); @@ -148,4 +160,20 @@ public class SyncRestClientForHttpsServerTest { .setSSLSocketFactory(socketFactory).build(); } + private KeyStore getMockedKeyStore() throws KeyStoreException { + KeyStore mock = mock(KeyStore.class); + doReturn(new Enumeration<String>() { + + @Override + public boolean hasMoreElements() { + return false; + } + + @Override + public String nextElement() { + return null; + } + }).when(mock).aliases(); + return mock; + } } |