Fixed sonar issue in policy-api 20/133220/1
authorSuresh Charan <suresh.charan@bell.ca>
Thu, 9 Feb 2023 17:17:13 +0000 (12:17 -0500)
committerSuresh Charan <suresh.charan@bell.ca>
Thu, 9 Feb 2023 17:22:19 +0000 (12:22 -0500)
Fixed cognitive complexity reported by sonar.

Issue-ID: POLICY-4536
Change-Id: Ic3b5b7d79165afed15a1593e03b8374fb2122e12
Signed-off-by: Suresh Charan <suresh.charan@bell.ca>
main/src/main/java/org/onap/policy/api/main/startstop/ApiDatabaseInitializer.java

index d9f4f77..55f49b1 100644 (file)
@@ -120,15 +120,7 @@ public class ApiDatabaseInitializer {
         var multiVersionTemplates = new ArrayList<ToscaServiceTemplate>();
 
         for (String entity : entities) {
-            var entityAsStringYaml = ResourceUtils.getResourceAsString(entity);
-            if (entityAsStringYaml == null) {
-                throw new PolicyApiException("Preloaded entity cannot be found " + entity);
-            }
-
-            ToscaServiceTemplate singleEntity = coder.decode(entityAsStringYaml, ToscaServiceTemplate.class);
-            if (singleEntity == null) {
-                throw new PolicyApiException("Error deserializing entity from file: " + entity);
-            }
+            ToscaServiceTemplate singleEntity = deserializeServiceTemplate(entity);
 
             if (isMultiVersion(serviceTemplate.getPolicyTypes(), singleEntity.getPolicyTypes())) {
                 // if this entity introduces a new policy version of an existing policy type,
@@ -173,6 +165,19 @@ public class ApiDatabaseInitializer {
         return createdServiceTemplate;
     }
 
+    private ToscaServiceTemplate deserializeServiceTemplate(String entity) throws PolicyApiException, CoderException {
+        var entityAsStringYaml = ResourceUtils.getResourceAsString(entity);
+        if (entityAsStringYaml == null) {
+            throw new PolicyApiException("Preloaded entity cannot be found " + entity);
+        }
+
+        ToscaServiceTemplate singleEntity = coder.decode(entityAsStringYaml, ToscaServiceTemplate.class);
+        if (singleEntity == null) {
+            throw new PolicyApiException("Error deserializing entity from file: " + entity);
+        }
+        return singleEntity;
+    }
+
     // This method is templated, so it can be used with other derivations of ToscaEntity in the future,
     // if multi-version are desired.