Merge pull request #90 from vishrutshah/chain-result

Chain result upto compare method

CHANGELOG.md

@@ -1,6 +1,10 @@
+## 0.1.7
+Released on 2017-08-10.
+- Chaining the promises upto `compare` method and gracefully exiting application [#88](https://github.com/Azure/openapi-diff/issues/88)
+
 ## 0.1.x
 Released on 2017-07-18. All issues associated with this release can be found using this filter [Sprint-103](https://github.com/Azure/openapi-diff/issues?q=label%3ASprint-103+is%3Aclosed) [Sprint-104](https://github.com/Azure/openapi-diff/issues?utf8=%E2%9C%93&q=label%3ASprint-104%20is%3Aclosed)
 
 #### Added
 - Initial release of oad.
-  - Added by [Vishrut Shah](https://github.com/vishrutshah).
+  - Added by [Vishrut Shah](https://github.com/vishrutshah).

README.md

@@ -1,6 +1,6 @@
 ### How to install
 ```
-npm install -g [email protected].6
+npm install -g [email protected].7
 ```
 
 ### Command Usage

lib/commands/oad.js

@@ -27,7 +27,12 @@ exports.handler = function (argv) {
   vOptions.logFilepath = argv.f;
   vOptions.json = argv.j;
 
-  return validate.compare(oldSpec, newSpec, vOptions);
+  return validate.compare(oldSpec, newSpec, vOptions).then((result) => {
+    console.log(result);
+  }).catch(err => {
+    console.log(err);
+    process.exitCode = 1;
+  });
 }
 
 exports = module.exports;

lib/validators/openApiDiff.js

@@ -60,12 +60,7 @@ class OpenApiDiff {
     var promise2 = self.processViaAutoRest(newSwagger, 'new');
 
     return Promise.all([promise1, promise2]).then(results => {
-      return self.processViaOpenApiDiff(results[0], results[1]).then((result, error) => {
-        console.log(result);
-      })
-    }).catch(err => {
-      console.log(err);
-      process.exit(1);
+      return self.processViaOpenApiDiff(results[0], results[1]);
     });
   }
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "oad",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "author": {
     "name": "Microsoft Corporation",
     "email": "[email protected]",