From: Kanagaraj Manickam k00365106 <kanagaraj.manickam@huawei.com>
Date: Mon, 12 Feb 2018 07:58:06 +0000 (+0530)
Subject: Handle CLI warning effectively
X-Git-Tag: v2.0.0^0
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=7ed67d67439e5945302bbab030a9627193de6616;p=cli.git

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 bd910748..c5ebbc21 100644
--- 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 62b78fd5..b8af6f51 100644
--- 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();
             }
         }
     }