Fix Stream 71/68371/2
authorSmokowski, Steve (ss835w) <ss835w@us.att.com>
Fri, 21 Sep 2018 09:20:24 +0000 (05:20 -0400)
committerMarcus G K Williams <marcus.williams@intel.com>
Tue, 25 Sep 2018 00:47:59 +0000 (17:47 -0700)
Fix Bean scanning so it picks up the resttemplate
removed unnecessary RestTemplate Bean configurations

Change-Id: I094bec8e9513a2e02ebf00215291136ed6d54a18
Issue-ID: SO-1080
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>
mso-api-handlers/mso-requests-db-repositories/src/test/java/org/onap/so/TestApplication.java
mso-api-handlers/mso-requests-db/src/main/java/org/onap/so/db/request/client/RequestsDbClient.java

index 5b6fbc8..9bb5cab 100644 (file)
@@ -27,7 +27,7 @@ import org.springframework.context.annotation.Profile;
 import org.springframework.http.client.HttpComponentsClientHttpRequestFactory;
 import org.springframework.web.client.RestTemplate;
 
-@SpringBootApplication(scanBasePackages = { "org.onap.so.db" })
+@SpringBootApplication(scanBasePackages = { "org.onap.so" })
 @Profile("test")
 public class TestApplication {
        public static void main(String... args) {
@@ -35,8 +35,4 @@ public class TestApplication {
                System.getProperties().setProperty("mso.db", "MARIADB");
                System.getProperties().setProperty("server.name", "Springboot");
        }
-       @Bean
-       public RestTemplate restTemplate() {
-               return new RestTemplate( new HttpComponentsClientHttpRequestFactory());
-       }
 }
index e68bdb3..07d185b 100644 (file)
 
 package org.onap.so.db.request.client;
 
+import java.net.URI;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+
+import javax.annotation.PostConstruct;
+import javax.ws.rs.core.MediaType;
+import javax.ws.rs.core.UriBuilder;
+
 import org.apache.http.HttpStatus;
 import org.onap.so.db.request.beans.ArchivedInfraRequests;
 import org.onap.so.db.request.beans.InfraActiveRequests;
@@ -41,6 +52,7 @@ import org.springframework.core.ParameterizedTypeReference;
 import org.springframework.http.HttpEntity;
 import org.springframework.http.HttpHeaders;
 import org.springframework.http.HttpMethod;
+import org.springframework.http.ResponseEntity;
 import org.springframework.http.client.BufferingClientHttpRequestFactory;
 import org.springframework.http.client.ClientHttpRequestFactory;
 import org.springframework.http.client.HttpComponentsClientHttpRequestFactory;
@@ -50,16 +62,6 @@ import org.springframework.web.client.RestTemplate;
 import uk.co.blackpepper.bowman.ClientFactory;
 import uk.co.blackpepper.bowman.Configuration;
 
-import javax.annotation.PostConstruct;
-import javax.ws.rs.core.MediaType;
-import javax.ws.rs.core.UriBuilder;
-import java.net.URI;
-import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
-import java.util.Iterator;
-
 @Component("RequestsDbClient")
 @Primary
 public class RequestsDbClient {
@@ -188,7 +190,12 @@ public class RequestsDbClient {
                URI uri = getUri(checkInstanceNameDuplicate);
                HttpEntity<InstanceNameDuplicateCheckRequest> entity = new HttpEntity<>(new InstanceNameDuplicateCheckRequest(instanceIdMap, instanceName, requestScope), headers);
                try{
-                       return restTemplate.exchange(uri, HttpMethod.POST, entity, InfraActiveRequests.class).getBody();
+                       ResponseEntity<InfraActiveRequests> response = restTemplate.exchange(uri, HttpMethod.POST, entity, InfraActiveRequests.class);
+                       if(response != null && response.hasBody()) {
+                               return restTemplate.exchange(uri, HttpMethod.POST, entity, InfraActiveRequests.class).getBody();
+                       } else {
+                               return null;
+                       }
                }catch(HttpClientErrorException e){
                        if(HttpStatus.SC_NOT_FOUND == e.getStatusCode().value()){
                                return null;