Code Improvements-Sonar Issue Fixes

Signed-off-by: SravanKumarGunda <sravan.kumar1@huawei.com>
Issue-ID: CLI-270
Change-Id: Iba00ddc8c471bc5990d87ba3729af373e2bd6711

4 files changed, 34 insertions, 1 deletions

diff --git a/framework/src/test/java/org/onap/cli/cmd/sample/OnapCommandSampleTest.java b/framework/src/test/java/org/onap/cli/cmd/sample/OnapCommandSampleTest.java
index 3e32f3e1..60aa5e7d 100644
--- a/framework/src/test/java/org/onap/cli/cmd/sample/OnapCommandSampleTest.java
+++ b/framework/src/test/java/org/onap/cli/cmd/sample/OnapCommandSampleTest.java
@@ -114,7 +114,7 @@ public class OnapCommandSampleTest {
             UUID.fromString(attrValue.substring(4));
             attr = sample.getResult().getRecordsMap().get("output-2");
             attrValue = attr.getValues().get(0);
-            assertEquals(attrValue, "test");
+            assertEquals("test", attrValue);
         } catch (IllegalArgumentException e) {
             fail("Failed to replace the output default value on output-1");
         } catch (OnapCommandException e) {
diff --git a/framework/src/test/java/org/onap/cli/fw/conf/OnapCommandConfgTest.java b/framework/src/test/java/org/onap/cli/fw/conf/OnapCommandConfgTest.java
index d6ef3a34..ae620632 100644
--- a/framework/src/test/java/org/onap/cli/fw/conf/OnapCommandConfgTest.java
+++ b/framework/src/test/java/org/onap/cli/fw/conf/OnapCommandConfgTest.java
@@ -16,6 +16,14 @@
 
 package org.onap.cli.fw.conf;
 
+import org.junit.Test;
+
 public class OnapCommandConfgTest {
 
+	@Test
+	public void addPropertiesTest() {
+
+	    OnapCommandConfig.getPropertyValue("cli.schema.profile.confs");
+	}
+
 }
diff --git a/framework/src/test/java/org/onap/cli/fw/schema/ValidateSchemaTest.java b/framework/src/test/java/org/onap/cli/fw/schema/ValidateSchemaTest.java
index 5831c272..4c64ca2a 100644
--- a/framework/src/test/java/org/onap/cli/fw/schema/ValidateSchemaTest.java
+++ b/framework/src/test/java/org/onap/cli/fw/schema/ValidateSchemaTest.java
@@ -123,4 +123,18 @@ public class ValidateSchemaTest {
         assertTrue(errorList5.size() == 0);
 
     }
+
+    @Test
+    public void validateAfterRemovingIfElseTest() throws OnapCommandException {
+        OnapCommand cmd2 = new OnapCommand() {
+            @Override
+            protected void run() throws OnapCommandException {
+
+            }
+        };
+        List < String > errorList2 = OnapCommandSchemaLoader.loadSchema(cmd2,
+            "schema-validate-basic-default-attr.yaml", true, true);
+        assertTrue(errorList2.size() > 0);
+    }
+
 }
diff --git a/framework/src/test/java/org/onap/cli/fw/utils/OnapCommandUtilsTest.java b/framework/src/test/java/org/onap/cli/fw/utils/OnapCommandUtilsTest.java
index e01c3878..b7da1c2f 100644
--- a/framework/src/test/java/org/onap/cli/fw/utils/OnapCommandUtilsTest.java
+++ b/framework/src/test/java/org/onap/cli/fw/utils/OnapCommandUtilsTest.java
@@ -533,4 +533,15 @@ public class OnapCommandUtilsTest {
    public void testMd5(){
     assertNotNull(OnapCommandUtils.md5("a"));
     }
+
+   @Test
+   public void replaceLineFromResults() {
+	   Map<String, String> mapExample = new HashMap<String, String>();
+	   mapExample.put("key1", "paramA");
+	   mapExample.put("key2", "paramB");
+	   OnapCommandUtils.replaceLineFromResults("line $r{paramA} line $r{paramB}", mapExample);
+	   assertTrue(mapExample.get("key1").equals("paramA"));
+	   assertTrue(mapExample.get("key2").equals("paramB"));
+   }
+
 }