fix: fail when collection don't have ´variable´ parameter

joolfe · Aug 5, 2021 · bb02ad6 · bb02ad6
1 parent d48517a
commit bb02ad6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/var-replacer.js b/lib/var-replacer.js
@@ -10,7 +10,7 @@ Mustache.Writer.prototype.escapedValue = function escapedValue (token, context,
 
 function replacePostmanVariables (collectionString, additionalVars = {}) {
   const postmanJson = JSON.parse(collectionString)
-  const { variable } = postmanJson
+  const { variable = [] } = postmanJson
   const formatVars = variable.reduce((obj, { key, value }) => {
     obj[key] = value
     return obj

diff --git a/test/index.spec.js b/test/index.spec.js
@@ -428,6 +428,11 @@ describe('Library specs', function () {
         })
         equal(result, EXPECTED_VARIABLES_ADDITIONAL)
       })
+
+      it('should not fail if no variable are defined and want to replace', async function () {
+        const result = await postmanToOpenApi(COLLECTION_FORM_DATA, OUTPUT_PATH, { replaceVars: true })
+        equal(result, EXPECTED_FORM_DATA)
+      })
     })
   })