Add readme tag support

Author: alvadb

lib/commands/oad.js

@@ -15,6 +15,14 @@ exports.builder = {
     describe: 'A boolean flag indicating whether output format of the messages is json.',
     boolean: true,
     default: true
+  },
+  o: {
+    alias: 'old-tag-name',
+    describe: 'The tag name for the readme.  If include also indicates that the old spec file is a readme file',
+  },
+  n: {
+    alias: 'new-tag-name',
+    describe: 'The tag name for the readme.  If include also indicates that the old spec file is a readme file',
   }
 };
 
@@ -26,6 +34,8 @@ exports.handler = function (argv) {
   vOptions.consoleLogLevel = argv.logLevel;
   vOptions.logFilepath = argv.f;
   vOptions.json = argv.j;
+  vOptions.oldTagName = argv.o;
+  vOptions.newTagName = argv.n;
 
   return validate.compare(oldSpec, newSpec, vOptions).then((result) => {
     console.log(result);

lib/validators/openApiDiff.js

@@ -56,8 +56,8 @@ class OpenApiDiff {
     log.silly(`compare is being called`);
 
     let self = this;
-    var promise1 = self.processViaAutoRest(oldSwagger, 'old');
-    var promise2 = self.processViaAutoRest(newSwagger, 'new');
+    var promise1 = self.processViaAutoRest(oldSwagger, 'old', self.options.oldTagName);
+    var promise2 = self.processViaAutoRest(newSwagger, 'new', self.options.newTagName);
 
     return Promise.all([promise1, promise2]).then(results => {
       return self.processViaOpenApiDiff(results[0], results[1]);
@@ -119,7 +119,7 @@ class OpenApiDiff {
    * @param {string} outputFileName Name of the output file to which autorest outputs swagger-doc.
    *
    */
-  processViaAutoRest(swaggerPath, outputFileName) {
+  processViaAutoRest(swaggerPath, outputFileName, tagName) {
     log.silly(`processViaAutoRest is being called`);
 
     let self = this;
@@ -141,7 +141,12 @@ class OpenApiDiff {
 
       let outputFolder = os.tmpdir();
       let outputFilePath = path.join(outputFolder, `${outputFileName}.json`);
-      let autoRestCmd = `${self.autoRestPath()} --input-file=${swaggerPath} --output-artifact=swagger-document.json --output-file=${outputFileName} --output-folder=${outputFolder}`;
+      var autoRestCmd;
+      if(tagName) {
+        autoRestCmd = `${self.autoRestPath()} ${swaggerPath} --tag=${tagName} --output-artifact=swagger-document.json --output-file=${outputFileName} --output-folder=${outputFolder}`;
+      }else {
+        autoRestCmd = `${self.autoRestPath()} --input-file=${swaggerPath} --output-artifact=swagger-document.json --output-file=${outputFileName} --output-folder=${outputFolder}`;
+      }
 
       log.debug(`Executing: "${autoRestCmd}"`);
 

openapi-diff/src/modeler/AutoRest.Swagger/Model/ServiceDefinition.cs

@@ -124,7 +124,10 @@ public override IEnumerable<ComparisonMessage> Compare(ComparisonContext context
             if (previousDefinition == null)
                 throw new ArgumentException("Comparing a service definition with something else.");
 
-            if (Info != null && previousDefinition.Info != null)
+            if (Info != null && 
+                previousDefinition.Info != null &&
+                Info.Version != null && 
+                previousDefinition.Info.Version != null)
             {
                 context.PushProperty("info");
                 context.PushProperty("version");