Nested Dependency issues

Remove nested depends On connectivity

Change-Id: Ic140e88fac74c31200cdd662dcbf6e296183ade3
Issue-ID: SDC-1315
Signed-off-by: siddharth0905 <siddharth.singh4@amdocs.com>

3 files changed, 16 insertions, 18 deletions

diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationEntityType.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationEntityType.java
index d4b85605d2..790be7cd54 100644
--- a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationEntityType.java
+++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationEntityType.java
@@ -20,12 +20,14 @@ import static org.openecomp.sdc.translator.services.heattotosca.ConsolidationDat
 import static org.openecomp.sdc.translator.services.heattotosca.ConsolidationDataUtil.isPortResource;
 import static org.openecomp.sdc.translator.services.heattotosca.ConsolidationDataUtil.isVolumeResource;
 
-import java.util.*;
-import java.util.stream.Collectors;
-
 import org.openecomp.sdc.heat.datatypes.model.Resource;
 import org.openecomp.sdc.translator.datatypes.heattotosca.TranslationContext;
 
+import java.util.Collections;
+import java.util.EnumSet;
+import java.util.Set;
+import java.util.stream.Collectors;
+
 /**
  * The enum Entity type.
  */
@@ -76,6 +78,8 @@ public enum ConsolidationEntityType {
             consolidationEntityType = ConsolidationEntityType.COMPUTE;
         } else if (isPortResource(resource)) {
             consolidationEntityType = ConsolidationEntityType.PORT;
+        } else if (HeatToToscaUtil.isSubInterfaceResource(resource, context)) {
+            consolidationEntityType = ConsolidationEntityType.SUB_INTERFACE;
         } else if (isVolumeResource(resource)) {
             consolidationEntityType = ConsolidationEntityType.VOLUME;
         } else if (HeatToToscaUtil.isNestedResource(resource)) {
@@ -87,8 +91,4 @@ public enum ConsolidationEntityType {
         }
         return consolidationEntityType;
     }
-
-    public static boolean isEntityTypeNested(ConsolidationEntityType entityType) {
-        return NESTED == entityType || VFC_NESTED == entityType;
-    }
 }
\ No newline at end of file
diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationTypesConnectivity.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationTypesConnectivity.java
index 7b258d651f..628f204da5 100644
--- a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationTypesConnectivity.java
+++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationTypesConnectivity.java
@@ -27,13 +27,7 @@ public class ConsolidationTypesConnectivity {
     }
 
     static boolean isDependsOnRelationshipValid(ConsolidationEntityType source, ConsolidationEntityType target) {
-        return !eitherSourceOrTargetIsNested(source, target)
-                       && (!entityToEntitiesWithoutRelationship.containsKey(source)
-                                   || !entityToEntitiesWithoutRelationship.containsEntry(source, target));
-    }
-
-    private static boolean eitherSourceOrTargetIsNested(ConsolidationEntityType source,
-                                                               ConsolidationEntityType target) {
-        return ConsolidationEntityType.isEntityTypeNested(source) || ConsolidationEntityType.isEntityTypeNested(target);
+        return !entityToEntitiesWithoutRelationship.containsKey(source)
+                                   || !entityToEntitiesWithoutRelationship.containsEntry(source, target);
     }
 }
diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/impl/resourcetranslation/ResourceTranslationBase.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/impl/resourcetranslation/ResourceTranslationBase.java
index 92ce4c19ae..f93d3714ca 100644
--- a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/impl/resourcetranslation/ResourceTranslationBase.java
+++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/impl/resourcetranslation/ResourceTranslationBase.java
@@ -276,9 +276,7 @@ public abstract class ResourceTranslationBase {
       Resource targetResource = HeatToToscaUtil
           .getResource(heatOrchestrationTemplate, dependsOnResourceId,
               translateTo.getHeatFileName());
-      if (HeatToToscaUtil
-          .isValidDependsOnCandidate(sourceResource, targetResource,
-              ConsolidationEntityType.OTHER, translateTo.getContext())) {
+      if (isValidDependency(sourceResource, targetResource, translateTo)) {
         requirementAssignment.setNode(resourceTranslatedId.get());
         requirementAssignment.setCapability(ToscaCapabilityType.NATIVE_NODE);
         requirementAssignment.setRelationship(ToscaRelationshipType.NATIVE_DEPENDS_ON);
@@ -293,6 +291,12 @@ public abstract class ResourceTranslationBase {
     }
   }
 
+  private boolean isValidDependency(Resource sourceResource, Resource targetResource, TranslateTo translateTo) {
+      return !(HeatToToscaUtil.isNestedResource(sourceResource) || HeatToToscaUtil.isNestedResource(targetResource))
+              && HeatToToscaUtil.isValidDependsOnCandidate(sourceResource, targetResource,
+              ConsolidationEntityType.OTHER, translateTo.getContext());
+  }
+
   Optional<List<Map.Entry<String, Resource>>> getResourceByTranslatedResourceId(
       String heatFileName,
       HeatOrchestrationTemplate heatOrchestrationTemplate,