Refactor ResourceConfig class

[dcaegen2/platform.git] / mod / bpgenerator / src / main / java / org / onap / blueprintgenerator / models / blueprint / ResourceConfig.java

diff --git a/mod/bpgenerator/src/main/java/org/onap/blueprintgenerator/models/blueprint/ResourceConfig.java b/mod/bpgenerator/src/main/java/org/onap/blueprintgenerator/models/blueprint/ResourceConfig.java
index 6595a67..030374f 100644 (file)
--- a/mod/bpgenerator/src/main/java/org/onap/blueprintgenerator/models/blueprint/ResourceConfig.java
+++ b/mod/bpgenerator/src/main/java/org/onap/blueprintgenerator/models/blueprint/ResourceConfig.java
@@ -21,6 +21,8 @@
 
 package org.onap.blueprintgenerator.models.blueprint;
 
+import static org.onap.blueprintgenerator.common.blueprint.BlueprintHelper.createStringInput;
+
 import java.util.LinkedHashMap;
 import java.util.TreeMap;
 
@@ -32,92 +34,94 @@ import lombok.Setter;
 
 //TODO: Auto-generated Javadoc
 /* (non-Javadoc)
-* @see java.lang.Object#toString()
-*/
-@Getter @Setter
+ * @see java.lang.Object#toString()
+ */
+@Getter
+@Setter
 
 /* (non-Javadoc)
-* @see java.lang.Object#toString()
-*/
+ * @see java.lang.Object#toString()
+ */
 @Builder
 
 /**
-* Instantiates a new resource config obj.
-*/
+ * Instantiates a new resource config obj.
+ */
 @NoArgsConstructor
 
 /**
-* Instantiates a new resource config obj.
-*
-* @param limits the limits
-* @param requests the requests
-*/
+ * Instantiates a new resource config obj.
+ *
+ * @param limits the limits
+ * @param requests the requests
+ */
 @AllArgsConstructor
 
 public class ResourceConfig {
 
-       /** The limits. */
-       private TreeMap<String, GetInput> limits;
-
-       /** The requests. */
-       private TreeMap<String, GetInput> requests;
+    /**
+     * The limits.
+     */
+    private TreeMap<String, GetInput> limits;
 
+    /**
+     * The requests.
+     */
+    private TreeMap<String, GetInput> requests;
 
-       /**
-        * Creates the resource config.
-        *
-        * @param inps the inps
-        * @param name the name
-        * @return the tree map
-        */
-       public TreeMap<String, LinkedHashMap<String, Object>> createResourceConfig(TreeMap<String, LinkedHashMap<String, Object>> inps, String name){
 
-               LinkedHashMap<String, Object> mi = new LinkedHashMap<>();
-               mi.put("type", "string");
-               mi.put("default", "128Mi");
+    /**
+     * Creates the resource config.
+     *
+     * @param inps the inps
+     * @param name the name
+     * @return the tree map
+     */
+    public TreeMap<String, LinkedHashMap<String, Object>> createResourceConfig(
+        TreeMap<String, LinkedHashMap<String, Object>> inps, String name) {
 
-               LinkedHashMap<String, Object> m = new LinkedHashMap<>();
-               m.put("type", "string");
-               m.put("default", "250m");
+        LinkedHashMap<String, Object> memoryLimit = createStringInput("128Mi");
+        LinkedHashMap<String, Object> cpuLimit = createStringInput("250m");
 
+        String namePrefix = getNamePrefix(name);
 
-               if(!name.equals("")) {
-                       name = name + "_";
-               }
+        //set the limits
+        TreeMap<String, GetInput> limits = new TreeMap<>();
 
-               //set the limits
-               TreeMap<String, GetInput> lim = new TreeMap<>();
+        GetInput cpu = new GetInput();
+        cpu.setBpInputName(namePrefix + "cpu_limit");
+        limits.put("cpu", cpu);
 
-               GetInput cpu = new GetInput();
-               cpu.setBpInputName(name + "cpu_limit");
-               lim.put("cpu", cpu);
+        GetInput memL = new GetInput();
+        memL.setBpInputName(namePrefix + "memory_limit");
+        limits.put("memory", memL);
 
-               GetInput memL = new GetInput();
-               memL.setBpInputName(name + "memory_limit");
-               lim.put("memory", memL);
+        inps.put(namePrefix + "cpu_limit", cpuLimit);
+        inps.put(namePrefix + "memory_limit", memoryLimit);
 
-               inps.put(name + "cpu_limit", m);
-               inps.put(name + "memory_limit", mi);
+        this.setLimits(limits);
 
-               this.setLimits(lim);
+        //set the requests
+        TreeMap<String, GetInput> requests = new TreeMap<>();
 
-               //set the requests
-               TreeMap<String, GetInput> req = new TreeMap<>();
+        GetInput cpuR = new GetInput();
+        cpuR.setBpInputName(namePrefix + "cpu_request");
+        requests.put("cpu", cpuR);
 
-               GetInput cpuR = new GetInput();
-               cpuR.setBpInputName(name + "cpu_request");
-               req.put("cpu", cpuR);
+        GetInput memR = new GetInput();
+        memR.setBpInputName(namePrefix + "memory_request");
+        requests.put("memory", memR);
 
-               GetInput memR = new GetInput();
-               memR.setBpInputName(name + "memory_request");
-               req.put("memory", memR);
+        inps.put(namePrefix + "cpu_request", cpuLimit);
+        inps.put(namePrefix + "memory_request", memoryLimit);
 
-               inps.put(name + "cpu_request", m);
-               inps.put(name + "memory_request", mi);
+        this.setRequests(requests);
 
-               this.setRequests(req);
+        return inps;
+    }
 
-               return inps;
-       }
+    private String getNamePrefix(String name) {
+        return name.isEmpty() ? "" : name + "_";
+    }
 }