From 145055933d4231d4e3d91f0188f73b484507c056 Mon Sep 17 00:00:00 2001
From: Nicholas Azar <NicholasAzar@users.noreply.github.com>
Date: Mon, 18 Feb 2019 16:25:32 -0500
Subject: [PATCH] Propagate exception to impact return code of command (#229)

* Propagate exception to impact return code of command

* Remove unnecessary try/catch
---
 .../main/java/com/networknt/codegen/Cli.java  | 54 +++++++++----------
 1 file changed, 25 insertions(+), 29 deletions(-)

diff --git a/codegen-cli/src/main/java/com/networknt/codegen/Cli.java b/codegen-cli/src/main/java/com/networknt/codegen/Cli.java
index 507d2bd55..cbba5db2c 100644
--- a/codegen-cli/src/main/java/com/networknt/codegen/Cli.java
+++ b/codegen-cli/src/main/java/com/networknt/codegen/Cli.java
@@ -40,7 +40,7 @@ public class Cli {
     @Parameter(names={"--help", "-h"}, help = true)
     private boolean help;
 
-    public static void main(String ... argv) {
+    public static void main(String ... argv) throws Exception {
         try {
             Cli cli = new Cli();
             JCommander jCommander = JCommander.newBuilder()
@@ -55,7 +55,7 @@ public static void main(String ... argv) {
         }
     }
 
-    public void run(JCommander jCommander) {
+    public void run(JCommander jCommander) throws Exception {
         if (help) {
             jCommander.usage();
             return;
@@ -67,39 +67,35 @@ public void run(JCommander jCommander) {
         Set<String> frameworks = registry.getFrameworks();
         if(frameworks.contains(framework)) {
             Generator generator = registry.getGenerator(framework);
-            try {
-                Object anyModel = null;
-                // model can be empty in some cases.
-                if(model != null) {
-                    // check if model is json or not before loading.
-                    if(model.endsWith("json")) {
-                        if(isUrl(model)) {
-                            anyModel = JsonIterator.deserialize(urlToByteArray(new URL(model)));
-                        } else {
-                            anyModel = JsonIterator.deserialize(Files.readAllBytes(Paths.get(model)));
-                        }
+            Object anyModel = null;
+            // model can be empty in some cases.
+            if(model != null) {
+                // check if model is json or not before loading.
+                if(model.endsWith("json")) {
+                    if(isUrl(model)) {
+                        anyModel = JsonIterator.deserialize(urlToByteArray(new URL(model)));
                     } else {
-                        if(isUrl(model)) {
-                            anyModel = new String(urlToByteArray(new URL(model)), StandardCharsets.UTF_8);
-                        } else {
-                            anyModel = new String(Files.readAllBytes(Paths.get(model)), StandardCharsets.UTF_8);
-                        }
+                        anyModel = JsonIterator.deserialize(Files.readAllBytes(Paths.get(model)));
                     }
-                }
-
-                Any anyConfig = null;
-                if(config != null) {
-                    if(isUrl(config)) {
-                        anyConfig = JsonIterator.deserialize(urlToByteArray(new URL(config)));
+                } else {
+                    if(isUrl(model)) {
+                        anyModel = new String(urlToByteArray(new URL(model)), StandardCharsets.UTF_8);
                     } else {
-                        anyConfig = JsonIterator.deserialize(Files.readAllBytes(Paths.get(config)));
+                        anyModel = new String(Files.readAllBytes(Paths.get(model)), StandardCharsets.UTF_8);
                     }
                 }
-                generator.generate(output, anyModel, anyConfig);
-                System.out.println("A project has been generated successfully in " + output + " folder. Have fun!!!");
-            } catch (Exception e) {
-                e.printStackTrace();
             }
+
+            Any anyConfig = null;
+            if(config != null) {
+                if(isUrl(config)) {
+                    anyConfig = JsonIterator.deserialize(urlToByteArray(new URL(config)));
+                } else {
+                    anyConfig = JsonIterator.deserialize(Files.readAllBytes(Paths.get(config)));
+                }
+            }
+            generator.generate(output, anyModel, anyConfig);
+            System.out.println("A project has been generated successfully in " + output + " folder. Have fun!!!");
         } else {
             System.out.printf("Invalid framework: %s\navaliable frameworks:\n", framework);
             for(String frm : frameworks) {