Merge branch 'release/4.1.0' of github.com:postmanlabs/openapi-to-postman

Author: Vishal Shingala

.npmignore

@@ -43,4 +43,16 @@ test/data/.temp
 
 # Prevent unit test coverage reports from being added
 .coverage
-.nyc_output
+.nyc_output
+
+# - Prevent config and test files from being added
+.git*
+.github/
+scripts/
+test/
+examples/
+.eslintrc
+.eslintignore
+.nycrc
+.editorconfig
+.jsdoc-config.json

CHANGELOG.md

@@ -1,5 +1,16 @@
 # OpenAPI-Postman Changelog
 
+#### v4.1.0 (July 20, 2022)
+* Fixed issue where conversion was failing for definitions with info object as null.
+* Fixed issue where generated collection did not have correct examples value from XML type of content.
+* Fixed issue [#518](https://github.com/postmanlabs/openapi-to-postman/issues/518) where generated collection were having NaN as value for integer query params with enum values.
+* Fixed issue [#496](https://github.com/postmanlabs/openapi-to-postman/issues/496) where validateTransactions() was returning missing endpoints even though corresponding requests are present in collection.
+* Fixed issue [#478](https://github.com/postmanlabs/openapi-to-postman/issues/478) where updation of path parameter in collection resulted in MISSING_IN_REQUEST error.
+* Fixed issue [#559](https://github.com/postmanlabs/openapi-to-postman/issues/559) where parameter description was undefined for formdata type of content even if defined.
+* Fixed issue where bundle() API didn't handle circular references correctly.
+* Added non-required files to be published as npm module in .npmignore
+* Fixed issue where disableBodyPruning option was not set for requests with no request body.
+
 #### v4.0.0 (July 12, 2022)
 * Added support for new multi-file API detectRootFiles() for OpenAPI 3 and Swagger 2 formats to support detection of root files among multiple files.
 * Added support for new multi-file API detectRelatedFiles() for OpenAPI 3 and Swagger 2 formats to support detection of related files for a provided root file amongst multiple files.

assets/json-schema-faker.js

@@ -23782,7 +23782,7 @@ function extend() {
                   var min = Math.max(params.minimum || 0, 0);
                   var max = Math.min(params.maximum || Infinity, Infinity);
                   min = handleExclusiveMinimum(schema, min);
-                  max = handleExclusiveMaximum(schema, min);
+                  max = handleExclusiveMaximum(schema, max);
                   // discard out-of-bounds enumerations
                   schema.enum = schema.enum.filter(function (x) {
                       if (x >= min && x <= max) {

lib/bundle.js

@@ -22,7 +22,7 @@ let path = require('path'),
   { DFS } = require('./dfs'),
   deref = require('./deref.js'),
   { isSwagger, getBundleRulesDataByVersion } = require('./common/versionUtils'),
-  CIRCULAR_REF_EXT_PROP = 'x-circularRef';
+  CIRCULAR_OR_REF_EXT_PROP = 'x-orRef';
 
 
 /**
@@ -481,6 +481,34 @@ function findReferenceByMainKeyInTraceFromContext(documentContext, mainKeyInTrac
   return relatedRef;
 }
 
+/**
+ * Verifies if a node has same content as one of the parents so it is a circular ref
+ * @param {function} traverseContext - The context of the traverse function
+ * @param {object} contentFromTrace -  The resolved content of the node to deref
+ * @returns {boolean} whether is circular reference or not.
+ */
+function isCircularReference(traverseContext, contentFromTrace) {
+  return traverseContext.parents.find((parent) => { return parent.node === contentFromTrace; }) !== undefined;
+}
+
+/**
+ * Modifies content of a node if it is circular reference.
+ *
+ * @param {function} traverseContext - The context of the traverse function
+ * @param {object} documentContext The document context from root
+ * @returns {undefined}  nothing
+ */
+function handleCircularReference(traverseContext, documentContext) {
+  let relatedRef = '';
+  if (traverseContext.circular) {
+    relatedRef = findReferenceByMainKeyInTraceFromContext(documentContext, traverseContext.circular.key);
+    traverseContext.update({ $ref: relatedRef });
+  }
+  if (traverseContext.keys && traverseContext.keys.includes(CIRCULAR_OR_REF_EXT_PROP)) {
+    traverseContext.update({ $ref: traverseContext.node[CIRCULAR_OR_REF_EXT_PROP] });
+  }
+}
+
 /**
  * Generates the components object from the documentContext data
  * @param {object} documentContext The document context from root
@@ -492,8 +520,9 @@ function findReferenceByMainKeyInTraceFromContext(documentContext, mainKeyInTrac
  */
 function generateComponentsObject (documentContext, rootContent, refTypeResolver, components, version) {
   let notInLine = Object.entries(documentContext.globalReferences).filter(([, value]) => {
-    return value.keyInComponents.length !== 0;
-  });
+      return value.keyInComponents.length !== 0;
+    }),
+    circularRefsSet = new Set();
   const { COMPONENTS_KEYS } = getBundleRulesDataByVersion(version);
   notInLine.forEach(([key, value]) => {
     let [, partial] = key.split(localPointer);
@@ -541,6 +570,17 @@ function generateComponentsObject (documentContext, rootContent, refTypeResolver
             if (!contentFromTrace) {
               refData.nodeContent = { $ref: `${localPointer + local}` };
             }
+            else if (isCircularReference(this, contentFromTrace)) {
+              if (refData.inline) {
+                refData.nodeContent = { [CIRCULAR_OR_REF_EXT_PROP]: tempRef };
+                circularRefsSet.add(tempRef);
+              }
+              else {
+                refData.node = { [CIRCULAR_OR_REF_EXT_PROP]: refData.reference };
+                refData.nodeContent = contentFromTrace;
+                circularRefsSet.add(refData.reference);
+              }
+            }
             else {
               refData.nodeContent = contentFromTrace;
             }
@@ -551,39 +591,30 @@ function generateComponentsObject (documentContext, rootContent, refTypeResolver
             refData.node = hasSiblings ?
               _.merge(referenceSiblings, refData.nodeContent) :
               refData.nodeContent;
-            documentContext.globalReferences[property.$ref].reference =
+            documentContext.globalReferences[tempRef].reference =
               resolveJsonPointerInlineNodes(this.parents, this.key);
           }
-          this.update(refData.node);
-          if (!refData.inline) {
-            if (documentContext.globalReferences[tempRef].refHasContent) {
-              setValueInComponents(
-                refData.keyInComponents,
-                components,
-                refData.nodeContent,
-                COMPONENTS_KEYS
-              );
-            }
+          else if (refData.refHasContent) {
+            setValueInComponents(
+              refData.keyInComponents,
+              components,
+              refData.nodeContent,
+              COMPONENTS_KEYS
+            );
           }
+          this.update(refData.node);
         }
       }
     });
   });
   return {
     resRoot: traverseUtility(rootContent).map(function () {
-      let relatedRef = '';
-      if (this.circular) {
-        relatedRef = findReferenceByMainKeyInTraceFromContext(documentContext, this.circular.key);
-        this.update({ $ref: relatedRef, [CIRCULAR_REF_EXT_PROP]: true });
-      }
+      handleCircularReference(this, documentContext);
     }),
     newComponents: traverseUtility(components).map(function () {
-      let relatedRef = '';
-      if (this.circular) {
-        relatedRef = findReferenceByMainKeyInTraceFromContext(documentContext, this.circular.key);
-        this.update({ $ref: relatedRef, [CIRCULAR_REF_EXT_PROP]: true });
-      }
-    })
+      handleCircularReference(this, documentContext);
+    }),
+    circularRefs: [...circularRefsSet]
   };
 }
 
@@ -618,7 +649,7 @@ function generateComponentsWrapper(parsedOasObject, version, nodesContent = {})
 }
 
 /**
- * Generates a map of generated refernce to the original reference
+ * Generates a map of generated reference to the original reference
  *
  * @param {object} globalReferences - Global references present at each root file context
  * @returns {object} reference map
@@ -712,7 +743,8 @@ module.exports = {
       fileContent: finalElements.resRoot,
       components: finalElements.newComponents,
       fileName: specRoot.fileName,
-      referenceMap: getReferenceMap(rootContextData.globalReferences)
+      referenceMap: getReferenceMap(rootContextData.globalReferences),
+      circularRefs: finalElements.circularRefs
     };
   },
   getReferences,

lib/common/schemaUtilsCommon.js

@@ -63,7 +63,7 @@ function removeWordFromLastPosition(schemaPath, word) {
  * "<integer>"
  *
  * @param {string} value - Value to check for
- * @param {string} type - The type in the schemna
+ * @param {string} type - The type in the schema
  * @returns {Boolean} the value is the representation of its type
  */
 function compareType(value, type) {
@@ -75,7 +75,7 @@ function compareType(value, type) {
  * "<integer>"
  * Works in array types
  * @param {string} value - Value to check for
- * @param {*} types - The types in the schemna
+ * @param {*} types - The types in the schema
  * @returns {Boolean} the value is the representation of its type
  */
 function isTypeValueArrayCheck(value, types) {
@@ -89,7 +89,7 @@ function isTypeValueArrayCheck(value, types) {
  * "<integer>"
  * Works in array types
  * @param {string} value - Value to check for
- * @param {*} types - The types in the schemna
+ * @param {*} types - The types in the schema
  * @returns {Boolean} the value is the representation of its type
  */
 function checkValueOnlyTypes(value, types) {
@@ -118,7 +118,7 @@ function getDefaultFromTypeAndFormat(type, format) {
  * @returns {Boolean} the value is the representation of its type
  */
 function checkValueTypesAndFormat(value, types, format) {
-  let typesNotInMapp = [],
+  let typesNotInMap = [],
     typesArray = Array.isArray(types) ? types : [types],
     found = typesArray.find((type) => {
       let defaultValue;
@@ -132,7 +132,7 @@ function checkValueTypesAndFormat(value, types, format) {
         }
       }
       else {
-        typesNotInMapp.push(type);
+        typesNotInMap.push(type);
       }
       return defaultValue === value;
     });
@@ -141,7 +141,7 @@ function checkValueTypesAndFormat(value, types, format) {
     return true;
   }
 
-  found = typesNotInMapp.find((type) => {
+  found = typesNotInMap.find((type) => {
     let defaultValue;
     defaultValue = '<' + type + (format ? ('-' + format) : '') + '>';
     return defaultValue === value;
@@ -185,7 +185,7 @@ function isTypeValue(value, schema) {
 }
 
 /**
- * Checks if value is correcta according to schema
+ * Checks if value is correct according to schema
  * If the value should be numeric, it tries to convert and then validate
  * also validates if the value is a correct representation in the form of
  * <long> or <integer> etc for integers format 32 or format 64
@@ -332,7 +332,7 @@ module.exports = {
   isTypeValue,
 
   /**
-   * Checks if value is correcta according to schema
+   * Checks if value is correct according to schema
    * If the value should be numeric, it tries to convert and then validate
    * also validates if the value is a correct representation in the form of
    * <long> or <integer> etc for integers format 32 or format 64
@@ -345,5 +345,16 @@ module.exports = {
 
   isKnownType: function(schema) {
     return typeof schemaTypeToJsValidator[schema.type] === 'function';
+  },
+
+  getServersPathVars: function(servers) {
+    return servers.reduce((acc, current) => {
+      const newVarNames = current.hasOwnProperty('variables') ?
+        Object.keys(current.variables).filter((varName) => {
+          return !acc.includes(varName);
+        }) :
+        [];
+      return [...acc, ...newVarNames];
+    }, []);
   }
 };