From 3107992db616ae37f0bfd206a6c94dda45b33b73 Mon Sep 17 00:00:00 2001
From: EvgenBabenko <evgen.babenko@gmail.com>
Date: Wed, 22 Sep 2021 09:15:50 +0300
Subject: [PATCH 1/2] fix(reject): fix reject and exit with code 1 if error

---
 index.js       |  5 +++++
 src/swagger.js | 10 ++++++++--
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/index.js b/index.js
index a42b61a50..3de6f7b33 100755
--- a/index.js
+++ b/index.js
@@ -137,4 +137,9 @@ generateApi({
   silent: !!silent,
   typePrefix,
   typeSuffix,
+}).catch((err) => {
+  // NOTE collect all errors on top level and shows to users in any case
+  console.log(err);
+
+  process.exit(1);
 });
diff --git a/src/swagger.js b/src/swagger.js
index 80c16ba15..88f53fa91 100644
--- a/src/swagger.js
+++ b/src/swagger.js
@@ -18,7 +18,7 @@ const parseSwaggerFile = (file) => {
 };
 
 const getSwaggerFile = (pathToSwagger, urlToSwagger, disableStrictSSL, disableProxy) =>
-  new Promise((resolve) => {
+  new Promise((resolve, reject) => {
     if (pathIsExist(pathToSwagger)) {
       logger.log(`try to get swagger by path "${pathToSwagger}"`);
       resolve(getFileContent(pathToSwagger));
@@ -38,7 +38,13 @@ const getSwaggerFile = (pathToSwagger, urlToSwagger, disableStrictSSL, disablePr
       axios
         .get(urlToSwagger, axiosOptions)
         .then((res) => resolve(res.data))
-        .catch((err) => logger.error(`error while getting swagger by URL ${urlToSwagger}:`, err));
+        .catch(() => {
+          const message = `error while getting swagger by URL ${urlToSwagger}`;
+
+          logger.error(message);
+
+          reject(message);
+        });
     }
   });
 

From baa53ed8d46e2df89095215648524b743f782047 Mon Sep 17 00:00:00 2001
From: EvgenBabenko <evgen.babenko@gmail.com>
Date: Thu, 23 Sep 2021 10:13:16 +0300
Subject: [PATCH 2/2] changed log to error

---
 index.js | 23 +++++------------------
 1 file changed, 5 insertions(+), 18 deletions(-)

diff --git a/index.js b/index.js
index 709cea970..32c9e7a1c 100755
--- a/index.js
+++ b/index.js
@@ -33,29 +33,20 @@ program
   )
   .option(
     "-r, --responses",
-    "generate additional information about request responses\n" +
-      "also add typings for bad responses",
+    "generate additional information about request responses\n" + "also add typings for bad responses",
     false,
   )
   .option("--union-enums", 'generate all "enum" types as union types (T1 | T2 | TN)', false)
   .option("--route-types", "generate type definitions for API routes", false)
   .option("--no-client", "do not generate an API class", false)
-  .option(
-    "--enum-names-as-values",
-    "use values in 'x-enumNames' as enum values (not only as keys)",
-    false,
-  )
+  .option("--enum-names-as-values", "use values in 'x-enumNames' as enum values (not only as keys)", false)
   .option(
     "--extract-request-params",
     "extract request params to data contract (Also combine path params and query params into one object)",
     false,
   )
   .option("--extract-request-body", "extract request body type to data contract", false)
-  .option(
-    "--modular",
-    "generate separated files for http client, data contracts, and routes",
-    false,
-  )
+  .option("--modular", "generate separated files for http client, data contracts, and routes", false)
   .option("--js", "generate js api module with declaration file", false)
   .option(
     "--module-name-index <number>",
@@ -72,11 +63,7 @@ program
   .option("--default-response <type>", "default type for empty response schema", TS_KEYWORDS.VOID)
   .option("--type-prefix <string>", "data contract name prefix", "")
   .option("--type-suffix <string>", "data contract name suffix", "")
-  .option(
-    "--clean-output",
-    "clean output folder before generate api. WARNING: May cause data loss",
-    false,
-  )
+  .option("--clean-output", "clean output folder before generate api. WARNING: May cause data loss", false)
   .option("--patch", "fix up small errors in the swagger source definition", false);
 
 program.parse(process.argv);
@@ -142,7 +129,7 @@ generateApi({
   patch: !!patch,
 }).catch((err) => {
   // NOTE collect all errors on top level and shows to users in any case
-  console.log(err);
+  console.error(err);
 
   process.exit(1);
 });