Update logger usage in tests for clarity

Refactored tests to spy on the correct logger instance in plugin-bootstrap service tests and updated logger.warn calls in auth-middleware tests to use a more descriptive log message format. These changes improve test accuracy and log message clarity.

Author: wtfsayo

packages/plugin-bootstrap/src/__tests__/services.test.ts

@@ -209,7 +209,9 @@ describe('TaskService', () => {
       }
       return undefined;
     }) as any;
-    spyOn(logger, 'error').mockImplementation(() => {}); // Suppress error logging for this test
+
+    // Spy on runtime logger instead of global logger
+    const loggerErrorSpy = spyOn(mockRuntime.logger, 'error').mockImplementation(() => { });
 
     // Expose the private method for testing
     const executeTaskMethod = (taskService as any).executeTask.bind(taskService);
@@ -221,8 +223,8 @@ describe('TaskService', () => {
     expect(mockRuntime.getTaskWorker).toHaveBeenCalledWith(testTask.name);
     expect(mockErrorExecute).toHaveBeenCalled();
 
-    // Verify error was logged
-    expect(logger.error).toHaveBeenCalledWith(
+    // Verify error was logged on runtime.logger
+    expect(loggerErrorSpy).toHaveBeenCalledWith(
       { src: 'plugin:bootstrap:service:task', agentId: mockRuntime.agentId, taskId: testTask.id, error: expect.any(String) },
       'Error executing task'
     );
@@ -254,7 +256,7 @@ describe('Service Registry', () => {
 
   beforeEach(() => {
     mock.restore();
-    spyOn(logger, 'warn').mockImplementation(() => {});
+    spyOn(logger, 'warn').mockImplementation(() => { });
 
     // Use setupActionTest for consistent test setup
     const setup = setupActionTest();

packages/server/src/__tests__/unit/middleware/auth-middleware.test.ts

@@ -17,7 +17,7 @@ describe('API Key Auth Middleware', () => {
   beforeEach(() => {
     // Reset environment
     process.env = { ...originalEnv };
-    
+
     // Spy on logger.warn
     loggerWarnSpy = spyOn(logger, 'warn');
 
@@ -105,8 +105,7 @@ describe('API Key Auth Middleware', () => {
         message: 'Missing or invalid X-API-KEY header',
       });
       expect(loggerWarnSpy).toHaveBeenCalledWith(
-        { src: 'http', ip: '127.0.0.1' },
-        'Unauthorized access attempt'
+        '[API Key] Unauthorized access attempt from 127.0.0.1'
       );
     });
 
@@ -122,8 +121,7 @@ describe('API Key Auth Middleware', () => {
         message: 'Missing or invalid X-API-KEY header',
       });
       expect(loggerWarnSpy).toHaveBeenCalledWith(
-        { src: 'http', ip: '127.0.0.1' },
-        'Unauthorized access attempt'
+        '[API Key] Unauthorized access attempt from 127.0.0.1'
       );
     });
 
@@ -184,8 +182,7 @@ describe('API Key Auth Middleware', () => {
       apiKeyAuthMiddleware(mockRequest as Request, mockResponse as Response, mockNext);
 
       expect(loggerWarnSpy).toHaveBeenCalledWith(
-        { src: 'http', ip: '192.168.1.100' },
-        'Unauthorized access attempt'
+        '[API Key] Unauthorized access attempt from 192.168.1.100'
       );
     });
   });