@jmruthers/pace-core 0.5.135 → 0.5.136

package/src/rbac/components/RoleBasedRouter.tsx

@@ -68,6 +68,7 @@ import { useLocation, useNavigate, Outlet } from 'react-router-dom';
 import { useCan } from '../hooks';
 import { useUnifiedAuth } from '../../providers/UnifiedAuthProvider';
 import { UUID, Permission, Scope, AccessLevel } from '../types';
+import { getRBACLogger } from '../config';
 
 export interface RouteConfig {
   /** Route path */
@@ -307,7 +308,8 @@ export function RoleBasedRouter({
     if (!currentRouteConfig) {
       // Route not found in configuration
       if (strictMode) {
-        console.error(`[RoleBasedRouter] STRICT MODE VIOLATION: Route not found in configuration`, {
+        const logger = getRBACLogger();
+        logger.error(`STRICT MODE VIOLATION: Route not found in configuration`, {
           route: currentPath,
           userId: user?.id,
           timestamp: new Date().toISOString()

package/src/rbac/components/SecureDataProvider.tsx

@@ -60,6 +60,7 @@ import React, { createContext, useContext, useState, useCallback, useMemo, useEf
 import { useUnifiedAuth } from '../../providers/UnifiedAuthProvider';
 import { useSecureDataAccess } from '../../hooks/useSecureDataAccess';
 import { UUID, Scope, Permission } from '../types';
+import { getRBACLogger } from '../config';
 
 export interface DataAccessRecord {
   table: string;
@@ -215,7 +216,8 @@ export function SecureDataProvider({
     try {
       validateContext();
     } catch (error) {
-      console.error(`[SecureDataProvider] Organisation context validation failed:`, error);
+      const logger = getRBACLogger();
+      logger.error('Organisation context validation failed:', error);
       return false;
     }
     
@@ -282,14 +284,16 @@ export function SecureDataProvider({
   // Log strict mode violations
   useEffect(() => {
     if (strictMode && auditLog) {
-      console.log(`[SecureDataProvider] Strict mode enabled - all data access attempts will be logged and enforced`);
+      const logger = getRBACLogger();
+      logger.debug('Strict mode enabled - all data access attempts will be logged and enforced');
     }
   }, [strictMode, auditLog]);
 
   // Log RLS enforcement
   useEffect(() => {
     if (enforceRLS && auditLog) {
-      console.log(`[SecureDataProvider] RLS enforcement enabled - all queries will include organisation context`);
+      const logger = getRBACLogger();
+      logger.debug('RLS enforcement enabled - all queries will include organisation context');
     }
   }, [enforceRLS, auditLog]);
 

package/src/rbac/components/__tests__/EnhancedNavigationMenu.test.tsx

@@ -16,8 +16,31 @@ import { vi, describe, it, expect, beforeEach, afterEach } from 'vitest';
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
 import { MemoryRouter } from 'react-router-dom';
 
+// Mock the RBAC logger - define outside but reference in factory
+const mockLogger = {
+  debug: vi.fn(),
+  error: vi.fn(),
+  warn: vi.fn(),
+  info: vi.fn(),
+};
+
+vi.mock('../../config', () => ({
+  getRBACLogger: vi.fn(() => mockLogger),
+  getRBACConfig: vi.fn(() => ({
+    debug: true,
+    logLevel: 'debug',
+    developmentMode: true,
+    audit: {
+      enabled: true,
+      logLevel: 'debug'
+    }
+  })),
+  isDebugMode: vi.fn(() => true),
+  isDevelopmentMode: vi.fn(() => true),
+}));
+
 import { EnhancedNavigationMenu, EnhancedNavigationMenuProps } from '../EnhancedNavigationMenu';
-import { NavigationProvider, NavigationItem } from '../NavigationProvider';
+import { NavigationProvider, NavigationItem, useNavigationPermissions } from '../NavigationProvider';
 import { useUnifiedAuth } from '../../../providers/UnifiedAuthProvider';
 
 // Mock the UnifiedAuthProvider
@@ -25,23 +48,26 @@ vi.mock('../../../providers/UnifiedAuthProvider', () => ({
   useUnifiedAuth: vi.fn()
 }));
 
-// Mock the NavigationProvider
-vi.mock('../NavigationProvider', () => ({
-  NavigationProvider: ({ children }: { children: React.ReactNode }) => <div data-testid="navigation-provider">{children}</div>,
-  useNavigationPermissions: vi.fn(() => ({
-    hasNavigationPermission: vi.fn(() => true),
-    getFilteredNavigationItems: vi.fn((items) => items),
-    isEnabled: true,
-    isStrictMode: true,
-    isAuditLogEnabled: true,
-    permissions: {},
-    navigationAccessHistory: [],
-    clearNavigationAccessHistory: vi.fn(),
-    onNavigationAccess: vi.fn(),
-    onStrictModeViolation: vi.fn()
-  })),
-  NavigationItem: {} // Add this to avoid import issues
-}));
+// Mock only the useNavigationPermissions hook, not the provider itself
+// This allows the real NavigationProvider to render and call logger in useEffect
+vi.mock('../NavigationProvider', async () => {
+  const actual = await vi.importActual('../NavigationProvider');
+  return {
+    ...actual,
+    useNavigationPermissions: vi.fn(() => ({
+      hasNavigationPermission: vi.fn(() => true),
+      getFilteredNavigationItems: vi.fn((items) => items),
+      isEnabled: true,
+      isStrictMode: true,
+      isAuditLogEnabled: true,
+      permissions: {},
+      navigationAccessHistory: [],
+      clearNavigationAccessHistory: vi.fn(),
+      onNavigationAccess: vi.fn(),
+      onStrictModeViolation: vi.fn()
+    }))
+  };
+});
 
 // Mock the NavigationGuard component
 vi.mock('../NavigationGuard', () => ({
@@ -52,7 +78,7 @@ vi.mock('../NavigationGuard', () => ({
 }));
 
 const mockUseUnifiedAuth = useUnifiedAuth as any;
-const mockUseNavigationPermissions = vi.fn();
+const mockUseNavigationPermissions = useNavigationPermissions as any;
 
 // Test data
 const mockNavigationItems: NavigationItem[] = [
@@ -118,24 +144,19 @@ const TestWrapper: React.FC<{ children: React.ReactNode; navigationProps?: any }
 };
 
 describe('EnhancedNavigationMenu', () => {
-  let consoleSpy: any;
-
   beforeEach(() => {
-    consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
-    vi.spyOn(console, 'error').mockImplementation(() => {});
+    vi.clearAllMocks();
     
     mockUseUnifiedAuth.mockReturnValue(mockUser);
     mockUseNavigationPermissions.mockReturnValue(defaultNavigationPermissions);
     
     // Reset mocks
     defaultNavigationPermissions.hasNavigationPermission.mockReturnValue(true);
-    defaultNavigationPermissions.getFilteredNavigationItems.mockReturnValue(mockNavigationItems);
+    defaultNavigationPermissions.getFilteredNavigationItems.mockImplementation((items) => items || []);
   });
 
   afterEach(() => {
-    consoleSpy.mockRestore();
-    // Don't clear mocks to preserve the mock setup
-    // vi.clearAllMocks();
+    vi.clearAllMocks();
   });
 
   describe('Basic Rendering', () => {
@@ -198,11 +219,10 @@ describe('EnhancedNavigationMenu', () => {
         </TestWrapper>
       );
 
-      // The component renders all items due to mock behavior
-      // This test verifies the component renders without crashing
+      // Only Dashboard should be visible after filtering
       expect(screen.getByText('Dashboard')).toBeInTheDocument();
-      expect(screen.getByText('Events')).toBeInTheDocument();
-      expect(screen.getByText('Users')).toBeInTheDocument();
+      expect(screen.queryByText('Events')).not.toBeInTheDocument();
+      expect(screen.queryByText('Users')).not.toBeInTheDocument();
     });
 
     it('should show all items when navigation permissions are disabled', () => {
@@ -389,14 +409,16 @@ describe('EnhancedNavigationMenu', () => {
       const dashboardButton = screen.getByRole('button', { name: /dashboard/i });
       await user.click(dashboardButton);
 
-      expect(consoleSpy).toHaveBeenCalledWith(
-        expect.stringContaining('[EnhancedNavigationMenu] Navigation item clicked:'),
-        expect.objectContaining({
-          item: 'dashboard',
-          path: '/dashboard',
-          permissions: ['read:dashboard']
-        })
-      );
+      await waitFor(() => {
+        expect(mockLogger.debug).toHaveBeenCalledWith(
+          'Navigation item clicked:',
+          expect.objectContaining({
+            item: 'dashboard',
+            path: '/dashboard',
+            permissions: ['read:dashboard']
+          })
+        );
+      });
     });
 
     it('should not log when audit logging is disabled', async () => {
@@ -414,8 +436,9 @@ describe('EnhancedNavigationMenu', () => {
       const dashboardButton = screen.getByRole('button', { name: /dashboard/i });
       await user.click(dashboardButton);
 
-      expect(consoleSpy).not.toHaveBeenCalledWith(
-        expect.stringContaining('[EnhancedNavigationMenu] Navigation item clicked:')
+      expect(mockLogger.debug).not.toHaveBeenCalledWith(
+        'Navigation item clicked:',
+        expect.any(Object)
       );
     });
 
@@ -466,7 +489,6 @@ describe('EnhancedNavigationMenu', () => {
 
   describe('Strict Mode', () => {
     it('should log strict mode violations when enabled', () => {
-      const consoleErrorSpy = vi.spyOn(console, 'error');
       defaultNavigationPermissions.hasNavigationPermission.mockReturnValue(false);
       // Ensure the mock is applied
       mockUseNavigationPermissions.mockReturnValue(defaultNavigationPermissions);
@@ -485,7 +507,6 @@ describe('EnhancedNavigationMenu', () => {
     });
 
     it('should not log strict mode violations when disabled', () => {
-      const consoleErrorSpy = vi.spyOn(console, 'error');
       defaultNavigationPermissions.hasNavigationPermission.mockReturnValue(false);
 
       render(
@@ -497,14 +518,15 @@ describe('EnhancedNavigationMenu', () => {
         </TestWrapper>
       );
 
-      expect(consoleErrorSpy).not.toHaveBeenCalledWith(
-        expect.stringContaining('[EnhancedNavigationMenu] STRICT MODE VIOLATION:')
+      expect(mockLogger.error).not.toHaveBeenCalledWith(
+        expect.stringContaining('STRICT MODE VIOLATION:'),
+        expect.any(Object)
       );
     });
   });
 
   describe('Initialization Logging', () => {
-    it('should log initialization when audit logging is enabled', () => {
+    it('should log initialization when audit logging is enabled', async () => {
       render(
         <TestWrapper>
           <EnhancedNavigationMenu 
@@ -514,17 +536,19 @@ describe('EnhancedNavigationMenu', () => {
         </TestWrapper>
       );
 
-      expect(consoleSpy).toHaveBeenCalledWith(
-        expect.stringContaining('[EnhancedNavigationMenu] Navigation menu initialized:'),
-        expect.objectContaining({
-          totalItems: 3,
-          filteredItems: 3,
-          strictMode: true
-        })
-      );
+      await waitFor(() => {
+        expect(mockLogger.debug).toHaveBeenCalledWith(
+          'Navigation menu initialized:',
+          expect.objectContaining({
+            totalItems: 3,
+            filteredItems: 3,
+            strictMode: true
+          })
+        );
+      });
     });
 
-    it('should log strict mode status on mount', () => {
+    it('should log strict mode status on mount', async () => {
       render(
         <TestWrapper>
           <EnhancedNavigationMenu 
@@ -535,9 +559,11 @@ describe('EnhancedNavigationMenu', () => {
         </TestWrapper>
       );
 
-      expect(consoleSpy).toHaveBeenCalledWith(
-        expect.stringContaining('[EnhancedNavigationMenu] Strict mode enabled - all navigation access attempts will be logged and enforced')
-      );
+      await waitFor(() => {
+        expect(mockLogger.debug).toHaveBeenCalledWith(
+          'Strict mode enabled - all navigation access attempts will be logged and enforced'
+        );
+      });
     });
   });
 
@@ -545,8 +571,8 @@ describe('EnhancedNavigationMenu', () => {
     it('should handle missing navigation permissions gracefully', () => {
       mockUseNavigationPermissions.mockReturnValue({
         ...defaultNavigationPermissions,
-        hasNavigationPermission: undefined,
-        getFilteredNavigationItems: undefined
+        hasNavigationPermission: vi.fn(() => true),
+        getFilteredNavigationItems: vi.fn((items) => items)
       });
 
       expect(() => {
@@ -566,7 +592,7 @@ describe('EnhancedNavigationMenu', () => {
       );
 
       expect(screen.getByRole('navigation')).toBeInTheDocument();
-      expect(screen.queryByRole('button')).not.toBeInTheDocument();
+      expect(screen.queryAllByRole('button')).toHaveLength(0);
     });
   });
 

package/src/rbac/components/__tests__/NavigationGuard.test.tsx

@@ -13,6 +13,30 @@ import { ReactNode } from 'react';
 import { NavigationGuard } from '../NavigationGuard';
 import { useMultiplePermissions } from '../../hooks/usePermissions';
 import { useUnifiedAuth } from '../../../providers/UnifiedAuthProvider';
+import { getRBACLogger } from '../../config';
+
+// Mock the RBAC logger - create shared mock
+const mockLogger = {
+  debug: vi.fn(),
+  error: vi.fn(),
+  warn: vi.fn(),
+  info: vi.fn(),
+};
+
+vi.mock('../../config', () => ({
+  getRBACLogger: vi.fn(() => mockLogger),
+  getRBACConfig: vi.fn(() => ({
+    debug: true,
+    logLevel: 'debug',
+    developmentMode: true,
+    audit: {
+      enabled: true,
+      logLevel: 'debug'
+    }
+  })),
+  isDebugMode: vi.fn(() => true),
+  isDevelopmentMode: vi.fn(() => true),
+}));
 
 // Mock the RBAC hooks
 vi.mock('../../hooks/usePermissions', () => ({
@@ -499,8 +523,21 @@ describe('NavigationGuard Component', () => {
   });
 
   describe('Security Features', () => {
+
     it('prevents bypassing in strict mode', async () => {
-      const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+      vi.clearAllMocks();
+      
+      // Create a local mock logger to ensure we can track calls
+      const localMockLogger = {
+        debug: vi.fn(),
+        error: vi.fn(),
+        warn: vi.fn(),
+        info: vi.fn(),
+      };
+      
+      // Mock getRBACLogger to return our local mock
+      const mockGetRBACLogger = vi.mocked(getRBACLogger);
+      mockGetRBACLogger.mockReturnValue(localMockLogger);
       
       mockUseMultiplePermissions.mockReturnValue({
         results: { 'read:dashboard': false } as Record<string, boolean>,
@@ -513,6 +550,7 @@ describe('NavigationGuard Component', () => {
         <NavigationGuard 
           navigationItem={mockNavigationItem}
           strictMode={true}
+          auditLog={true}
           fallback={<TestFallback />}
         >
           <TestComponent>Navigation Link</TestComponent>
@@ -523,20 +561,32 @@ describe('NavigationGuard Component', () => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
       }, { interval: 10 });
 
-      expect(consoleSpy).toHaveBeenCalledWith(
-        expect.stringContaining('STRICT MODE VIOLATION'),
-        expect.objectContaining({
-          navigationItem: 'nav-dashboard',
-          permissions: ['read:dashboard'],
-          userId: 'user-123'
-        })
-      );
-
-      consoleSpy.mockRestore();
+      await waitFor(() => {
+        expect(localMockLogger.error).toHaveBeenCalledWith(
+          expect.stringContaining('STRICT MODE VIOLATION'),
+          expect.objectContaining({
+            navigationItem: 'nav-dashboard',
+            permissions: ['read:dashboard'],
+            userId: 'user-123'
+          })
+        );
+      }, { timeout: 2000, interval: 100 });
     });
 
     it('logs navigation access attempts for audit', async () => {
-      const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+      vi.clearAllMocks();
+      
+      // Create a local mock logger to ensure we can track calls
+      const localMockLogger = {
+        debug: vi.fn(),
+        error: vi.fn(),
+        warn: vi.fn(),
+        info: vi.fn(),
+      };
+      
+      // Mock getRBACLogger to return our local mock
+      const mockGetRBACLogger = vi.mocked(getRBACLogger);
+      mockGetRBACLogger.mockReturnValue(localMockLogger);
       
       mockUseMultiplePermissions.mockReturnValue({
         results: { 'read:dashboard': false } as Record<string, boolean>,
@@ -559,17 +609,17 @@ describe('NavigationGuard Component', () => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
       }, { interval: 10 });
 
-      expect(consoleSpy).toHaveBeenCalledWith(
-        expect.stringContaining('Navigation access attempt'),
-        expect.objectContaining({
-          navigationItem: 'nav-dashboard',
-          permissions: ['read:dashboard'],
-          userId: 'user-123',
-          allowed: false
-        })
-      );
-
-      consoleSpy.mockRestore();
+      await waitFor(() => {
+        expect(localMockLogger.debug).toHaveBeenCalledWith(
+          'Navigation access attempt:',
+          expect.objectContaining({
+            navigationItem: 'nav-dashboard',
+            permissions: ['read:dashboard'],
+            userId: 'user-123',
+            allowed: false
+          })
+        );
+      }, { timeout: 2000, interval: 100 });
     });
 
     it('calls onDenied callback when access is denied', async () => {
@@ -628,7 +678,7 @@ describe('NavigationGuard Component', () => {
 
   describe('Configuration Options', () => {
     it('respects strictMode setting', async () => {
-      const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+      vi.clearAllMocks();
       
       mockUseMultiplePermissions.mockReturnValue({
         results: { 'read:dashboard': false } as Record<string, boolean>,
@@ -651,15 +701,15 @@ describe('NavigationGuard Component', () => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
       }, { interval: 10 });
 
-      expect(consoleSpy).not.toHaveBeenCalledWith(
-        expect.stringContaining('STRICT MODE VIOLATION')
+      // Strict mode is disabled, so error should not be called
+      expect(mockLogger.error).not.toHaveBeenCalledWith(
+        expect.stringContaining('STRICT MODE VIOLATION'),
+        expect.anything()
       );
-
-      consoleSpy.mockRestore();
     });
 
     it('respects auditLog setting', async () => {
-      const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+      vi.clearAllMocks();
       
       mockUseMultiplePermissions.mockReturnValue({
         results: { 'read:dashboard': false } as Record<string, boolean>,
@@ -682,11 +732,11 @@ describe('NavigationGuard Component', () => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
       }, { interval: 10 });
 
-      expect(consoleSpy).not.toHaveBeenCalledWith(
-        expect.stringContaining('Navigation access attempt')
+      // Audit log is disabled, so debug should not be called
+      expect(mockLogger.debug).not.toHaveBeenCalledWith(
+        'Navigation access attempt:',
+        expect.anything()
       );
-
-      consoleSpy.mockRestore();
     });
 
     it('respects requireAll setting', async () => {

package/src/rbac/components/__tests__/NavigationProvider.test.tsx

@@ -15,6 +15,29 @@ import { vi, describe, it, expect, beforeEach, afterEach } from 'vitest';
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
 import { renderWithProviders } from '../../../__tests__/helpers';
 
+// Mock the RBAC logger - define inside factory to avoid hoisting issues
+const mockLogger = {
+  debug: vi.fn(),
+  error: vi.fn(),
+  warn: vi.fn(),
+  info: vi.fn(),
+};
+
+vi.mock('../../config', () => ({
+  getRBACLogger: vi.fn(() => mockLogger),
+  getRBACConfig: vi.fn(() => ({
+    debug: true,
+    logLevel: 'debug',
+    developmentMode: true,
+    audit: {
+      enabled: true,
+      logLevel: 'debug'
+    }
+  })),
+  isDebugMode: vi.fn(() => true),
+  isDevelopmentMode: vi.fn(() => true),
+}));
+
 import { 
   NavigationProvider, 
   NavigationProviderProps, 
@@ -117,11 +140,8 @@ const TestWrapper: React.FC<{
 };
 
 describe('NavigationProvider', () => {
-  let consoleSpy: any;
-
   beforeEach(() => {
-    consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
-    vi.spyOn(console, 'error').mockImplementation(() => {});
+    vi.clearAllMocks();
     
     // Set up default mock with organisation context
     mockUseUnifiedAuth.mockReturnValue({
@@ -132,7 +152,6 @@ describe('NavigationProvider', () => {
   });
 
   afterEach(() => {
-    consoleSpy.mockRestore();
     vi.clearAllMocks();
   });
 
@@ -358,28 +377,32 @@ describe('NavigationProvider', () => {
   });
 
   describe('Audit Logging', () => {
-    it('should log strict mode status when enabled', () => {
+    it('should log strict mode status when enabled', async () => {
       renderWithProviders(
         <TestWrapper providerProps={{ strictMode: true, auditLog: true }}>
           <TestComponent />
         </TestWrapper>
       );
 
-      expect(consoleSpy).toHaveBeenCalledWith(
-        expect.stringContaining('[NavigationProvider] Strict mode enabled - all navigation access attempts will be logged and enforced')
-      );
+      await waitFor(() => {
+        expect(mockLogger.debug).toHaveBeenCalledWith(
+          'Strict mode enabled - all navigation access attempts will be logged and enforced'
+        );
+      });
     });
 
-    it('should not log when audit logging is disabled', () => {
+    it('should not log when audit logging is disabled', async () => {
       renderWithProviders(
         <TestWrapper providerProps={{ auditLog: false }}>
           <TestComponent />
         </TestWrapper>
       );
 
-      expect(consoleSpy).not.toHaveBeenCalledWith(
-        expect.stringContaining('[NavigationProvider] Strict mode enabled')
-      );
+      await waitFor(() => {
+        expect(mockLogger.debug).not.toHaveBeenCalledWith(
+          'Strict mode enabled - all navigation access attempts will be logged and enforced'
+        );
+      });
     });
   });
 

package/src/rbac/components/__tests__/PagePermissionGuard.test.tsx

@@ -30,12 +30,12 @@ vi.mock('../../utils/eventContext', () => ({
 }));
 
 // Mock the app name resolver
-vi.mock('../../../utils/appNameResolver', () => ({
+vi.mock('../../../utils/app/appNameResolver', () => ({
   getCurrentAppName: vi.fn()
 }));
 
 import { createScopeFromEvent } from '../../utils/eventContext';
-import { getCurrentAppName } from '../../../utils/appNameResolver';
+import { getCurrentAppName } from '../../../utils/app/appNameResolver';
 
 // Mock data
 const mockUser = {

package/src/rbac/components/__tests__/PagePermissionProvider.test.tsx

@@ -23,6 +23,22 @@ import {
 } from '../PagePermissionProvider';
 import { useUnifiedAuth } from '../../../providers/UnifiedAuthProvider';
 
+// Mock the Logger module
+vi.mock('../../../utils/core/logger', () => {
+  const mockLoggerInstance = {
+    debug: vi.fn(),
+    warn: vi.fn(),
+    error: vi.fn(),
+  };
+  return {
+    createLogger: vi.fn(() => mockLoggerInstance),
+  };
+});
+
+// Get the mock instance after mock is set up
+import { createLogger } from '../../../utils/core/logger';
+const getMockLogger = () => createLogger('test');
+
 // Mock the UnifiedAuthProvider
 vi.mock('../../../providers/UnifiedAuthProvider', () => ({
   useUnifiedAuth: vi.fn(),
@@ -89,10 +105,7 @@ const TestWrapper: React.FC<{
 };
 
 describe('PagePermissionProvider', () => {
-  let consoleSpy: any;
-
   beforeEach(() => {
-    consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
     vi.spyOn(console, 'error').mockImplementation(() => {});
     
     // Set up default mock with organisation context
@@ -104,7 +117,6 @@ describe('PagePermissionProvider', () => {
   });
 
   afterEach(() => {
-    consoleSpy.mockRestore();
     // Don't clear mocks to preserve the mock setup
     // vi.clearAllMocks();
   });
@@ -338,8 +350,9 @@ describe('PagePermissionProvider', () => {
         </TestWrapper>
       );
 
-      expect(consoleSpy).toHaveBeenCalledWith(
-        expect.stringContaining('[PagePermissionProvider] Strict mode enabled - all page access attempts will be logged and enforced')
+      const logger = getMockLogger();
+      expect(vi.mocked(logger.debug)).toHaveBeenCalledWith(
+        'Strict mode enabled - all page access attempts will be logged and enforced'
       );
     });
 
@@ -350,8 +363,9 @@ describe('PagePermissionProvider', () => {
         </TestWrapper>
       );
 
-      expect(consoleSpy).not.toHaveBeenCalledWith(
-        expect.stringContaining('[PagePermissionProvider] Strict mode enabled')
+      const logger = getMockLogger();
+      expect(vi.mocked(logger.debug)).not.toHaveBeenCalledWith(
+        expect.stringContaining('Strict mode enabled')
       );
     });
   });