remove ability to disable cert chain validation

Remove the ability to disbale certificate chain validation

Issue-ID: AAI-1908

Change-Id: I5803cec657594bfbc814be1e0122a67206d28cc4
Signed-off-by: renealr <reneal.rogers@amdocs.com>

2 files changed, 7 insertions, 21 deletions

diff --git a/src/test/java/org/onap/aai/restclient/client/RestfulClientTest.java b/src/test/java/org/onap/aai/restclient/client/RestfulClientTest.java
index e2a728d..5eb7f1f 100644
--- a/src/test/java/org/onap/aai/restclient/client/RestfulClientTest.java
+++ b/src/test/java/org/onap/aai/restclient/client/RestfulClientTest.java
@@ -342,6 +342,7 @@ public class RestfulClientTest {
     public void testGetClient() throws Exception {
         RestClientBuilder restClientBuilder= new RestClientBuilder();
         restClientBuilder.setAuthenticationMode(RestAuthenticationMode.SSL_BASIC);
+        restClientBuilder.setTruststoreFilename("truststore");
         assertTrue(restClientBuilder.getClient() instanceof Client);
     }
 
diff --git a/src/test/java/org/onap/aai/restclient/rest/RestClientBuilderTest.java b/src/test/java/org/onap/aai/restclient/rest/RestClientBuilderTest.java
index 5e7d8c1..7155f9a 100644
--- a/src/test/java/org/onap/aai/restclient/rest/RestClientBuilderTest.java
+++ b/src/test/java/org/onap/aai/restclient/rest/RestClientBuilderTest.java
@@ -141,6 +141,7 @@ public class RestClientBuilderTest {
     restClientBuilder.setReadTimeoutInMs(54321);

     restClientBuilder.setBasicAuthUsername("username");

     restClientBuilder.setBasicAuthPassword("password");

+    restClientBuilder.setTruststoreFilename("truststore");

     

     Client client = restClientBuilder.getClient();

    

@@ -155,7 +156,7 @@ public class RestClientBuilderTest {
     

   }

 

-  @Test

+  @Test (expected=IllegalArgumentException.class)

   public void validateSslCertClient_noHostOrCertChainValidation() throws Exception {

     

     RestClientBuilder restClientBuilder = new RestClientBuilder();

@@ -166,18 +167,10 @@ public class RestClientBuilderTest {
     restClientBuilder.setValidateServerCertChain(false);

     restClientBuilder.setValidateServerHostname(false);

     

-    Client client = restClientBuilder.getClient();

-   

-    Object sslPropertiesObj = client.getProperties().get(HTTPSProperties.PROPERTY_HTTPS_PROPERTIES);

-    HTTPSProperties sslProps = null;

-    if ( sslPropertiesObj instanceof HTTPSProperties ) {

-      sslProps = (HTTPSProperties)sslPropertiesObj;

-      assertNotNull(sslProps.getHostnameVerifier());

-    } else {

-      fail("Unexpected value for https properties object");

-    }  }

+    Client client = restClientBuilder.getClient(); 

+  }

   

-  @Test

+  @Test (expected=IllegalArgumentException.class)

   public void validateSslCertClient_hostOnlyValidation() throws Exception {

     

     RestClientBuilder restClientBuilder = new RestClientBuilder();

@@ -190,15 +183,7 @@ public class RestClientBuilderTest {
     

     Client client = restClientBuilder.getClient();

    

-    Object sslPropertiesObj = client.getProperties().get(HTTPSProperties.PROPERTY_HTTPS_PROPERTIES);

-    HTTPSProperties sslProps = null;

-    if ( sslPropertiesObj instanceof HTTPSProperties ) {

-      sslProps = (HTTPSProperties)sslPropertiesObj;

-      assertNull(sslProps.getHostnameVerifier());

-    } else {

-      fail("Unexpected value for https properties object");

-    }

-   }

+  }

   

   @Test

   public void validateSslCertClient_certChainOnlyValidation() throws Exception {