Add some safety checks in get-auth incase there's no auth

readmeio · Jan 28, 2019 · aea2002 · aea2002
1 parent 7937b2a
commit aea2002
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/packages/api-explorer/__tests__/lib/get-auth.test.js b/packages/api-explorer/__tests__/lib/get-auth.test.js
@@ -15,6 +15,16 @@ it('should fetch all auths from the OAS files', () => {
   });
 });
 
+it('should not error if oas.components is not set', () => {
+  expect(() => {
+    getAuth({ oauthScheme: 'oauth', apiKeyScheme: 'apikey' }, { 'api-setting': {} })
+  }).not.toThrow();
+
+  expect(() => {
+    getAuth({ oauthScheme: 'oauth', apiKeyScheme: 'apikey' }, { 'api-setting': { components: {} } })
+  }).not.toThrow();
+});
+
 const { getSingle } = getAuth;
 
 const topLevelUser = { apiKey: '123456', user: 'user', pass: 'pass' };

diff --git a/packages/api-explorer/src/lib/get-auth.js b/packages/api-explorer/src/lib/get-auth.js
@@ -23,6 +23,9 @@ function getAuth(user, oasFiles) {
   return Object.keys(oasFiles)
     .map(id => {
       const oas = oasFiles[id];
+
+      if (Object.keys(oas.components || {}).length === 0) return {};
+
       return Object.keys(oas.components.securitySchemes)
         .map(scheme => {
           return {