fix: support empty auth options

joolfe · May 18, 2021 · 704ef36 · 704ef36
1 parent 4d9b400
commit 704ef36
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 4 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -253,10 +253,12 @@ function parseOptsAuth (optAuth) {
       security.push({ [secName]: [] })
     }
   }
-  return {
-    components: { securitySchemes },
-    security
-  }
+  return Object.keys(securitySchemes).length === 0
+    ? {}
+    : {
+        components: { securitySchemes },
+        security
+      }
 }
 
 /* From the path array compose the real path for OpenApi specs */
@@ -291,6 +293,7 @@ function scrapeURL (url) {
 function parseServers (domains, serversOpts) {
   let servers
   if (serversOpts != null) {
+    // This map is just to filter not supported fields while no validations are implemented
     servers = serversOpts.map(({ url, description }) => ({ url, description }))
   } else {
     servers = Array.from(domains).map(domain => ({ url: domain }))

diff --git a/test/index.spec.js b/test/index.spec.js
@@ -221,6 +221,14 @@ describe('Library specs', function () {
         equal(result, EXPECTED_BASIC)
       })
 
+      it('should not fail if auth is empty object', async function () {
+        const result = await postmanToOpenApi(COLLECTION_BASIC, OUTPUT_PATH,
+          {
+            auth: {}
+          })
+        equal(result, EXPECTED_BASIC)
+      })
+
       it('should use depth configuration for parse paths', async function () {
         const result = await postmanToOpenApi(COLLECTION_DEPTH_PATH_PARAMS, OUTPUT_PATH, { pathDepth: 1 })
         equal(result, EXPECTED_DEPTH_PATH_PARAMS)