diff options
author | xuegao <xue.gao@intl.att.com> | 2021-03-25 10:31:08 +0100 |
---|---|---|
committer | Vasyl Razinkov <vasyl.razinkov@est.tech> | 2021-03-25 16:27:18 +0000 |
commit | de9224b1ae01755c7cc46e4bdd1a50c1bc08936e (patch) | |
tree | 86369286ebb95412426a73f1e36e317040f6b2d0 /catalog-model/src/test/java/org | |
parent | 7af307919d19761959a61ac64603312b3fb45b58 (diff) |
Improve test coverage
Add unit tests to improve test coverage.
Issue-ID: SDC-3428
Change-Id: I47da0decd6b6df93ace68b2af586b255ef0b792e
Signed-off-by: xuegao <xue.gao@intl.att.com>
Diffstat (limited to 'catalog-model/src/test/java/org')
-rw-r--r-- | catalog-model/src/test/java/org/openecomp/sdc/be/model/ComponentInstInputsMapTest.java | 137 | ||||
-rw-r--r-- | catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/VersionUtilTest.java | 70 |
2 files changed, 158 insertions, 49 deletions
diff --git a/catalog-model/src/test/java/org/openecomp/sdc/be/model/ComponentInstInputsMapTest.java b/catalog-model/src/test/java/org/openecomp/sdc/be/model/ComponentInstInputsMapTest.java index d256cd15c8..92026dd211 100644 --- a/catalog-model/src/test/java/org/openecomp/sdc/be/model/ComponentInstInputsMapTest.java +++ b/catalog-model/src/test/java/org/openecomp/sdc/be/model/ComponentInstInputsMapTest.java @@ -20,14 +20,17 @@ package org.openecomp.sdc.be.model; -import org.junit.Assert; -import org.junit.Test; +import org.apache.commons.lang3.tuple.Pair; +import org.junit.jupiter.api.Test; import java.util.HashMap; import java.util.LinkedList; import java.util.List; import java.util.Map; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertThrows; + public class ComponentInstInputsMapTest { private ComponentInstInputsMap createTestSubject() { @@ -35,85 +38,121 @@ public class ComponentInstInputsMapTest { } @Test - public void testGetComponentInstanceInputsMap() throws Exception { - ComponentInstInputsMap testSubject; - Map<String, List<ComponentInstancePropInput>> result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getComponentInstanceInputsMap(); - } - - @Test - public void testSetComponentInstanceInputsMap() throws Exception { - ComponentInstInputsMap testSubject; + public void testComponentInstanceInputsMap() { + ComponentInstInputsMap testSubject = createTestSubject(); Map<String, List<ComponentInstancePropInput>> componentInstanceInputsMap = null; // default test - testSubject = createTestSubject(); testSubject.setComponentInstanceInputsMap(componentInstanceInputsMap); - } - @Test - public void testGetComponentInstanceProperties() throws Exception { - ComponentInstInputsMap testSubject; - Map<String, List<ComponentInstancePropInput>> result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getComponentInstanceProperties(); + Map<String, List<ComponentInstancePropInput>> result = testSubject.getComponentInstanceInputsMap(); + assertEquals(0, result.size()); } @Test - public void testSetComponentInstancePropInput() throws Exception { - ComponentInstInputsMap testSubject; + public void testComponentInstanceProperties() { + ComponentInstInputsMap testSubject = createTestSubject(); Map<String, List<ComponentInstancePropInput>> componentInstanceProperties = new HashMap<>(); - - // default test - testSubject = createTestSubject(); testSubject.setComponentInstancePropertiesToPolicies(componentInstanceProperties); + Map<String, List<ComponentInstancePropInput>> result = testSubject.getComponentInstanceProperties(); + assertEquals(0, result.size()); } @Test - public void testResolvePropertiesToDeclareEmpty() throws Exception { + public void testResolvePropertiesToDeclareEmpty() { ComponentInstInputsMap testSubject; - Map<String, List<ComponentInstancePropInput>> componentInstanceProperties = null; // default test testSubject = createTestSubject(); - try { + assertThrows(IllegalStateException.class, () -> { testSubject.resolvePropertiesToDeclare(); - } catch (Exception e) { - Assert.assertTrue(e.getClass() == IllegalStateException.class); - } + }); } @Test - public void testResolvePropertiesToDeclare() throws Exception { + public void testResolvePropertiesToDeclare() { ComponentInstInputsMap testSubject; - Map<String, List<ComponentInstancePropInput>> componentInstanceProperties = null; - Map<String, List<ComponentInstancePropInput>> inputs = new HashMap<>(); - inputs.put("mock", new LinkedList<>()); + inputs.put("test", new LinkedList<>()); // default test testSubject = createTestSubject(); testSubject.setComponentInstanceInputsMap(inputs); - testSubject.resolvePropertiesToDeclare(); + Pair<String, List<ComponentInstancePropInput>> result = testSubject.resolvePropertiesToDeclare(); + assertEquals(0, result.getValue().size()); + testSubject = createTestSubject(); testSubject.setComponentInstancePropertiesToPolicies(inputs); - testSubject.resolvePropertiesToDeclare(); + result = testSubject.resolvePropertiesToDeclare(); + assertEquals(0, result.getValue().size()); + testSubject = createTestSubject(); testSubject.setPolicyProperties(inputs); - testSubject.resolvePropertiesToDeclare(); + result = testSubject.resolvePropertiesToDeclare(); + assertEquals(0, result.getValue().size()); + + testSubject = createTestSubject(); + testSubject.setComponentInstancePropInput(inputs); + result = testSubject.resolvePropertiesToDeclare(); + assertEquals(0, result.getValue().size()); + + testSubject = createTestSubject(); + testSubject.setServiceProperties(inputs); + result = testSubject.resolvePropertiesToDeclare(); + assertEquals(0, result.getValue().size()); + + testSubject = createTestSubject(); + testSubject.setGroupProperties(inputs); + result = testSubject.resolvePropertiesToDeclare(); + assertEquals(0, result.getValue().size()); + + testSubject = createTestSubject(); + testSubject.setComponentPropertiesToPolicies(inputs); + result = testSubject.resolvePropertiesToDeclare(); + assertEquals(0, result.getValue().size()); } - + @Test - public void testGetPolicyProperties() throws Exception { - ComponentInstInputsMap testSubject; - Map<String, List<ComponentInstancePropInput>> componentInstanceProperties = null; + public void testPolicyProperties() { + ComponentInstInputsMap testSubject = createTestSubject(); + Map<String, List<ComponentInstancePropInput>> policyProperties = new HashMap<>(); + testSubject.setPolicyProperties(policyProperties); + Map<String, List<ComponentInstancePropInput>> result = testSubject.getPolicyProperties(); + assertEquals(0, result.size()); + } - // default test - testSubject = createTestSubject(); - testSubject.getPolicyProperties(); + @Test + public void testServiceProperties() { + ComponentInstInputsMap testSubject = createTestSubject(); + Map<String, List<ComponentInstancePropInput>> serviceProperties = new HashMap<>(); + testSubject.setServiceProperties(serviceProperties); + Map<String, List<ComponentInstancePropInput>> result = testSubject.getServiceProperties(); + assertEquals(0, result.size()); + } + + @Test + public void testGroupProperties() { + ComponentInstInputsMap testSubject = createTestSubject(); + Map<String, List<ComponentInstancePropInput>> groupProperties = new HashMap<>(); + testSubject.setGroupProperties(groupProperties); + Map<String, List<ComponentInstancePropInput>> result = testSubject.getGroupProperties(); + assertEquals(0, result.size()); + } + + @Test + public void testComponentPropertiesToPolicies() { + ComponentInstInputsMap testSubject = createTestSubject(); + Map<String, List<ComponentInstancePropInput>> componentPropertiesToPolicies = new HashMap<>(); + testSubject.setComponentPropertiesToPolicies(componentPropertiesToPolicies); + Map<String, List<ComponentInstancePropInput>> result = testSubject.getComponentPropertiesToPolicies(); + assertEquals(0, result.size()); + } + + @Test + public void testComponentInstancePropertiesToPolicies() { + ComponentInstInputsMap testSubject = createTestSubject(); + Map<String, List<ComponentInstancePropInput>> componentInstancePropertiesToPolicies = new HashMap<>(); + testSubject.setComponentInstancePropertiesToPolicies(componentInstancePropertiesToPolicies); + Map<String, List<ComponentInstancePropInput>> result = testSubject.getComponentInstancePropertiesToPolicies(); + assertEquals(0, result.size()); } } diff --git a/catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/VersionUtilTest.java b/catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/VersionUtilTest.java new file mode 100644 index 0000000000..89faa3afba --- /dev/null +++ b/catalog-model/src/test/java/org/openecomp/sdc/be/model/tosca/VersionUtilTest.java @@ -0,0 +1,70 @@ +/*- + * ============LICENSE_START======================================================= + * SDC + * ================================================================================ + * Copyright (C) 2021 AT&T Intellectual Property. All rights reserved. + * ================================================================================ + * 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. + * See the License for the specific language governing permissions and + * limitations under the License. + * ============LICENSE_END========================================================= + */ + +package org.openecomp.sdc.be.model.tosca; + +import org.junit.jupiter.api.Test; +import org.openecomp.sdc.be.model.tosca.version.ApplicationVersionException; +import org.openecomp.sdc.be.model.tosca.version.Version; + +import static org.junit.jupiter.api.Assertions.*; + + +public class VersionUtilTest { + + @Test + public void testIsSnapshot() { + assertTrue(VersionUtil.isSnapshot("test_snapshot")); + assertTrue(VersionUtil.isSnapshot("test_SNAPSHOT")); + assertFalse(VersionUtil.isSnapshot("test_SNAP")); + } + + @Test + public void testIsValid() { + assertTrue(VersionUtil.isValid("1.0.2")); + assertTrue(VersionUtil.isValid("1.0-2")); + assertFalse(VersionUtil.isValid("1!2")); + } + + @Test + public void testParseVersion() { + Version ver1 = VersionUtil.parseVersion("1.0.2"); + assertEquals(1, ver1.getMajorVersion()); + assertEquals(0, ver1.getMinorVersion()); + assertEquals(2, ver1.getIncrementalVersion()); + + Version ver2 = VersionUtil.parseVersion("1.0-2"); + assertEquals(1, ver2.getMajorVersion()); + assertEquals(0, ver2.getMinorVersion()); + assertEquals(2, ver2.getBuildNumber()); + + assertThrows( + ApplicationVersionException.class, + () -> VersionUtil.parseVersion("1!2") + ); + } + + @Test + public void testCompare() { + assertEquals(-1, VersionUtil.compare("1.0.2", "1.0.3")); + assertEquals(0, VersionUtil.compare("1.0.2", "1.0.2")); + assertEquals(1, VersionUtil.compare("1.0.2", "0.0.5")); + } +} |