Handle CLI warning effectively

Issue-ID: CLI-92

Change-Id: If5ce9499763fdec5980c4cdd1839d5ef31ccdd49
Signed-off-by: Kanagaraj Manickam k00365106 <kanagaraj.manickam@huawei.com>

diff --git a/framework/src/main/java/org/onap/cli/fw/utils/OnapCommandUtils.java b/framework/src/main/java/org/onap/cli/fw/utils/OnapCommandUtils.java
index bd91074..c5ebbc2 100644 (file)
--- a/framework/src/main/java/org/onap/cli/fw/utils/OnapCommandUtils.java
+++ b/framework/src/main/java/org/onap/cli/fw/utils/OnapCommandUtils.java
@@ -288,8 +288,8 @@ public class OnapCommandUtils {
             for (String value: entry.getValue()) {
                 try {
                     value = replaceLineFromInputParameters(value, params);
-                } catch(OnapCommandResultEmpty e) {
-                    // pass // NOSONAR
+                } catch(OnapCommandResultEmpty e) {  // NOSONAR
+                    // pass
                 }
                 resultsProcessed.get(key).add(value);
             }

diff --git a/main/src/main/java/org/onap/cli/main/OnapCli.java b/main/src/main/java/org/onap/cli/main/OnapCli.java
index 62b78fd..b8af6f5 100644 (file)
--- a/main/src/main/java/org/onap/cli/main/OnapCli.java
+++ b/main/src/main/java/org/onap/cli/main/OnapCli.java
@@ -376,14 +376,13 @@ public class OnapCli {
                 this.exitSuccessfully();
 
                 generateSmapleYaml(cmd);
+            } catch (OnapCommandWarning w) {
+                this.print(cmd.getResult().getDebugInfo());
+                this.exitSuccessfully();
             } catch (Exception e) {
+                this.print(e);
                 this.print(cmd.getResult().getDebugInfo());
-                if (e instanceof OnapCommandWarning) {
-                    this.exitSuccessfully();
-                } else {
-                    this.print(e);
-                    this.exitFailure();
-                }
+                this.exitFailure();
             }
         }
     }