fix: fix failing tests

Author: jescalada

src/config/index.ts

@@ -110,7 +110,7 @@ export const getAPIAuthMethods = (): Authentication[] => {
   const enabledAuthMethods = _apiAuthentication.filter(auth => auth.enabled);
 
   if (enabledAuthMethods.length === 0) {
-    throw new Error("No authentication method enabled.");
+    console.log("Warning: No authentication method enabled for API endpoints.");
   }
 
   return enabledAuthMethods;

src/service/passport/jwtAuthHandler.js

@@ -13,7 +13,7 @@ const jwtAuthHandler = (overrideConfig = null) => {
           : require('../../config').getAPIAuthMethods();
 
     const jwtAuthMethod = apiAuthMethods.find((method) => method.type.toLowerCase() === "jwt");
-      if (!jwtAuthMethod) {
+      if (!overrideConfig && (!jwtAuthMethod || !jwtAuthMethod.enabled)) {
           return next();
       }
 

test/testJwtAuthHandler.test.js

@@ -146,7 +146,7 @@ describe('jwtAuthHandler', () => {
 
   it('should return 401 if no token provided', async () => {
     req.header.returns(null);
-    await jwtAuthHandler()(req, res, next);
+    await jwtAuthHandler(jwtConfig)(req, res, next);
 
     expect(res.status.calledWith(401)).to.be.true;
     expect(res.send.calledWith('No token provided\n')).to.be.true;