fix: modify unit test

packages/spacecat-shared-http-utils/test/auth/authentication-manager.test.js

@@ -24,13 +24,20 @@ chai.use(chaiAsPromised);
 
 const { expect } = chai;
 
-const createHandler = (name, shouldAuthenticate) => class extends AbstractHandler {
+const createHandler = (
+  name,
+  shouldAuthenticate,
+  shouldThrowError = false,
+) => class extends AbstractHandler {
   constructor(log) {
     super(name, log);
   }
 
   // eslint-disable-next-line @typescript-eslint/no-unused-vars,class-methods-use-this
   async checkAuth(request, ctx) {
+    if (shouldThrowError) {
+      throw new Error('Authentication error');
+    }
     return shouldAuthenticate ? { user: 'testUser' } : null;
   }
 };
@@ -58,6 +65,17 @@ describe('AuthenticationManager', () => {
     expect(() => AuthenticationManager.create([], logStub)).to.throw('No handlers provided');
   });
 
+  it('handles errors thrown by checkAuth', async () => {
+    const ErrorAuthHandler = createHandler('ErrorAuthHandler', false, true);
+    const manager = AuthenticationManager.create([ErrorAuthHandler], logStub);
+    const request = {};
+    const context = {};
+
+    await expect(manager.authenticate(request, context)).to.be.rejectedWith(NotAuthenticatedError);
+    expect(logStub.error.calledWith('Failed to authenticate with ErrorAuthHandler: Authentication error')).to.be.true;
+    expect(logStub.info.calledWith('No authentication handler was able to authenticate the request')).to.be.true;
+  });
+
   it('creates an instance with registered handlers', () => {
     const manager = AuthenticationManager.create([DummyHandler], logStub);
     expect(manager).to.be.instanceof(AuthenticationManager);