@jmruthers/pace-core 0.6.5 → 0.6.7

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

@@ -1,661 +0,0 @@
-/**
- * @file Page Permission Provider Component Tests
- * @package @jmruthers/pace-core
- * @module RBAC/Components/PagePermissionProvider
- * @since 2.0.0
- *
- * Comprehensive test suite for the PagePermissionProvider component.
- * Tests cover all functionality including page permission checking, audit logging,
- * strict mode enforcement, context management, and error scenarios.
- */
-
-import React from 'react';
-import { render, screen, waitFor, act } from '@testing-library/react';
-import { vi, describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
-import { renderWithProviders } from '../../../__tests__/helpers';
-
-import { 
-  PagePermissionProvider, 
-  PagePermissionProviderProps, 
-  usePagePermissions,
-  PageAccessRecord 
-} from '../PagePermissionProvider';
-import { useUnifiedAuth } from '../../../providers/services/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
-const mockUseUnifiedAuthFn = vi.fn();
-
-vi.mock('../../../providers/services/UnifiedAuthProvider', () => ({
-  useUnifiedAuth: () => mockUseUnifiedAuthFn(),
-  UnifiedAuthProvider: ({ children }: { children: React.ReactNode }) => <div data-testid="unified-auth-provider">{children}</div>
-}));
-
-// Test data
-const mockUser = {
-  id: 'user-123',
-  email: 'test@example.com',
-  selectedOrganisationId: 'org-456',
-  selectedEventId: 'event-789'
-};
-
-const mockScope = {
-  organisationId: 'org-456',
-  eventId: 'event-789',
-  appId: undefined
-};
-
-// Test component that uses the context
-const TestComponent: React.FC<{ testId?: string }> = ({ testId = 'test-component' }) => {
-  const context = usePagePermissions();
-  
-  return (
-    <div data-testid={testId}>
-      <div data-testid="is-enabled">{context.isEnabled.toString()}</div>
-      <div data-testid="is-strict-mode">{context.isStrictMode.toString()}</div>
-      <div data-testid="is-audit-log-enabled">{context.isAuditLogEnabled.toString()}</div>
-      <div data-testid="page-permission">
-        {context.hasPagePermission('test-page', 'read').toString()}
-      </div>
-      <div data-testid="permissions">
-        {JSON.stringify(context.getPagePermissions())}
-      </div>
-      <div data-testid="history-length">
-        {context.getPageAccessHistory().length}
-      </div>
-    </div>
-  );
-};
-
-// Test wrapper component
-const TestWrapper: React.FC<{ 
-  children: React.ReactNode; 
-  providerProps?: Partial<PagePermissionProviderProps> 
-}> = ({ children, providerProps = {} }) => {
-  const queryClient = new QueryClient({
-    defaultOptions: {
-      queries: { retry: false },
-      mutations: { retry: false }
-    }
-  });
-
-  return (
-    <QueryClientProvider client={queryClient}>
-      <PagePermissionProvider {...providerProps}>
-        {children}
-      </PagePermissionProvider>
-    </QueryClientProvider>
-  );
-};
-
-describe('PagePermissionProvider', () => {
-  beforeEach(() => {
-    vi.spyOn(console, 'error').mockImplementation(() => {});
-    
-    // Set up default mock with organisation context
-    mockUseUnifiedAuthFn.mockReturnValue({
-      ...mockUser,
-      selectedOrganisationId: 'org-456',
-      selectedEventId: 'event-789'
-    });
-  });
-
-  afterEach(() => {
-    // Don't clear mocks to preserve the mock setup
-    // vi.clearAllMocks();
-  });
-
-  describe('Basic Functionality', () => {
-    it('should render children correctly', () => {
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
-    });
-
-    it('should provide context values with defaults', () => {
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('is-enabled')).toHaveTextContent('true');
-      expect(screen.getByTestId('is-strict-mode')).toHaveTextContent('true');
-      expect(screen.getByTestId('is-audit-log-enabled')).toHaveTextContent('true');
-    });
-
-    it('should allow page access when enabled', () => {
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      // The component returns false when there's no valid scope
-      // This is the expected behavior based on the implementation
-      expect(screen.getByTestId('page-permission')).toHaveTextContent('false');
-    });
-
-    it('should return empty permissions initially', () => {
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('permissions')).toHaveTextContent('{}');
-    });
-
-    it('should return empty history initially', () => {
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('history-length')).toHaveTextContent('0');
-    });
-  });
-
-  describe('Configuration Options', () => {
-    it('should respect strictMode prop', () => {
-      renderWithProviders(
-        <TestWrapper providerProps={{ strictMode: false }}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('is-strict-mode')).toHaveTextContent('false');
-    });
-
-    it('should respect auditLog prop', () => {
-      renderWithProviders(
-        <TestWrapper providerProps={{ auditLog: false }}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('is-audit-log-enabled')).toHaveTextContent('false');
-    });
-
-    it('should respect maxHistorySize prop', () => {
-      const TestComponentWithHistory = () => {
-        const context = usePagePermissions();
-        // Test that history management functions exist
-        return (
-          <div data-testid="history-functions">
-            {typeof context.getPageAccessHistory === 'function' ? 'has-get' : 'no-get'}
-            {typeof context.clearPageAccessHistory === 'function' ? 'has-clear' : 'no-clear'}
-          </div>
-        );
-      };
-
-      renderWithProviders(
-        <TestWrapper providerProps={{ maxHistorySize: 5 }}>
-          <TestComponentWithHistory />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('history-functions')).toHaveTextContent('has-gethas-clear');
-    });
-  });
-
-  describe('Page Permission Checking', () => {
-    it('should allow page access when user is authenticated', () => {
-      // Ensure user has organisation context
-      mockUseUnifiedAuthFn.mockReturnValue({
-        ...mockUser,
-        selectedOrganisationId: 'org-456',
-        selectedEventId: 'event-789'
-      });
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('page-permission')).toHaveTextContent('false');
-    });
-
-    it('should deny page access when user is not authenticated', () => {
-      mockUseUnifiedAuthFn.mockReturnValue({ ...mockUser, id: null });
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('page-permission')).toHaveTextContent('false');
-    });
-
-    it('should deny page access when organisation context is missing', () => {
-      mockUseUnifiedAuthFn.mockReturnValue({ 
-        ...mockUser, 
-        selectedOrganisationId: null 
-      });
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('page-permission')).toHaveTextContent('false');
-    });
-
-    it('should allow page access when disabled', () => {
-      const TestComponentDisabled = () => {
-        const context = usePagePermissions();
-        // We can't directly test isEnabled from outside, but we can test the behavior
-        return (
-          <div data-testid="page-permission">
-            {context.hasPagePermission('test-page', 'read').toString()}
-          </div>
-        );
-      };
-
-      // Ensure user has organisation context
-      mockUseUnifiedAuthFn.mockReturnValue({
-        ...mockUser,
-        selectedOrganisationId: 'org-456',
-        selectedEventId: 'event-789'
-      });
-
-      // Mock the context to return disabled state
-      vi.spyOn(React, 'useState').mockReturnValueOnce([false, vi.fn()]);
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponentDisabled />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('page-permission')).toHaveTextContent('false');
-    });
-
-    it('should handle page permission with pageId', () => {
-      const TestComponentWithPageId = () => {
-        const context = usePagePermissions();
-        const hasPermission = context.hasPagePermission('test-page', 'read', 'page-123');
-        return <div data-testid="page-permission-with-id">{hasPermission.toString()}</div>;
-      };
-
-      // Ensure user has organisation context
-      mockUseUnifiedAuthFn.mockReturnValue({
-        ...mockUser,
-        selectedOrganisationId: 'org-456',
-        selectedEventId: 'event-789'
-      });
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponentWithPageId />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('page-permission-with-id')).toHaveTextContent('false');
-    });
-
-    it('should handle page permission with custom scope', () => {
-      const TestComponentWithScope = () => {
-        const context = usePagePermissions();
-        const hasPermission = context.hasPagePermission('test-page', 'read', undefined, mockScope);
-        return <div data-testid="page-permission-with-scope">{hasPermission.toString()}</div>;
-      };
-
-      // Ensure user has organisation context
-      mockUseUnifiedAuthFn.mockReturnValue({
-        ...mockUser,
-        selectedOrganisationId: 'org-456',
-        selectedEventId: 'event-789'
-      });
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponentWithScope />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('page-permission-with-scope')).toHaveTextContent('false');
-    });
-  });
-
-  describe('Audit Logging', () => {
-    it('should log strict mode status when enabled', () => {
-      renderWithProviders(
-        <TestWrapper providerProps={{ strictMode: true, auditLog: true }}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      const logger = getMockLogger();
-      expect(vi.mocked(logger.debug)).toHaveBeenCalledWith(
-        'Strict mode enabled - all page access attempts will be logged and enforced'
-      );
-    });
-
-    it('should not log when audit logging is disabled', () => {
-      renderWithProviders(
-        <TestWrapper providerProps={{ auditLog: false }}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      const logger = getMockLogger();
-      expect(vi.mocked(logger.debug)).not.toHaveBeenCalledWith(
-        expect.stringContaining('Strict mode enabled')
-      );
-    });
-  });
-
-  describe('History Management', () => {
-    it('should clear page access history', () => {
-      const TestComponentWithClear = () => {
-        const context = usePagePermissions();
-        const clearHistory = () => context.clearPageAccessHistory();
-        
-        return (
-          <div>
-            <button onClick={clearHistory} data-testid="clear-button">
-              Clear History
-            </button>
-            <div data-testid="history-length">
-              {context.getPageAccessHistory().length}
-            </div>
-          </div>
-        );
-      };
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponentWithClear />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('history-length')).toHaveTextContent('0');
-    });
-
-    it('should maintain history within maxHistorySize limit', () => {
-      const TestComponentWithHistory = () => {
-        const context = usePagePermissions();
-        
-        // Test that history management functions exist
-        return (
-          <div data-testid="history-functions">
-            {typeof context.getPageAccessHistory === 'function' ? 'has-get' : 'no-get'}
-            {typeof context.clearPageAccessHistory === 'function' ? 'has-clear' : 'no-clear'}
-          </div>
-        );
-      };
-
-      renderWithProviders(
-        <TestWrapper providerProps={{ maxHistorySize: 5 }}>
-          <TestComponentWithHistory />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('history-functions')).toHaveTextContent('has-gethas-clear');
-    });
-  });
-
-  describe('Error Handling', () => {
-    it('should handle missing user gracefully', () => {
-      mockUseUnifiedAuthFn.mockReturnValue({});
-
-      expect(() => {
-        renderWithProviders(
-          <TestWrapper>
-            <TestComponent />
-          </TestWrapper>
-        );
-      }).not.toThrow();
-    });
-
-    it('should handle missing organisation context gracefully', () => {
-      mockUseUnifiedAuthFn.mockReturnValue({
-        ...mockUser,
-        selectedOrganisationId: null,
-        selectedEventId: null
-      });
-
-      expect(() => {
-        renderWithProviders(
-          <TestWrapper>
-            <TestComponent />
-          </TestWrapper>
-        );
-      }).not.toThrow();
-    });
-  });
-
-  describe('usePagePermissions Hook', () => {
-    it('should throw error when used outside provider', () => {
-      const TestComponentOutsideProvider = () => {
-        usePagePermissions();
-        return <div>Should not render</div>;
-      };
-
-      expect(() => {
-        renderWithProviders(<TestComponentOutsideProvider />);
-      }).toThrow('usePagePermissions must be used within a PagePermissionProvider');
-    });
-
-    it('should return context when used within provider', () => {
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
-    });
-  });
-
-  describe('Callback Functions', () => {
-    it('should call onPageAccess callback when provided', () => {
-      const onPageAccess = vi.fn();
-      
-      // We can't directly trigger page access from outside the provider,
-      // but we can test that the callback is properly set up
-      renderWithProviders(
-        <TestWrapper providerProps={{ onPageAccess }}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
-    });
-
-    it('should call onStrictModeViolation callback when provided', () => {
-      const onStrictModeViolation = vi.fn();
-      
-      renderWithProviders(
-        <TestWrapper providerProps={{ onStrictModeViolation }}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
-    });
-  });
-
-  describe('Scope Management', () => {
-    it('should create scope with organisation and event IDs', () => {
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      // The scope is created internally, but we can verify the component works
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
-    });
-
-    it('should handle missing event ID in scope', () => {
-      mockUseUnifiedAuthFn.mockReturnValue({
-        ...mockUser,
-        selectedEventId: null
-      });
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
-    });
-  });
-
-  describe('Performance', () => {
-    it('should memoize context value', () => {
-      const { rerender } = renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      const initialRender = screen.getByTestId('test-component');
-
-      // Rerender with same props
-      rerender(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      // Component should still be there
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
-    });
-  });
-
-  describe('Integration', () => {
-    it('should work with multiple consumers', () => {
-      const AnotherTestComponent = () => {
-        const context = usePagePermissions();
-        return <div data-testid="another-component">{context.isEnabled.toString()}</div>;
-      };
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-          <AnotherTestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      expect(screen.getByTestId('another-component')).toBeInTheDocument();
-    });
-
-    it('should maintain state across re-renders', () => {
-      const { rerender } = renderWithProviders(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('is-enabled')).toHaveTextContent('true');
-
-      rerender(
-        <TestWrapper>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('is-enabled')).toHaveTextContent('true');
-    });
-  });
-
-  describe('Edge Cases', () => {
-    it('should handle empty page name', () => {
-      const TestComponentEmptyPage = () => {
-        const context = usePagePermissions();
-        const hasPermission = context.hasPagePermission('', 'read');
-        return <div data-testid="empty-page-permission">{hasPermission.toString()}</div>;
-      };
-
-      // Ensure user has organisation context
-      mockUseUnifiedAuthFn.mockReturnValue({
-        ...mockUser,
-        selectedOrganisationId: 'org-456',
-        selectedEventId: 'event-789'
-      });
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponentEmptyPage />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('empty-page-permission')).toHaveTextContent('false');
-    });
-
-    it('should handle empty operation', () => {
-      const TestComponentEmptyOperation = () => {
-        const context = usePagePermissions();
-        const hasPermission = context.hasPagePermission('test-page', '');
-        return <div data-testid="empty-operation-permission">{hasPermission.toString()}</div>;
-      };
-
-      // Ensure user has organisation context
-      mockUseUnifiedAuthFn.mockReturnValue({
-        ...mockUser,
-        selectedOrganisationId: 'org-456',
-        selectedEventId: 'event-789'
-      });
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponentEmptyOperation />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('empty-operation-permission')).toHaveTextContent('false');
-    });
-
-    it('should handle undefined pageId', () => {
-      const TestComponentUndefinedPageId = () => {
-        const context = usePagePermissions();
-        const hasPermission = context.hasPagePermission('test-page', 'read', undefined);
-        return <div data-testid="undefined-page-id-permission">{hasPermission.toString()}</div>;
-      };
-
-      // Ensure user has organisation context
-      mockUseUnifiedAuthFn.mockReturnValue({
-        ...mockUser,
-        selectedOrganisationId: 'org-456',
-        selectedEventId: 'event-789'
-      });
-
-      renderWithProviders(
-        <TestWrapper>
-          <TestComponentUndefinedPageId />
-        </TestWrapper>
-      );
-
-      expect(screen.getByTestId('undefined-page-id-permission')).toHaveTextContent('false');
-    });
-  });
-});