Updates from feedback.

Author: alvadb

index.js

@@ -9,6 +9,7 @@ var utils = require('./lib/util/utils');
 // Easy to use methods from validate.js
 exports.log = require('./lib/util/logging');
 exports.compare = validate.compare;
+exports.compareTags = validate.compareTags;
 
 // Classes
 exports.OpenApiDiff = require('./lib/validators/openApiDiff');

lib/validate.js

@@ -57,5 +57,5 @@ exports.compareTags = function (oldSwagger, oldTag, newSwagger, newTag, options)
   log.filepath = options.logFilepath || log.filepath;
   let openApiDiff = new OpenApiDiff(options);
 
-  return openApiDiff.compareTags(oldSwagger, oldTag, newSwagger, newTag);
+  return openApiDiff.compare(oldSwagger, newSwagger, oldTag, newTag);
 };

lib/validators/openApiDiff.js

@@ -51,32 +51,12 @@ class OpenApiDiff {
    *
    * @param {string} newSwagger Path to the new specification file.
    *
-   */
-  compare(oldSwagger, newSwagger) {
-    log.silly(`compare is being called`);
-
-    let self = this;
-    var promise1 = self.processViaAutoRest(oldSwagger, 'old');
-    var promise2 = self.processViaAutoRest(newSwagger, 'new');
-
-    return Promise.all([promise1, promise2]).then(results => {
-      return self.processViaOpenApiDiff(results[0], results[1]);
-    });
-  }
-
-  /**
-   * Compares old and new specifications.
-   *
-   * @param {string} oldSwagger Path to the old specification file.
-   *
-   * @param {string} [oldTag] Tag name used for autorest with the old specification file.
-   *
-   * @param {string} newSwagger Path to the new specification file.
+   * @param {string} oldTag Tag name used for autorest with the old specification file.
    *
-   * @param {string} [newTag] Tag name used for autorest with the new specification file.
+   * @param {string} newTag Tag name used for autorest with the new specification file.
    *
    */
-  compareTags(oldSwagger, oldTag, newSwagger, newTag) {
+  compare(oldSwagger, newSwagger, oldTag, newTag) {
     log.silly(`compare is being called`);
 
     let self = this;