Merge pull request #194 from davolokh/console-error-in-test-removed

console.error pollution logs removed in tests

Author: jspahrsummers

src/client/sse.test.ts

@@ -56,6 +56,8 @@ describe("SSEClientTransport", () => {
       baseUrl = new URL(`http://127.0.0.1:${addr.port}`);
       done();
     });
+
+    jest.spyOn(console, 'error').mockImplementation(() => {});
   });
 
   afterEach(async () => {

src/server/auth/middleware/bearerAuth.test.ts

@@ -31,6 +31,10 @@ describe("requireBearerAuth middleware", () => {
       set: jest.fn().mockReturnThis(),
     };
     nextFunction = jest.fn();
+    jest.spyOn(console, 'error').mockImplementation(() => {});
+  })
+
+  afterEach(() => {
     jest.clearAllMocks();
   });
 

src/server/auth/router.test.ts

@@ -259,6 +259,11 @@ describe('MCP Auth Router', () => {
         issuerUrl: new URL('https://auth.example.com')
       };
       app.use(mcpAuthRouter(options));
+      jest.spyOn(console, 'error').mockImplementation(() => {});
+    });
+
+    afterEach(() => {
+      jest.restoreAllMocks();
     });
 
     it('routes to authorization endpoint', async () => {