Fixed Instance Properties Name

Fixed issue where the system put append _Instance1
so the end user can put their own custom name

Issue-ID: POLICY-3978
Change-Id: I04500148aeee49c64f278ce00daa55d14f964b90
Signed-off-by: brunomilitzer <bruno.militzer@est.tech>

2 files changed, 12 insertions, 30 deletions

diff --git a/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/commissioning/CommissioningProvider.java b/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/commissioning/CommissioningProvider.java
index dfb9d151b..9e07cde51 100644
--- a/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/commissioning/CommissioningProvider.java
+++ b/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/commissioning/CommissioningProvider.java
@@ -63,7 +63,7 @@ import org.springframework.transaction.annotation.Transactional;
 @Transactional
 public class CommissioningProvider {
     public static final String AUTOMATION_COMPOSITION_NODE_TYPE = "org.onap.policy.clamp.acm.AutomationComposition";
-    private static final String INSTANCE_TEXT = "_Instance";
+    private static final String HYPHEN = "-";
 
     private final ServiceTemplateProvider serviceTemplateProvider;
     private final AutomationCompositionProvider acProvider;
@@ -356,7 +356,7 @@ public class CommissioningProvider {
             Map<String, ToscaNodeTemplate> toscaNodeTemplates = new HashMap<>();
 
             serviceTemplate.getToscaTopologyTemplate().getNodeTemplates().forEach((key, nodeTemplate) -> {
-                if (!nodeTemplate.getName().contains(INSTANCE_TEXT)) {
+                if (!nodeTemplate.getName().contains(HYPHEN)) {
                     toscaNodeTemplates.put(key, nodeTemplate);
                 }
             });
@@ -377,7 +377,7 @@ public class CommissioningProvider {
      */
     private boolean verifyIfInstancePropertiesExists() {
         return acProvider.getAllNodeTemplates().stream()
-            .anyMatch(nodeTemplate -> nodeTemplate.getKey().getName().contains(INSTANCE_TEXT));
+            .anyMatch(nodeTemplate -> nodeTemplate.getKey().getName().contains(HYPHEN));
 
     }
 }
diff --git a/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/instantiation/AutomationCompositionInstantiationProvider.java b/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/instantiation/AutomationCompositionInstantiationProvider.java
index 39d84026b..40d4c5a85 100644
--- a/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/instantiation/AutomationCompositionInstantiationProvider.java
+++ b/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/instantiation/AutomationCompositionInstantiationProvider.java
@@ -81,7 +81,7 @@ public class AutomationCompositionInstantiationProvider {
     private static final String PARTICIPANT_TYPE_PROPERTY_KEY = "participantType";
     private static final String AC_ELEMENT_NAME = "name";
     private static final String AC_ELEMENT_VERSION = "version";
-    private static final String INSTANCE_TEXT = "_Instance";
+    private static final String HYPHEN = "-";
 
     private static final Gson GSON = new Gson();
 
@@ -101,7 +101,7 @@ public class AutomationCompositionInstantiationProvider {
     public InstancePropertiesResponse createInstanceProperties(ToscaServiceTemplate serviceTemplate)
         throws PfModelException {
 
-        String instanceName = generateSequentialInstanceName();
+        String instanceName = serviceTemplate.getName();
         AutomationComposition automationComposition = new AutomationComposition();
         Map<UUID, AutomationCompositionElement> automationCompositionElements = new HashMap<>();
 
@@ -114,9 +114,9 @@ public class AutomationCompositionInstantiationProvider {
 
         nodeTemplates.forEach((key, template) -> {
             ToscaNodeTemplate newNodeTemplate = new ToscaNodeTemplate();
-            String name = key + instanceName;
+            String name = key + "-" + instanceName;
             String version = template.getVersion();
-            String description = template.getDescription() + instanceName;
+            String description = template.getDescription() + " " + instanceName;
             newNodeTemplate.setName(name);
             newNodeTemplate.setVersion(version);
             newNodeTemplate.setDescription(description);
@@ -528,9 +528,9 @@ public class AutomationCompositionInstantiationProvider {
             automationCompositionElements.put(automationCompositionElement.getId(), automationCompositionElement);
         }
 
-        automationComposition.setName("PMSH" + instanceName);
+        automationComposition.setName(instanceName);
         automationComposition.setVersion(template.getVersion());
-        automationComposition.setDescription("PMSH automation composition " + instanceName);
+        automationComposition.setDescription("Automation composition " + instanceName);
         automationComposition.setState(AutomationCompositionState.UNINITIALISED);
         automationComposition.setOrderedState(AutomationCompositionOrderedState.UNINITIALISED);
     }
@@ -550,10 +550,10 @@ public class AutomationCompositionInstantiationProvider {
 
         return toscaDefinitionsNames.stream().reduce("", (s1, s2) -> {
 
-            if (s2.contains(INSTANCE_TEXT)) {
-                String[] instances = s2.split(INSTANCE_TEXT);
+            if (s2.contains(HYPHEN)) {
+                String[] instances = s2.split(HYPHEN);
 
-                return INSTANCE_TEXT + instances[1];
+                return HYPHEN + instances[1];
             }
 
             return s1;
@@ -561,24 +561,6 @@ public class AutomationCompositionInstantiationProvider {
     }
 
     /**
-     * Generates Instance Name in sequential order and return it to append to the Node Template Name.
-     *
-     * @return instanceName
-     */
-    private String generateSequentialInstanceName() {
-        List<ToscaNodeTemplate> nodeTemplates = automationCompositionProvider.getAllNodeTemplates();
-
-        int instanceNumber = nodeTemplates.stream().map(ToscaNodeTemplate::getName)
-            .filter(name -> name.contains(INSTANCE_TEXT)).map(n -> {
-                String[] defNameArr = n.split(INSTANCE_TEXT);
-
-                return Integer.parseInt(defNameArr[1]);
-            }).reduce(0, Math::max);
-
-        return INSTANCE_TEXT + (instanceNumber + 1);
-    }
-
-    /**
      * Retrieves automation composition Definition.
      *
      * @param template tosca node template