From e61d26cb2a74813b526e10864af4d73f04df2650 Mon Sep 17 00:00:00 2001 From: vasraz Date: Wed, 1 Feb 2023 17:56:35 +0000 Subject: Fix 'in_range constraints missing from TOSCA template'-bug Signed-off-by: Vasyl Razinkov Change-Id: Icec2183849a8852621ec98f3ca1d94ac7c4dae31 Issue-ID: SDC-4357 --- .../impl/CapabilityTypeOperationTest.java | 7 +- .../operations/impl/PropertyOperationTest.java | 1151 ++++++++++---------- .../impl/RelationshipTypeOperationTest.java | 11 +- .../sdc/be/model/tosca/ToscaTypeTest.java | 2 +- .../tosca/constraints/InRangeConstraintTest.java | 17 +- 5 files changed, 584 insertions(+), 604 deletions(-) (limited to 'catalog-model/src/test') diff --git a/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/CapabilityTypeOperationTest.java b/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/CapabilityTypeOperationTest.java index 39f9118337..12e5431c28 100644 --- a/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/CapabilityTypeOperationTest.java +++ b/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/CapabilityTypeOperationTest.java @@ -27,6 +27,7 @@ import static org.junit.Assert.assertTrue; import fj.data.Either; import java.util.ArrayList; +import java.util.Arrays; import java.util.HashMap; import java.util.Iterator; import java.util.List; @@ -523,13 +524,9 @@ public class CapabilityTypeOperationTest extends ModelTestBase { property2.setDescription("Number of (actual or virtual) CPUs associated with the Compute node."); property2.setType(ToscaType.INTEGER.name().toLowerCase()); List constraints3 = new ArrayList<>(); - List range = new ArrayList<>(); - range.add("1"); - range.add("4"); - InRangeConstraint propertyConstraint3 = new InRangeConstraint(range); + InRangeConstraint propertyConstraint3 = new InRangeConstraint(Arrays.asList("1", "4")); constraints3.add(propertyConstraint3); - // property2.setConstraints(constraints3); property2.setConstraints(constraints3); return property2; } diff --git a/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/PropertyOperationTest.java b/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/PropertyOperationTest.java index 3f78ff0b03..a1f863057e 100644 --- a/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/PropertyOperationTest.java +++ b/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/PropertyOperationTest.java @@ -7,9 +7,9 @@ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. * You may obtain a copy of the License at - * + * * http://www.apache.org/licenses/LICENSE-2.0 - * + * * Unless required by applicable law or agreed to in writing, software * distributed under the License is distributed on an "AS IS" BASIS, * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. @@ -168,11 +168,7 @@ public class PropertyOperationTest extends ModelTestBase { } private InRangeConstraint buildInRangeConstraint() { - List range = new ArrayList<>(); - range.add("23"); - range.add("67"); - InRangeConstraint inRangeConstraint = new InRangeConstraint(range); - return inRangeConstraint; + return new InRangeConstraint(Arrays.asList("23", "67")); } private ValidValuesConstraint buildValidValuesConstraint() { @@ -295,7 +291,7 @@ public class PropertyOperationTest extends ModelTestBase { List rules = new ArrayList<>(); PropertyRule propertyRule = new PropertyRule(); - String[] ruleArr = { "node1", ".+", "node3" }; + String[] ruleArr = {"node1", ".+", "node3"}; List rule1 = new ArrayList<>(Arrays.asList(ruleArr)); propertyRule.setRule(rule1); propertyRule.setValue("88"); @@ -338,13 +334,13 @@ public class PropertyOperationTest extends ModelTestBase { List rules = new ArrayList<>(); PropertyRule propertyRule1 = new PropertyRule(); - String[] ruleArr1 = { "node1", "node2", ".+" }; + String[] ruleArr1 = {"node1", "node2", ".+"}; List rule1 = new ArrayList<>(Arrays.asList(ruleArr1)); propertyRule1.setRule(rule1); propertyRule1.setValue("88"); PropertyRule propertyRule2 = new PropertyRule(); - String[] ruleArr2 = { "node1", "node2", "node3" }; + String[] ruleArr2 = {"node1", "node2", "node3"}; List rule2 = new ArrayList<>(Arrays.asList(ruleArr2)); propertyRule2.setRule(rule2); propertyRule2.setValue("99"); @@ -390,13 +386,13 @@ public class PropertyOperationTest extends ModelTestBase { List rules = new ArrayList<>(); PropertyRule propertyRule1 = new PropertyRule(); - String[] ruleArr1 = { "node1", "node2", ".+" }; + String[] ruleArr1 = {"node1", "node2", ".+"}; List rule1 = new ArrayList<>(Arrays.asList(ruleArr1)); propertyRule1.setRule(rule1); propertyRule1.setValue("88"); PropertyRule propertyRule2 = new PropertyRule(); - String[] ruleArr2 = { "node1", "node2", "node3" }; + String[] ruleArr2 = {"node1", "node2", "node3"}; List rule2 = new ArrayList<>(Arrays.asList(ruleArr2)); propertyRule2.setRule(rule2); propertyRule2.setValue("99"); @@ -413,7 +409,7 @@ public class PropertyOperationTest extends ModelTestBase { List rules3 = new ArrayList<>(); PropertyRule propertyRule3 = new PropertyRule(); - String[] ruleArr3 = { "node2", "node3" }; + String[] ruleArr3 = {"node2", "node3"}; List rule3 = new ArrayList<>(Arrays.asList(ruleArr3)); propertyRule3.setRule(rule3); propertyRule3.setValue("77"); @@ -452,13 +448,13 @@ public class PropertyOperationTest extends ModelTestBase { List rules = new ArrayList<>(); PropertyRule propertyRule1 = new PropertyRule(); - String[] ruleArr1 = { "node1", "node2", ".+" }; + String[] ruleArr1 = {"node1", "node2", ".+"}; List rule1 = new ArrayList<>(Arrays.asList(ruleArr1)); propertyRule1.setRule(rule1); propertyRule1.setValue("88"); PropertyRule propertyRule2 = new PropertyRule(); - String[] ruleArr2 = { "node1", "node2", "node3" }; + String[] ruleArr2 = {"node1", "node2", "node3"}; List rule2 = new ArrayList<>(Arrays.asList(ruleArr2)); propertyRule2.setRule(rule2); propertyRule2.setValue("99"); @@ -475,7 +471,7 @@ public class PropertyOperationTest extends ModelTestBase { List rules3 = new ArrayList<>(); PropertyRule propertyRule3 = new PropertyRule(); - String[] ruleArr3 = { "node2", "node333" }; + String[] ruleArr3 = {"node2", "node333"}; List rule3 = new ArrayList<>(Arrays.asList(ruleArr3)); propertyRule3.setRule(rule3); propertyRule3.setValue("77"); @@ -489,7 +485,7 @@ public class PropertyOperationTest extends ModelTestBase { assertEquals("check value", propertyRule2.getValue(), instanceProperty.getValue()); assertEquals("check default value", "vv1", instanceProperty.getDefaultValue()); - } + } private PropertyOperation createTestSubject() { final var propertyOperation = new PropertyOperation(new HealingJanusGraphGenericDao(new JanusGraphClient()), null); @@ -497,577 +493,578 @@ public class PropertyOperationTest extends ModelTestBase { return propertyOperation; } - @Test - public void testConvertPropertyDataToPropertyDefinition() throws Exception { - PropertyOperation testSubject; - PropertyData propertyDataResult = new PropertyData(); - String propertyName = ""; - String resourceId = ""; - PropertyDefinition result; - - // default test - testSubject = createTestSubject(); - result = testSubject.convertPropertyDataToPropertyDefinition(propertyDataResult, propertyName, resourceId); - } - - @Test - public void testAddProperty() throws Exception { - PropertyOperation testSubject; - String propertyName = ""; - PropertyDefinition propertyDefinition = new PropertyDefinition(); - String resourceId = ""; - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.addProperty(propertyName, propertyDefinition, resourceId); - } - - - @Test - public void testValidateAndUpdateProperty() throws Exception { - PropertyOperation testSubject; - IComplexDefaultValue propertyDefinition = new PropertyDefinition(); - Map dataTypes = new HashMap<>(); - dataTypes.put("", new DataTypeDefinition()); - StorageOperationStatus result; - - // default test - testSubject = createTestSubject(); - result = testSubject.validateAndUpdateProperty(propertyDefinition, dataTypes); - } - - - @Test - public void testAddPropertyToGraph() throws Exception { - PropertyOperation testSubject; - String propertyName = ""; - PropertyDefinition propertyDefinition = new PropertyDefinition(); - String resourceId = ""; - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.addPropertyToGraph(propertyName, propertyDefinition, resourceId); - } - - - @Test - public void testAddPropertyToGraphByVertex() throws Exception { - PropertyOperation testSubject; - JanusGraphVertex metadataVertex = null; - String propertyName = ""; - PropertyDefinition propertyDefinition = new PropertyDefinition(); - String resourceId = ""; - JanusGraphOperationStatus result; - - // default test - testSubject = createTestSubject(); - result = testSubject.addPropertyToGraphByVertex(metadataVertex, propertyName, propertyDefinition, resourceId); - } - - - @Test - public void testGetJanusGraphGenericDao() throws Exception { - PropertyOperation testSubject; - JanusGraphGenericDao result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getJanusGraphGenericDao(); - } - - @Test - public void testDeletePropertyFromGraph() throws Exception { - PropertyOperation testSubject; - String propertyId = ""; - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.deletePropertyFromGraph(propertyId); - } - - - @Test - public void testUpdateProperty() throws Exception { - PropertyOperation testSubject; - String propertyId = ""; - PropertyDefinition newPropertyDefinition = new PropertyDefinition(); - Map dataTypes = new HashMap<>(); - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.updateProperty(propertyId, newPropertyDefinition, dataTypes); - } - - - @Test - public void testUpdatePropertyFromGraph() throws Exception { - PropertyOperation testSubject; - String propertyId = ""; - PropertyDefinition propertyDefinition = null; - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.updatePropertyFromGraph(propertyId, propertyDefinition); - } - - - @Test - public void testSetJanusGraphGenericDao() { - - PropertyOperation testSubject; + @Test + public void testConvertPropertyDataToPropertyDefinition() throws Exception { + PropertyOperation testSubject; + PropertyData propertyDataResult = new PropertyData(); + String propertyName = ""; + String resourceId = ""; + PropertyDefinition result; + + // default test + testSubject = createTestSubject(); + result = testSubject.convertPropertyDataToPropertyDefinition(propertyDataResult, propertyName, resourceId); + } + + @Test + public void testAddProperty() throws Exception { + PropertyOperation testSubject; + String propertyName = ""; + PropertyDefinition propertyDefinition = new PropertyDefinition(); + String resourceId = ""; + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.addProperty(propertyName, propertyDefinition, resourceId); + } + + + @Test + public void testValidateAndUpdateProperty() throws Exception { + PropertyOperation testSubject; + IComplexDefaultValue propertyDefinition = new PropertyDefinition(); + Map dataTypes = new HashMap<>(); + dataTypes.put("", new DataTypeDefinition()); + StorageOperationStatus result; + + // default test + testSubject = createTestSubject(); + result = testSubject.validateAndUpdateProperty(propertyDefinition, dataTypes); + } + + + @Test + public void testAddPropertyToGraph() throws Exception { + PropertyOperation testSubject; + String propertyName = ""; + PropertyDefinition propertyDefinition = new PropertyDefinition(); + String resourceId = ""; + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.addPropertyToGraph(propertyName, propertyDefinition, resourceId); + } + + + @Test + public void testAddPropertyToGraphByVertex() throws Exception { + PropertyOperation testSubject; + JanusGraphVertex metadataVertex = null; + String propertyName = ""; + PropertyDefinition propertyDefinition = new PropertyDefinition(); + String resourceId = ""; + JanusGraphOperationStatus result; + + // default test + testSubject = createTestSubject(); + result = testSubject.addPropertyToGraphByVertex(metadataVertex, propertyName, propertyDefinition, resourceId); + } + + + @Test + public void testGetJanusGraphGenericDao() throws Exception { + PropertyOperation testSubject; + JanusGraphGenericDao result; + + // default test + testSubject = createTestSubject(); + result = testSubject.getJanusGraphGenericDao(); + } + + @Test + public void testDeletePropertyFromGraph() throws Exception { + PropertyOperation testSubject; + String propertyId = ""; + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.deletePropertyFromGraph(propertyId); + } + + + @Test + public void testUpdateProperty() throws Exception { + PropertyOperation testSubject; + String propertyId = ""; + PropertyDefinition newPropertyDefinition = new PropertyDefinition(); + Map dataTypes = new HashMap<>(); + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.updateProperty(propertyId, newPropertyDefinition, dataTypes); + } + + + @Test + public void testUpdatePropertyFromGraph() throws Exception { + PropertyOperation testSubject; + String propertyId = ""; + PropertyDefinition propertyDefinition = null; + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.updatePropertyFromGraph(propertyId, propertyDefinition); + } + + + @Test + public void testSetJanusGraphGenericDao() { + + PropertyOperation testSubject; HealingJanusGraphGenericDao janusGraphGenericDao = null; - // default test - testSubject = createTestSubject(); - testSubject.setJanusGraphGenericDao(janusGraphGenericDao); - } - - - @Test - public void testAddPropertyToNodeType() { - PropertyOperation testSubject; - String propertyName = ""; - PropertyDefinition propertyDefinition = new PropertyDefinition(); - NodeTypeEnum nodeType = NodeTypeEnum.Attribute; - String uniqueId = ""; - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.addPropertyToNodeType(propertyName, propertyDefinition, nodeType, uniqueId); - } - - - @Test - public void testFindPropertiesOfNode() throws Exception { - PropertyOperation testSubject; - NodeTypeEnum nodeType = null; - String uniqueId = ""; - Either, JanusGraphOperationStatus> result; - - // default test - testSubject = createTestSubject(); - result = testSubject.findPropertiesOfNode(nodeType, uniqueId); - } - - - @Test - public void testDeletePropertiesAssociatedToNode() throws Exception { - PropertyOperation testSubject; - NodeTypeEnum nodeType = null; - String uniqueId = ""; - Either, StorageOperationStatus> result; - - // default test - testSubject = createTestSubject(); - result = testSubject.deletePropertiesAssociatedToNode(nodeType, uniqueId); - } - - - @Test - public void testDeleteAllPropertiesAssociatedToNode() throws Exception { - PropertyOperation testSubject; - NodeTypeEnum nodeType = null; - String uniqueId = ""; - Either, StorageOperationStatus> result; - - // default test - testSubject = createTestSubject(); - result = testSubject.deleteAllPropertiesAssociatedToNode(nodeType, uniqueId); - } - - - @Test - public void testIsPropertyExist() throws Exception { - PropertyOperation testSubject; - List properties = null; - String resourceUid = ""; - String propertyName = ""; - String propertyType = ""; - boolean result; - - // default test - testSubject = createTestSubject(); - result = testSubject.isPropertyExist(properties, resourceUid, propertyName, propertyType); - } - - - @Test - public void testValidateAndUpdateRules() throws Exception { - PropertyOperation testSubject; - String propertyType = ""; - List rules = null; - String innerType = ""; - Map dataTypes = null; - boolean isValidate = false; - ImmutablePair result; - - // test 1 - testSubject = createTestSubject(); - rules = null; - result = testSubject.validateAndUpdateRules(propertyType, rules, innerType, dataTypes, isValidate); - } - - - @Test - public void testAddRulesToNewPropertyValue() throws Exception { - PropertyOperation testSubject; - PropertyValueData propertyValueData = new PropertyValueData(); - ComponentInstanceProperty resourceInstanceProperty = new ComponentInstanceProperty(); - String resourceInstanceId = ""; - - // default test - testSubject = createTestSubject(); - testSubject.addRulesToNewPropertyValue(propertyValueData, resourceInstanceProperty, resourceInstanceId); - } - - - @Test - public void testFindPropertyValue() throws Exception { - PropertyOperation testSubject; - String resourceInstanceId = ""; - String propertyId = ""; - ImmutablePair result; - - // default test - testSubject = createTestSubject(); - result = testSubject.findPropertyValue(resourceInstanceId, propertyId); - } - - - @Test - public void testUpdateRulesInPropertyValue() throws Exception { - PropertyOperation testSubject; - PropertyValueData propertyValueData = new PropertyValueData(); - ComponentInstanceProperty resourceInstanceProperty = new ComponentInstanceProperty(); - String resourceInstanceId = ""; - - // default test - testSubject = createTestSubject(); - testSubject.updateRulesInPropertyValue(propertyValueData, resourceInstanceProperty, resourceInstanceId); - } - - - @Test - public void testGetAllPropertiesOfResourceInstanceOnlyPropertyDefId() throws Exception { - PropertyOperation testSubject; - String resourceInstanceUid = ""; - Either, JanusGraphOperationStatus> result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getAllPropertiesOfResourceInstanceOnlyPropertyDefId(resourceInstanceUid); - } - - - @Test - public void testRemovePropertyOfResourceInstance() throws Exception { - PropertyOperation testSubject; - String propertyValueUid = ""; - String resourceInstanceId = ""; - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.removePropertyOfResourceInstance(propertyValueUid, resourceInstanceId); - } - - - @Test - public void testRemovePropertyValueFromResourceInstance() throws Exception { - PropertyOperation testSubject; - String propertyValueUid = ""; - String resourceInstanceId = ""; - boolean inTransaction = false; - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.removePropertyValueFromResourceInstance(propertyValueUid, resourceInstanceId, - inTransaction); - } - - - @Test - public void testBuildResourceInstanceProperty() throws Exception { - PropertyOperation testSubject; - PropertyValueData propertyValueData = new PropertyValueData(); - ComponentInstanceProperty resourceInstanceProperty = new ComponentInstanceProperty(); - ComponentInstanceProperty result; - - // default test - testSubject = createTestSubject(); - result = testSubject.buildResourceInstanceProperty(propertyValueData, resourceInstanceProperty); - } - - - @Test - public void testIsPropertyDefaultValueValid() throws Exception { - PropertyOperation testSubject; - IComplexDefaultValue propertyDefinition = null; - Map dataTypes = null; - boolean result; - - // test 1 - testSubject = createTestSubject(); - propertyDefinition = null; - result = testSubject.isPropertyDefaultValueValid(propertyDefinition, dataTypes); - Assert.assertEquals(false, result); - } - - - @Test - public void testIsPropertyTypeValid() throws Exception { - PropertyOperation testSubject; - IComplexDefaultValue property = null; - boolean result; - - // test 1 - testSubject = createTestSubject(); - property = null; - result = testSubject.isPropertyTypeValid(property, (String)null); - Assert.assertEquals(false, result); - } - - - @Test - public void testIsPropertyInnerTypeValid() throws Exception { - PropertyOperation testSubject; - IComplexDefaultValue property = null; - Map dataTypes = null; - ImmutablePair result; - - // test 1 - testSubject = createTestSubject(); - property = null; - result = testSubject.isPropertyInnerTypeValid(property, dataTypes); - } - - - @Test - public void testGetAllPropertiesOfResourceInstanceOnlyPropertyDefId_1() throws Exception { - PropertyOperation testSubject; - String resourceInstanceUid = ""; - NodeTypeEnum instanceNodeType = null; - Either, JanusGraphOperationStatus> result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getAllPropertiesOfResourceInstanceOnlyPropertyDefId(resourceInstanceUid, instanceNodeType); - } - - - @Test - public void testFindDefaultValueFromSecondPosition() throws Exception { - PropertyOperation testSubject; - List pathOfComponentInstances = null; - String propertyUniqueId = ""; - String defaultValue = ""; - Either result; - - // test 1 - testSubject = createTestSubject(); - pathOfComponentInstances = null; - result = testSubject.findDefaultValueFromSecondPosition(pathOfComponentInstances, propertyUniqueId, - defaultValue); - } - - - @Test - public void testUpdatePropertyByBestMatch() throws Exception { - PropertyOperation testSubject; - String propertyUniqueId = ""; - ComponentInstanceProperty instanceProperty = new ComponentInstanceProperty(); - List path = new ArrayList<>(); - path.add("path"); - instanceProperty.setPath(path); - Map instanceIdToValue = new HashMap<>(); - instanceIdToValue.put("123", instanceProperty); - - // default test - testSubject = createTestSubject(); - testSubject.updatePropertyByBestMatch(propertyUniqueId, instanceProperty, instanceIdToValue); - } - - - @Test - public void testGetDataTypeByUid() throws Exception { - PropertyOperation testSubject; - String uniqueId = ""; - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getDataTypeByUid(uniqueId); - } - - - @Test - public void testAddAndGetDataType() throws Exception { - final String dataTypeName = "myDataType"; - DataTypeDefinition dataTypeDefinition = new DataTypeDefinition(); - dataTypeDefinition.setName("myDataType"); - Either result; - + // default test + testSubject = createTestSubject(); + testSubject.setJanusGraphGenericDao(janusGraphGenericDao); + } + + + @Test + public void testAddPropertyToNodeType() { + PropertyOperation testSubject; + String propertyName = ""; + PropertyDefinition propertyDefinition = new PropertyDefinition(); + NodeTypeEnum nodeType = NodeTypeEnum.Attribute; + String uniqueId = ""; + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.addPropertyToNodeType(propertyName, propertyDefinition, nodeType, uniqueId); + } + + + @Test + public void testFindPropertiesOfNode() throws Exception { + PropertyOperation testSubject; + NodeTypeEnum nodeType = null; + String uniqueId = ""; + Either, JanusGraphOperationStatus> result; + + // default test + testSubject = createTestSubject(); + result = testSubject.findPropertiesOfNode(nodeType, uniqueId); + } + + + @Test + public void testDeletePropertiesAssociatedToNode() throws Exception { + PropertyOperation testSubject; + NodeTypeEnum nodeType = null; + String uniqueId = ""; + Either, StorageOperationStatus> result; + + // default test + testSubject = createTestSubject(); + result = testSubject.deletePropertiesAssociatedToNode(nodeType, uniqueId); + } + + + @Test + public void testDeleteAllPropertiesAssociatedToNode() throws Exception { + PropertyOperation testSubject; + NodeTypeEnum nodeType = null; + String uniqueId = ""; + Either, StorageOperationStatus> result; + + // default test + testSubject = createTestSubject(); + result = testSubject.deleteAllPropertiesAssociatedToNode(nodeType, uniqueId); + } + + + @Test + public void testIsPropertyExist() throws Exception { + PropertyOperation testSubject; + List properties = null; + String resourceUid = ""; + String propertyName = ""; + String propertyType = ""; + boolean result; + + // default test + testSubject = createTestSubject(); + result = testSubject.isPropertyExist(properties, resourceUid, propertyName, propertyType); + } + + + @Test + public void testValidateAndUpdateRules() throws Exception { + PropertyOperation testSubject; + String propertyType = ""; + List rules = null; + String innerType = ""; + Map dataTypes = null; + boolean isValidate = false; + ImmutablePair result; + + // test 1 + testSubject = createTestSubject(); + rules = null; + result = testSubject.validateAndUpdateRules(propertyType, rules, innerType, dataTypes, isValidate); + } + + + @Test + public void testAddRulesToNewPropertyValue() throws Exception { + PropertyOperation testSubject; + PropertyValueData propertyValueData = new PropertyValueData(); + ComponentInstanceProperty resourceInstanceProperty = new ComponentInstanceProperty(); + String resourceInstanceId = ""; + + // default test + testSubject = createTestSubject(); + testSubject.addRulesToNewPropertyValue(propertyValueData, resourceInstanceProperty, resourceInstanceId); + } + + + @Test + public void testFindPropertyValue() throws Exception { + PropertyOperation testSubject; + String resourceInstanceId = ""; + String propertyId = ""; + ImmutablePair result; + + // default test + testSubject = createTestSubject(); + result = testSubject.findPropertyValue(resourceInstanceId, propertyId); + } + + + @Test + public void testUpdateRulesInPropertyValue() throws Exception { + PropertyOperation testSubject; + PropertyValueData propertyValueData = new PropertyValueData(); + ComponentInstanceProperty resourceInstanceProperty = new ComponentInstanceProperty(); + String resourceInstanceId = ""; + + // default test + testSubject = createTestSubject(); + testSubject.updateRulesInPropertyValue(propertyValueData, resourceInstanceProperty, resourceInstanceId); + } + + + @Test + public void testGetAllPropertiesOfResourceInstanceOnlyPropertyDefId() throws Exception { + PropertyOperation testSubject; + String resourceInstanceUid = ""; + Either, JanusGraphOperationStatus> result; + + // default test + testSubject = createTestSubject(); + result = testSubject.getAllPropertiesOfResourceInstanceOnlyPropertyDefId(resourceInstanceUid); + } + + + @Test + public void testRemovePropertyOfResourceInstance() throws Exception { + PropertyOperation testSubject; + String propertyValueUid = ""; + String resourceInstanceId = ""; + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.removePropertyOfResourceInstance(propertyValueUid, resourceInstanceId); + } + + + @Test + public void testRemovePropertyValueFromResourceInstance() throws Exception { + PropertyOperation testSubject; + String propertyValueUid = ""; + String resourceInstanceId = ""; + boolean inTransaction = false; + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.removePropertyValueFromResourceInstance(propertyValueUid, resourceInstanceId, + inTransaction); + } + + + @Test + public void testBuildResourceInstanceProperty() throws Exception { + PropertyOperation testSubject; + PropertyValueData propertyValueData = new PropertyValueData(); + ComponentInstanceProperty resourceInstanceProperty = new ComponentInstanceProperty(); + ComponentInstanceProperty result; + + // default test + testSubject = createTestSubject(); + result = testSubject.buildResourceInstanceProperty(propertyValueData, resourceInstanceProperty); + } + + + @Test + public void testIsPropertyDefaultValueValid() throws Exception { + PropertyOperation testSubject; + IComplexDefaultValue propertyDefinition = null; + Map dataTypes = null; + boolean result; + + // test 1 + testSubject = createTestSubject(); + propertyDefinition = null; + result = testSubject.isPropertyDefaultValueValid(propertyDefinition, dataTypes); + Assert.assertEquals(false, result); + } + + + @Test + public void testIsPropertyTypeValid() throws Exception { + PropertyOperation testSubject; + IComplexDefaultValue property = null; + boolean result; + + // test 1 + testSubject = createTestSubject(); + property = null; + result = testSubject.isPropertyTypeValid(property, (String) null); + Assert.assertEquals(false, result); + } + + + @Test + public void testIsPropertyInnerTypeValid() throws Exception { + PropertyOperation testSubject; + IComplexDefaultValue property = null; + Map dataTypes = null; + ImmutablePair result; + + // test 1 + testSubject = createTestSubject(); + property = null; + result = testSubject.isPropertyInnerTypeValid(property, dataTypes); + } + + + @Test + public void testGetAllPropertiesOfResourceInstanceOnlyPropertyDefId_1() throws Exception { + PropertyOperation testSubject; + String resourceInstanceUid = ""; + NodeTypeEnum instanceNodeType = null; + Either, JanusGraphOperationStatus> result; + + // default test + testSubject = createTestSubject(); + result = testSubject.getAllPropertiesOfResourceInstanceOnlyPropertyDefId(resourceInstanceUid, instanceNodeType); + } + + + @Test + public void testFindDefaultValueFromSecondPosition() throws Exception { + PropertyOperation testSubject; + List pathOfComponentInstances = null; + String propertyUniqueId = ""; + String defaultValue = ""; + Either result; + + // test 1 + testSubject = createTestSubject(); + pathOfComponentInstances = null; + result = testSubject.findDefaultValueFromSecondPosition(pathOfComponentInstances, propertyUniqueId, + defaultValue); + } + + + @Test + public void testUpdatePropertyByBestMatch() throws Exception { + PropertyOperation testSubject; + String propertyUniqueId = ""; + ComponentInstanceProperty instanceProperty = new ComponentInstanceProperty(); + List path = new ArrayList<>(); + path.add("path"); + instanceProperty.setPath(path); + Map instanceIdToValue = new HashMap<>(); + instanceIdToValue.put("123", instanceProperty); + + // default test + testSubject = createTestSubject(); + testSubject.updatePropertyByBestMatch(propertyUniqueId, instanceProperty, instanceIdToValue); + } + + + @Test + public void testGetDataTypeByUid() throws Exception { + PropertyOperation testSubject; + String uniqueId = ""; + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.getDataTypeByUid(uniqueId); + } + + + @Test + public void testAddAndGetDataType() throws Exception { + final String dataTypeName = "myDataType"; + DataTypeDefinition dataTypeDefinition = new DataTypeDefinition(); + dataTypeDefinition.setName("myDataType"); + Either result; + Mockito.doReturn(Either.left(new DataTypeData(dataTypeDefinition))).when(janusGraphGenericDao) .createNode(Mockito.any(), Mockito.eq(DataTypeData.class)); - + Mockito.doReturn(Either.left(new DataTypeData(dataTypeDefinition))).when(janusGraphGenericDao) .getNode(GraphPropertiesDictionary.NAME.getProperty(), dataTypeName, DataTypeData.class, null); - + Mockito.doReturn(Either.left(Collections.EMPTY_LIST)).when(janusGraphGenericDao) - .getChildrenNodes(Mockito.anyString(), Mockito.anyString(), Mockito.eq(GraphEdgeLabels.PROPERTY), Mockito.eq(NodeTypeEnum.Property), Mockito.eq(PropertyData.class)); + .getChildrenNodes(Mockito.anyString(), Mockito.anyString(), Mockito.eq(GraphEdgeLabels.PROPERTY), Mockito.eq(NodeTypeEnum.Property), + Mockito.eq(PropertyData.class)); - result = propertyOperation.addDataType(dataTypeDefinition); + result = propertyOperation.addDataType(dataTypeDefinition); assertTrue(result.isLeft()); - + Mockito.doReturn(Either.right(JanusGraphOperationStatus.NOT_FOUND)).when(janusGraphGenericDao) - .getChild(Mockito.anyString(), Mockito.anyString(), Mockito.eq(GraphEdgeLabels.DERIVED_FROM), Mockito.eq(NodeTypeEnum.DataType), Mockito.eq(DataTypeData.class)); - - result = propertyOperation.getDataTypeByName(dataTypeName, null, false); - assertTrue(result.isLeft()); - - result = propertyOperation.getDataTypeByName(dataTypeName, null); - assertTrue(result.isLeft()); - + .getChild(Mockito.anyString(), Mockito.anyString(), Mockito.eq(GraphEdgeLabels.DERIVED_FROM), Mockito.eq(NodeTypeEnum.DataType), + Mockito.eq(DataTypeData.class)); + + result = propertyOperation.getDataTypeByName(dataTypeName, null, false); + assertTrue(result.isLeft()); + + result = propertyOperation.getDataTypeByName(dataTypeName, null); + assertTrue(result.isLeft()); + Mockito.doReturn(Either.left(new DataTypeData(dataTypeDefinition))).when(janusGraphGenericDao) .getNode(GraphPropertiesDictionary.UNIQUE_ID.getProperty(), dataTypeName + ".datatype", DataTypeData.class); - - Either resultGetByUid = propertyOperation.getDataTypeByUid("myDataType.datatype"); - assertTrue(resultGetByUid.isLeft()); - - Either resultIsDefinedDataType = propertyOperation.isDefinedInDataTypes(dataTypeName, null); + + Either resultGetByUid = propertyOperation.getDataTypeByUid("myDataType.datatype"); + assertTrue(resultGetByUid.isLeft()); + + Either resultIsDefinedDataType = propertyOperation.isDefinedInDataTypes(dataTypeName, null); assertTrue(resultIsDefinedDataType.isLeft()); - } - - @Test - public void testAddDataTypeToModel() throws Exception { - DataTypeDefinition dataTypeDefinition = new DataTypeDefinition(); - dataTypeDefinition.setName("testName"); - dataTypeDefinition.setModel("testModel"); - Either result; - - Mockito.doReturn(Either.left(new DataTypeData(dataTypeDefinition))).when(janusGraphGenericDao) - .createNode(Mockito.any(), Mockito.eq(DataTypeData.class)); - - Mockito.doReturn(Either.left(new GraphRelation())).when(janusGraphGenericDao) - .createRelation(Mockito.any(), Mockito.any(), Mockito.eq(GraphEdgeLabels.MODEL_ELEMENT), Mockito.any()); - - result = propertyOperation.addDataType(dataTypeDefinition); - assertTrue(result.isLeft()); - - Mockito.doReturn(Either.left(new DataTypeData(dataTypeDefinition))).when(janusGraphGenericDao) - .getNode(GraphPropertiesDictionary.UNIQUE_ID.getProperty(), "testModel.testName.datatype", DataTypeData.class); - - Mockito.doReturn(Either.left(Collections.EMPTY_LIST)).when(janusGraphGenericDao) - .getChildrenNodes(Mockito.anyString(), Mockito.anyString(), Mockito.eq(GraphEdgeLabels.PROPERTY), Mockito.eq(NodeTypeEnum.Property), Mockito.eq(PropertyData.class)); - - Mockito.doReturn(Either.right(JanusGraphOperationStatus.NOT_FOUND)).when(janusGraphGenericDao) - .getChild(Mockito.anyString(), Mockito.anyString(), Mockito.eq(GraphEdgeLabels.DERIVED_FROM), Mockito.eq(NodeTypeEnum.DataType), Mockito.eq(DataTypeData.class)); - - Either resultGetByUid = propertyOperation.getDataTypeByUid("testModel.testName.datatype"); - assertTrue(resultGetByUid.isLeft()); - } - - - @Test - public void testGetDataTypeByUidWithoutDerivedDataTypes() throws Exception { - PropertyOperation testSubject; - String uniqueId = ""; - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getDataTypeByUidWithoutDerivedDataTypes(uniqueId); - } - - - @Test - public void testGetAllDataTypes() throws Exception { - PropertyOperation testSubject; - Either>, JanusGraphOperationStatus> result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getAllDataTypes(); - } - - - @Test - public void testCheckInnerType() throws Exception { - PropertyOperation testSubject; - PropertyDataDefinition propDataDef = new PropertyDataDefinition(); - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.checkInnerType(propDataDef); - } - - @Test - public void testValidateAndUpdatePropertyValue() throws Exception { - PropertyOperation testSubject; - String propertyType = ""; - String value = ""; - boolean isValidate = false; - String innerType = ""; - Map dataTypes = null; - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.validateAndUpdatePropertyValue(propertyType, value, isValidate, innerType, dataTypes); - } - - - @Test - public void testValidateAndUpdatePropertyValue_1() throws Exception { - PropertyOperation testSubject; - String propertyType = ""; - String value = ""; - String innerType = ""; - Map dataTypes = new HashMap<>(); - dataTypes.put("", new DataTypeDefinition()); - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.validateAndUpdatePropertyValue(propertyType, value, innerType, dataTypes); - } - - - - - - @Test - public void testAddPropertiesToElementType() throws Exception { - PropertyOperation testSubject; - String uniqueId = ""; - NodeTypeEnum elementType = null; - List properties = null; - Either, JanusGraphOperationStatus> result; - - // test 1 - testSubject = createTestSubject(); - properties = null; - result = testSubject.addPropertiesToElementType(uniqueId, elementType, properties); - } - - - @Test - public void testUpdateDataType() throws Exception { - PropertyOperation testSubject; - DataTypeDefinition newDataTypeDefinition = new DataTypeDefinition(); - DataTypeDefinition oldDataTypeDefinition = new DataTypeDefinition(); - Either result; - - // default test - testSubject = createTestSubject(); - result = testSubject.updateDataType(newDataTypeDefinition, oldDataTypeDefinition); - } + } + + @Test + public void testAddDataTypeToModel() throws Exception { + DataTypeDefinition dataTypeDefinition = new DataTypeDefinition(); + dataTypeDefinition.setName("testName"); + dataTypeDefinition.setModel("testModel"); + Either result; + + Mockito.doReturn(Either.left(new DataTypeData(dataTypeDefinition))).when(janusGraphGenericDao) + .createNode(Mockito.any(), Mockito.eq(DataTypeData.class)); + + Mockito.doReturn(Either.left(new GraphRelation())).when(janusGraphGenericDao) + .createRelation(Mockito.any(), Mockito.any(), Mockito.eq(GraphEdgeLabels.MODEL_ELEMENT), Mockito.any()); + + result = propertyOperation.addDataType(dataTypeDefinition); + assertTrue(result.isLeft()); + + Mockito.doReturn(Either.left(new DataTypeData(dataTypeDefinition))).when(janusGraphGenericDao) + .getNode(GraphPropertiesDictionary.UNIQUE_ID.getProperty(), "testModel.testName.datatype", DataTypeData.class); + + Mockito.doReturn(Either.left(Collections.EMPTY_LIST)).when(janusGraphGenericDao) + .getChildrenNodes(Mockito.anyString(), Mockito.anyString(), Mockito.eq(GraphEdgeLabels.PROPERTY), Mockito.eq(NodeTypeEnum.Property), + Mockito.eq(PropertyData.class)); + + Mockito.doReturn(Either.right(JanusGraphOperationStatus.NOT_FOUND)).when(janusGraphGenericDao) + .getChild(Mockito.anyString(), Mockito.anyString(), Mockito.eq(GraphEdgeLabels.DERIVED_FROM), Mockito.eq(NodeTypeEnum.DataType), + Mockito.eq(DataTypeData.class)); + + Either resultGetByUid = propertyOperation.getDataTypeByUid("testModel.testName.datatype"); + assertTrue(resultGetByUid.isLeft()); + } + + + @Test + public void testGetDataTypeByUidWithoutDerivedDataTypes() throws Exception { + PropertyOperation testSubject; + String uniqueId = ""; + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.getDataTypeByUidWithoutDerivedDataTypes(uniqueId); + } + + + @Test + public void testGetAllDataTypes() throws Exception { + PropertyOperation testSubject; + Either>, JanusGraphOperationStatus> result; + + // default test + testSubject = createTestSubject(); + result = testSubject.getAllDataTypes(); + } + + + @Test + public void testCheckInnerType() throws Exception { + PropertyOperation testSubject; + PropertyDataDefinition propDataDef = new PropertyDataDefinition(); + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.checkInnerType(propDataDef); + } + + @Test + public void testValidateAndUpdatePropertyValue() throws Exception { + PropertyOperation testSubject; + String propertyType = ""; + String value = ""; + boolean isValidate = false; + String innerType = ""; + Map dataTypes = null; + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.validateAndUpdatePropertyValue(propertyType, value, isValidate, innerType, dataTypes); + } + + + @Test + public void testValidateAndUpdatePropertyValue_1() throws Exception { + PropertyOperation testSubject; + String propertyType = ""; + String value = ""; + String innerType = ""; + Map dataTypes = new HashMap<>(); + dataTypes.put("", new DataTypeDefinition()); + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.validateAndUpdatePropertyValue(propertyType, value, innerType, dataTypes); + } + + + @Test + public void testAddPropertiesToElementType() throws Exception { + PropertyOperation testSubject; + String uniqueId = ""; + NodeTypeEnum elementType = null; + List properties = null; + Either, JanusGraphOperationStatus> result; + + // test 1 + testSubject = createTestSubject(); + properties = null; + result = testSubject.addPropertiesToElementType(uniqueId, elementType, properties); + } + + + @Test + public void testUpdateDataType() throws Exception { + PropertyOperation testSubject; + DataTypeDefinition newDataTypeDefinition = new DataTypeDefinition(); + DataTypeDefinition oldDataTypeDefinition = new DataTypeDefinition(); + Either result; + + // default test + testSubject = createTestSubject(); + result = testSubject.updateDataType(newDataTypeDefinition, oldDataTypeDefinition); + } } diff --git a/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/RelationshipTypeOperationTest.java b/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/RelationshipTypeOperationTest.java index 5f8bde8090..079874c59c 100644 --- a/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/RelationshipTypeOperationTest.java +++ b/catalog-model/src/test/java/org/openecomp/sdc/be/model/operations/impl/RelationshipTypeOperationTest.java @@ -26,6 +26,7 @@ import static org.junit.Assert.assertTrue; import fj.data.Either; import java.util.ArrayList; +import java.util.Arrays; import java.util.Collections; import java.util.HashMap; import java.util.List; @@ -302,10 +303,7 @@ public class RelationshipTypeOperationTest extends ModelTestBase { property2.setDescription("Number of (actual or virtual) CPUs associated with the Compute node."); property2.setType(ToscaType.INTEGER.name().toLowerCase()); List constraints3 = new ArrayList<>(); - List range = new ArrayList<>(); - range.add("4"); - range.add("1"); - InRangeConstraint propertyConstraint3 = new InRangeConstraint(range); + InRangeConstraint propertyConstraint3 = new InRangeConstraint(Arrays.asList("1", "4")); constraints3.add(propertyConstraint3); property2.setConstraints(constraints3); return property2; @@ -593,10 +591,7 @@ public class RelationshipTypeOperationTest extends ModelTestBase { propertyDefinition.setDescription(PROP + "_" + value); propertyDefinition.setType(ToscaType.INTEGER.name().toLowerCase()); List constraints = new ArrayList<>(); - List range = new ArrayList<>(); - range.add("1"); - range.add("4"); - InRangeConstraint propertyConstraint = new InRangeConstraint(range); + InRangeConstraint propertyConstraint = new InRangeConstraint(Arrays.asList("1", "4")); constraints.add(propertyConstraint); propertyDefinition.setConstraints(constraints); Map propertiesMap = new HashMap<>(); diff --git a/catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/ToscaTypeTest.java b/catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/ToscaTypeTest.java index 103b15b79d..204820698b 100644 --- a/catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/ToscaTypeTest.java +++ b/catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/ToscaTypeTest.java @@ -176,7 +176,7 @@ class ToscaTypeTest { @Test void testConvert() throws Exception { ToscaType typeInt = ToscaType.INTEGER; - assertEquals(123l, typeInt.convert("123")); + assertEquals(123, typeInt.convert("123")); ToscaType typeBool = ToscaType.BOOLEAN; assertEquals(true, typeBool.convert("true")); diff --git a/catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/constraints/InRangeConstraintTest.java b/catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/constraints/InRangeConstraintTest.java index 1491e0547d..f7223e8949 100644 --- a/catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/constraints/InRangeConstraintTest.java +++ b/catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/constraints/InRangeConstraintTest.java @@ -25,7 +25,7 @@ import static org.junit.jupiter.api.Assertions.assertFalse; import static org.junit.jupiter.api.Assertions.assertThrows; import static org.junit.jupiter.api.Assertions.assertTrue; -import java.util.ArrayList; +import java.util.Arrays; import java.util.List; import org.junit.jupiter.api.Test; import org.openecomp.sdc.be.model.tosca.constraints.exception.ConstraintValueDoNotMatchPropertyTypeException; @@ -33,17 +33,11 @@ import org.openecomp.sdc.be.model.tosca.constraints.exception.ConstraintValueDoN class InRangeConstraintTest { private InRangeConstraint createStringTestSubject() { - List validValues = new ArrayList<>(); - validValues.add("test1"); - validValues.add("test10"); - return new InRangeConstraint(validValues); + return new InRangeConstraint(Arrays.asList("test1", "test10")); } private InRangeConstraint createIntegerTestSubject() { - List validValues = new ArrayList<>(); - validValues.add(1); - validValues.add(10); - return new InRangeConstraint(validValues); + return new InRangeConstraint(Arrays.asList(1, 10)); } @Test @@ -59,10 +53,7 @@ class InRangeConstraintTest { @Test void testSetInRange() { InRangeConstraint testSubject = createStringTestSubject(); - List validValues = new ArrayList<>(); - validValues.add("test21"); - validValues.add("test30"); - testSubject.setInRange(validValues); + testSubject.setInRange(Arrays.asList("test21", "test30")); List result = testSubject.getInRange(); assertEquals(2, result.size()); -- cgit 1.2.3-korg