Type list of floats not generated correctly in tosca

[sdc.git] / catalog-be / src / main / java / org / openecomp / sdc / be / tosca / PropertyConvertor.java

diff --git a/catalog-be/src/main/java/org/openecomp/sdc/be/tosca/PropertyConvertor.java b/catalog-be/src/main/java/org/openecomp/sdc/be/tosca/PropertyConvertor.java
index cecb131..d1cd495 100644 (file)
--- a/catalog-be/src/main/java/org/openecomp/sdc/be/tosca/PropertyConvertor.java
+++ b/catalog-be/src/main/java/org/openecomp/sdc/be/tosca/PropertyConvertor.java
@@ -133,7 +133,7 @@ public class PropertyConvertor {
 
         if (CollectionUtils.isNotEmpty(property.getConstraints())) {
             try {
-                prop.setConstraints(convertConstraints(property.getConstraints(), property.getType()));
+                prop.setConstraints(convertConstraints(property.getConstraints(), property.getType(), property.getSchemaType()));
             } catch (ConstraintValueDoNotMatchPropertyTypeException e) {
                 log.error(e.getMessage());
             }
@@ -141,14 +141,14 @@ public class PropertyConvertor {
         return prop;
     }
 
-    private List<ToscaPropertyConstraint> convertConstraints(List<PropertyConstraint> constraints, String propertyType)
+    private List<ToscaPropertyConstraint> convertConstraints(List<PropertyConstraint> constraints, String propertyType, String schemaType)
         throws ConstraintValueDoNotMatchPropertyTypeException {
         List<ToscaPropertyConstraint> convertedConstraints = new ArrayList<>();
         for (PropertyConstraint constraint : constraints) {
             if (constraint instanceof EqualConstraint) {
                 EqualConstraint equalConstraint = ((EqualConstraint) constraint);
 
-                if (doesPropertyTypeNeedConverted(propertyType)) {
+                if (doesTypeNeedConvertingToIntOrFloat(propertyType)) {
                     equalConstraint.changeConstraintValueTypeTo(propertyType);
                 }
 
@@ -158,7 +158,7 @@ public class PropertyConvertor {
             if (constraint instanceof GreaterThanConstraint) {
                 GreaterThanConstraint greaterThanConstraint = ((GreaterThanConstraint) constraint);
 
-                if (doesPropertyTypeNeedConverted(propertyType)) {
+                if (doesTypeNeedConvertingToIntOrFloat(propertyType)) {
                     greaterThanConstraint.changeConstraintValueTypeTo(propertyType);
                 }
 
@@ -168,7 +168,7 @@ public class PropertyConvertor {
             if (constraint instanceof GreaterOrEqualConstraint) {
                 GreaterOrEqualConstraint greaterOrEqualConstraint = ((GreaterOrEqualConstraint) constraint);
 
-                if (doesPropertyTypeNeedConverted(propertyType)) {
+                if (doesTypeNeedConvertingToIntOrFloat(propertyType)) {
                     greaterOrEqualConstraint.changeConstraintValueTypeTo(propertyType);
                 }
 
@@ -178,7 +178,7 @@ public class PropertyConvertor {
             if (constraint instanceof LessThanConstraint) {
                 LessThanConstraint lessThanConstraint = ((LessThanConstraint) constraint);
 
-                if (doesPropertyTypeNeedConverted(propertyType)) {
+                if (doesTypeNeedConvertingToIntOrFloat(propertyType)) {
                     lessThanConstraint.changeConstraintValueTypeTo(propertyType);
                 }
 
@@ -188,7 +188,7 @@ public class PropertyConvertor {
             if (constraint instanceof LessOrEqualConstraint) {
                 LessOrEqualConstraint lessOrEqualConstraint = ((LessOrEqualConstraint) constraint);
 
-                if (doesPropertyTypeNeedConverted(propertyType)) {
+                if (doesTypeNeedConvertingToIntOrFloat(propertyType)) {
                     lessOrEqualConstraint.changeConstraintValueTypeTo(propertyType);
                 }
 
@@ -198,19 +198,18 @@ public class PropertyConvertor {
             if (constraint instanceof InRangeConstraint) {
                 InRangeConstraint inRangeConstraint = (InRangeConstraint) constraint;
 
-                if (doesPropertyTypeNeedConverted(propertyType)) {
+                if (doesTypeNeedConvertingToIntOrFloat(propertyType)) {
                     inRangeConstraint.changeConstraintValueTypeTo(propertyType);
                 }
 
-                List<Object> range = new ArrayList<>();
-                range.add(inRangeConstraint.getMin());
-                range.add(inRangeConstraint.getMax());
-                convertedConstraints.add(new ToscaPropertyConstraintInRange(range));
+                convertedConstraints.add(new ToscaPropertyConstraintInRange(inRangeConstraint.getInRange()));
             }
             if (constraint instanceof ValidValuesConstraint) {
                 ValidValuesConstraint validValues = ((ValidValuesConstraint) constraint);
 
-                if (propertyType.equals(ToscaType.INTEGER.toString()) || propertyType.equals(ToscaType.FLOAT.toString())) {
+                if (isTypeMapOrList(propertyType) && doesTypeNeedConvertingToIntOrFloat(schemaType)) {
+                    validValues.changeConstraintValueTypeTo(schemaType);
+                } else if (doesTypeNeedConvertingToIntOrFloat(propertyType)) {
                     validValues.changeConstraintValueTypeTo(propertyType);
                 }
 
@@ -233,8 +232,12 @@ public class PropertyConvertor {
         return convertedConstraints;
     }
 
-    private boolean doesPropertyTypeNeedConverted(String propertyType) {
-        return propertyType.equals(ToscaType.INTEGER.getType()) || propertyType.equals(ToscaType.FLOAT.getType());
+    private boolean doesTypeNeedConvertingToIntOrFloat(String propertyType) {
+        return ToscaType.INTEGER.getType().equals(propertyType) || ToscaType.FLOAT.getType().equals(propertyType);
+    }
+
+    private boolean isTypeMapOrList (String type) {
+        return ToscaType.MAP.getType().equals(type) || ToscaType.LIST.getType().equals(type);
     }
 
     public Object convertToToscaObject(PropertyDataDefinition property, String value, Map<String, DataTypeDefinition> dataTypes,