@jmruthers/pace-core 0.5.74 → 0.5.75

package/src/providers/__tests__/OrganisationProvider.test.tsx.backup

@@ -1,820 +0,0 @@
-/**
- * @file OrganisationProvider Tests
- * @description Comprehensive tests for OrganisationProvider component
- * @package @jmruthers/pace-core
- * @module Providers/__tests__
- * @since 0.4.0
- * 
- * Comprehensive test suite for OrganisationProvider component covering all critical functionality.
- * Follows testing guidelines with proper structure, naming, and best practices.
- */
-
-import React from 'react';
-import { render, screen, waitFor, act } from '@testing-library/react';
-import userEvent from '@testing-library/user-event';
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { BrowserRouter } from 'react-router-dom';
-import { OrganisationProvider, useOrganisations } from '../OrganisationProvider';
-import { createMockSupabaseClient, testDataGenerators } from '../../__tests__/helpers/test-utils';
-
-// Mock the debug logger
-vi.mock('../../utils/debugLogger', () => ({
-  DebugLogger: {
-    log: vi.fn(),
-  },
-}));
-
-// Mock the organisation context utility
-vi.mock('../../utils/organisationContext', () => ({
-  setOrganisationContext: vi.fn().mockResolvedValue(undefined),
-}));
-
-// Mock react-router-dom
-vi.mock('react-router-dom', () => ({
-  BrowserRouter: ({ children }: { children: React.ReactNode }) => <div data-testid="browser-router">{children}</div>,
-  useNavigate: () => vi.fn(),
-}));
-
-// Mock UnifiedAuthProvider with comprehensive state management
-const mockUnifiedAuthState = {
-  supabaseClient: null as any,
-  user: null as any,
-  session: null as any,
-  signOut: vi.fn(),
-};
-
-vi.mock('../UnifiedAuthProvider', () => ({
-  UnifiedAuthProvider: ({ children, supabaseClient }: { children: React.ReactNode; supabaseClient?: any }) => {
-    mockUnifiedAuthState.supabaseClient = supabaseClient;
-    return (
-      <div data-testid="unified-auth-provider" supabaseclient={supabaseClient} appname="test-app">
-        {children}
-      </div>
-    );
-  },
-  useUnifiedAuth: () => ({
-    user: mockUnifiedAuthState.user,
-    session: mockUnifiedAuthState.session,
-    supabase: mockUnifiedAuthState.supabaseClient,
-    signOut: mockUnifiedAuthState.signOut,
-  }),
-}));
-
-// Test component that uses the organisation context
-const TestComponent = () => {
-  const org = useOrganisations();
-  
-  return (
-    <div>
-      <div data-testid="selectedOrg">{org.selectedOrganisation?.display_name || 'No organisation'}</div>
-      <div data-testid="isLoading">{org.isLoading ? 'true' : 'false'}</div>
-      <div data-testid="error">{org.error?.message || 'No error'}</div>
-      <div data-testid="hasValidContext">{org.hasValidOrganisationContext ? 'true' : 'false'}</div>
-      <div data-testid="userRole">{org.getUserRole()}</div>
-      <div data-testid="isSecure">{org.isOrganisationSecure() ? 'true' : 'false'}</div>
-      <div data-testid="organisationsCount">{org.organisations.length}</div>
-      <div data-testid="membershipsCount">{org.userMemberships.length}</div>
-      <button onClick={() => org.switchOrganisation('org-2')}>
-        Switch Organisation
-      </button>
-      <button onClick={() => org.refreshOrganisations()}>
-        Refresh Organisations
-      </button>
-      <button onClick={() => org.ensureOrganisationContext()}>
-        Ensure Context
-      </button>
-      <button onClick={() => org.validateOrganisationAccess('org-1')}>
-        Validate Access
-      </button>
-      <button onClick={() => org.getPrimaryOrganisation()}>
-        Get Primary
-      </button>
-    </div>
-  );
-};
-
-// Wrapper component that provides auth context
-const TestWrapper = ({ 
-  children, 
-  supabaseClient, 
-  user = null, 
-  session = null 
-}: { 
-  children: React.ReactNode; 
-  supabaseClient?: any;
-  user?: any;
-  session?: any;
-}) => {
-  // Update mock state
-  mockUnifiedAuthState.user = user;
-  mockUnifiedAuthState.session = session;
-  mockUnifiedAuthState.supabaseClient = supabaseClient;
-  
-  return (
-    <BrowserRouter>
-      <div data-testid="unified-auth-provider" supabaseclient={supabaseClient} appname="test-app">
-        <OrganisationProvider>
-          {children}
-        </OrganisationProvider>
-      </div>
-    </BrowserRouter>
-  );
-};
-
-describe('[component] OrganisationProvider', () => {
-  let mockSupabaseClient: any;
-  let mockQueryBuilder: any;
-
-  const mockOrganisations = [
-    {
-      id: '11111111-1111-1111-1111-111111111111',
-      name: 'test-org-1',
-      display_name: 'Test Organisation 1',
-      subscription_tier: 'standard',
-      settings: {},
-      is_active: true,
-      parent_id: null,
-      created_at: '2023-01-01T00:00:00Z',
-      updated_at: '2023-01-01T00:00:00Z'
-    },
-    {
-      id: '22222222-2222-2222-2222-222222222222',
-      name: 'test-org-2',
-      display_name: 'Test Organisation 2',
-      subscription_tier: 'premium',
-      settings: {},
-      is_active: true,
-      parent_id: null,
-      created_at: '2023-01-01T00:00:00Z',
-      updated_at: '2023-01-01T00:00:00Z'
-    }
-  ];
-
-  const mockMemberships = [
-    {
-      id: 'aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa',
-      user_id: 'user-1',
-      organisation_id: '11111111-1111-1111-1111-111111111111',
-      role: 'org_admin',
-      status: 'active',
-      granted_at: '2023-01-01T00:00:00Z',
-      granted_by: 'admin-1',
-      revoked_at: null,
-      revoked_by: null,
-      notes: null,
-      created_at: '2023-01-01T00:00:00Z',
-      updated_at: '2023-01-01T00:00:00Z'
-    },
-    {
-      id: 'bbbbbbbb-bbbb-bbbb-bbbb-bbbbbbbbbbbb',
-      user_id: 'user-1',
-      organisation_id: '22222222-2222-2222-2222-222222222222',
-      role: 'member',
-      status: 'active',
-      granted_at: '2023-01-01T00:00:00Z',
-      granted_by: 'admin-1',
-      revoked_at: null,
-      revoked_by: null,
-      notes: null,
-      created_at: '2023-01-01T00:00:00Z',
-      updated_at: '2023-01-01T00:00:00Z'
-    }
-  ];
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    
-    // Create mock Supabase client
-    mockSupabaseClient = createMockSupabaseClient();
-    
-    // Mock RPC function for getting user organisation roles
-    mockSupabaseClient.rpc = vi.fn().mockImplementation((functionName: string, params: any) => {
-      if (functionName === 'data_user_organisation_roles_get') {
-        return Promise.resolve({
-          data: mockMemberships,
-          error: null
-        });
-      }
-      return Promise.resolve({ data: null, error: null });
-    });
-
-    // Mock database queries
-    mockSupabaseClient.from = vi.fn((table: string) => {
-      const createMockQueryBuilder = (data: any) => {
-        const queryBuilder = {
-          select: vi.fn().mockReturnThis(),
-          eq: vi.fn().mockReturnThis(),
-          is: vi.fn().mockReturnThis(),
-          in: vi.fn().mockReturnThis(),
-          order: vi.fn().mockReturnThis(),
-          limit: vi.fn().mockReturnThis(),
-          single: vi.fn().mockResolvedValue({ data: data[0] || null, error: null }),
-          maybeSingle: vi.fn().mockResolvedValue({ data: data[0] || null, error: null }),
-          then: vi.fn().mockImplementation((resolve) => {
-            resolve({ data, error: null });
-          }),
-        };
-        return queryBuilder;
-      };
-
-      if (table === 'rbac_organisation_roles') {
-        return createMockQueryBuilder(mockMemberships);
-      } else if (table === 'organisations') {
-        return createMockQueryBuilder(mockOrganisations);
-      }
-      return createMockQueryBuilder([]);
-    });
-    
-    // Mock auth state
-    mockSupabaseClient.auth.getUser = vi.fn().mockResolvedValue({
-      data: { user: testDataGenerators.user({ id: 'user-1' }) },
-      error: null
-    });
-    mockSupabaseClient.auth.getSession = vi.fn().mockResolvedValue({
-      data: { session: testDataGenerators.session() },
-      error: null
-    });
-    mockSupabaseClient.auth.onAuthStateChange = vi.fn(() => ({
-      data: { subscription: { unsubscribe: vi.fn() } }
-    }));
-  });
-
-  afterEach(() => {
-    vi.restoreAllMocks();
-    localStorage.clear();
-    sessionStorage.clear();
-  });
-
-  describe('Rendering', () => {
-    it('renders children without crashing', () => {
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient}>
-          <div>Test content</div>
-        </TestWrapper>
-      );
-      
-      expect(screen.getByText('Test content')).toBeInTheDocument();
-    });
-
-    it('shows loading state initially', () => {
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient}>
-          <TestComponent />
-        </TestWrapper>
-      );
-      
-      expect(screen.getByText('Loading organisation context...')).toBeInTheDocument();
-    });
-
-    it('renders without supabase client', () => {
-      render(
-        <TestWrapper>
-          <div>Test content</div>
-        </TestWrapper>
-      );
-      
-      expect(screen.getByText('Test content')).toBeInTheDocument();
-    });
-  });
-
-  describe('Context Hook', () => {
-    it('throws error when used outside provider', () => {
-      // Suppress console.error for this test
-      const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
-      
-      expect(() => {
-        render(<TestComponent />);
-      }).toThrow('useOrganisations must be used within an OrganisationProvider');
-      
-      consoleSpy.mockRestore();
-    });
-  });
-
-  describe('Organisation Loading', () => {
-    it('loads organisations successfully', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      // Should show loading initially
-      expect(screen.getByText('Loading organisation context...')).toBeInTheDocument();
-
-      // Wait for loading to complete with shorter timeout
-      await waitFor(() => {
-        expect(screen.getByTestId('isLoading')).toHaveTextContent('false');
-      }, { timeout: 3000 });
-
-      // Should have loaded organisations
-      expect(screen.getByTestId('organisationsCount')).toHaveTextContent('2');
-      expect(screen.getByTestId('membershipsCount')).toHaveTextContent('2');
-      expect(screen.getByTestId('selectedOrg')).toHaveTextContent('Test Organisation 1');
-      expect(screen.getByTestId('hasValidContext')).toHaveTextContent('true');
-    }, 10000);
-
-    it('handles no organisation memberships', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      // Mock empty memberships
-      mockSupabaseClient.rpc.mockImplementation((functionName: string, params: any) => {
-        if (functionName === 'data_user_organisation_roles_get') {
-          return Promise.resolve({
-            data: [],
-            error: null
-          });
-        }
-        return Promise.resolve({ data: null, error: null });
-      });
-
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByText('Organisation Access Required')).toBeInTheDocument();
-      }, { timeout: 5000 });
-    });
-
-    it('handles organisation loading errors', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      // Mock API error
-      mockSupabaseClient.rpc.mockImplementation((functionName: string, params: any) => {
-        if (functionName === 'data_user_organisation_roles_get') {
-          return Promise.resolve({
-            data: null,
-            error: new Error('Database error')
-          });
-        }
-        return Promise.resolve({ data: null, error: null });
-      });
-
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByText('Organisation Access Required')).toBeInTheDocument();
-      }, { timeout: 5000 });
-    });
-
-    it('handles RPC timeout with fallback query', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      // Mock RPC timeout
-      mockSupabaseClient.rpc.mockImplementation((functionName: string, params: any) => {
-        if (functionName === 'data_user_organisation_roles_get') {
-          return Promise.reject(new Error('RPC call timeout after 10 seconds'));
-        }
-        return Promise.resolve({ data: null, error: null });
-      });
-
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('isLoading')).toHaveTextContent('false');
-      }, { timeout: 5000 });
-    });
-
-    it('handles invalid organisation IDs in memberships', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      // Mock memberships with invalid organisation IDs
-      const invalidMemberships = [
-        {
-          id: 'aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa',
-          user_id: 'user-1',
-          organisation_id: '', // Invalid empty ID
-          role: 'org_admin',
-          status: 'active',
-          granted_at: '2023-01-01T00:00:00Z',
-          granted_by: 'admin-1',
-          revoked_at: null,
-          revoked_by: null,
-          notes: null,
-          created_at: '2023-01-01T00:00:00Z',
-          updated_at: '2023-01-01T00:00:00Z'
-        }
-      ];
-
-      mockSupabaseClient.rpc.mockImplementation((functionName: string, params: any) => {
-        if (functionName === 'data_user_organisation_roles_get') {
-          return Promise.resolve({
-            data: invalidMemberships,
-            error: null
-          });
-        }
-        return Promise.resolve({ data: null, error: null });
-      });
-
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByText('Organisation Access Required')).toBeInTheDocument();
-      }, { timeout: 5000 });
-    });
-  });
-
-  describe('Organisation Selection', () => {
-    it('auto-selects primary organisation', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('selectedOrg')).toHaveTextContent('Test Organisation 1');
-        expect(screen.getByTestId('userRole')).toHaveTextContent('org_admin');
-      }, { timeout: 5000 });
-    });
-
-    it('restores persisted organisation selection', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      // Set persisted organisation
-      localStorage.setItem('pace-core-selected-organisation', JSON.stringify(mockOrganisations[1]));
-
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('selectedOrg')).toHaveTextContent('Test Organisation 2');
-        expect(screen.getByTestId('userRole')).toHaveTextContent('member');
-      }, { timeout: 5000 });
-    });
-
-    it('handles invalid persisted organisation', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      // Set invalid persisted organisation
-      localStorage.setItem('pace-core-selected-organisation', JSON.stringify({ id: 'invalid' }));
-
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('selectedOrg')).toHaveTextContent('Test Organisation 1');
-      }, { timeout: 5000 });
-    });
-  });
-
-  describe('Organisation Switching', () => {
-    it('switches to valid organisation', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('selectedOrg')).toHaveTextContent('Test Organisation 1');
-      }, { timeout: 5000 });
-
-      // Switch to second organisation
-      const switchButton = screen.getByText('Switch Organisation');
-      await userEvent.click(switchButton);
-
-      await waitFor(() => {
-        expect(screen.getByTestId('selectedOrg')).toHaveTextContent('Test Organisation 2');
-        expect(screen.getByTestId('userRole')).toHaveTextContent('member');
-      });
-    });
-
-    it('handles switching to invalid organisation', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('selectedOrg')).toHaveTextContent('Test Organisation 1');
-      }, { timeout: 5000 });
-
-      // Try to switch to invalid organisation
-      const TestInvalidSwitchComponent = () => {
-        const org = useOrganisations();
-        
-        const handleInvalidSwitch = async () => {
-          try {
-            await org.switchOrganisation('invalid-org-id');
-          } catch (error) {
-            // Error should be thrown
-          }
-        };
-
-        return (
-          <button onClick={handleInvalidSwitch}>
-            Switch to Invalid
-          </button>
-        );
-      };
-
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestInvalidSwitchComponent />
-        </TestWrapper>
-      );
-
-      const invalidSwitchButton = screen.getByText('Switch to Invalid');
-      await userEvent.click(invalidSwitchButton);
-
-      // Should not change organisation
-      expect(screen.getByTestId('selectedOrg')).toHaveTextContent('Test Organisation 1');
-    });
-  });
-
-  describe('Security Functions', () => {
-    it('validates organisation access', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('isLoading')).toHaveTextContent('false');
-      }, { timeout: 5000 });
-
-      const validateButton = screen.getByText('Validate Access');
-      await userEvent.click(validateButton);
-
-      // Should not throw error for valid organisation
-      expect(screen.getByTestId('selectedOrg')).toHaveTextContent('Test Organisation 1');
-    });
-
-    it('ensures organisation context', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('isLoading')).toHaveTextContent('false');
-      }, { timeout: 5000 });
-
-      const ensureButton = screen.getByText('Ensure Context');
-      await userEvent.click(ensureButton);
-
-      // Should not throw error when context is available
-      expect(screen.getByTestId('selectedOrg')).toHaveTextContent('Test Organisation 1');
-    });
-
-    it('checks organisation security status', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('isLoading')).toHaveTextContent('false');
-      }, { timeout: 5000 });
-
-      expect(screen.getByTestId('isSecure')).toHaveTextContent('true');
-    });
-
-    it('gets primary organisation', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('isLoading')).toHaveTextContent('false');
-      }, { timeout: 5000 });
-
-      const primaryButton = screen.getByText('Get Primary');
-      await userEvent.click(primaryButton);
-
-      // Should not throw error
-      expect(screen.getByTestId('selectedOrg')).toHaveTextContent('Test Organisation 1');
-    });
-  });
-
-  describe('Refresh Functionality', () => {
-    it('refreshes organisations', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('isLoading')).toHaveTextContent('false');
-      }, { timeout: 5000 });
-
-      const refreshButton = screen.getByText('Refresh Organisations');
-      await userEvent.click(refreshButton);
-
-      // Should show loading state during refresh
-      expect(screen.getByTestId('isLoading')).toHaveTextContent('true');
-    });
-  });
-
-  describe('Error States', () => {
-    it('shows error when no user', () => {
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      // Should show loading initially, then error state
-      expect(screen.getByText('Loading organisation context...')).toBeInTheDocument();
-    });
-
-    it('shows error when no session', () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      // Should show loading initially, then error state
-      expect(screen.getByText('Loading organisation context...')).toBeInTheDocument();
-    });
-
-    it('shows error when no supabase client', () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      render(
-        <TestWrapper user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      // Should show loading initially, then error state
-      expect(screen.getByText('Loading organisation context...')).toBeInTheDocument();
-    });
-  });
-
-  describe('Context Values', () => {
-    it('provides all required context values', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId('isLoading')).toHaveTextContent('false');
-      }, { timeout: 5000 });
-
-      // Check all context values are available
-      expect(screen.getByTestId('selectedOrg')).toBeInTheDocument();
-      expect(screen.getByTestId('isLoading')).toBeInTheDocument();
-      expect(screen.getByTestId('error')).toBeInTheDocument();
-      expect(screen.getByTestId('hasValidContext')).toBeInTheDocument();
-      expect(screen.getByTestId('userRole')).toBeInTheDocument();
-      expect(screen.getByTestId('isSecure')).toBeInTheDocument();
-      expect(screen.getByTestId('organisationsCount')).toBeInTheDocument();
-      expect(screen.getByTestId('membershipsCount')).toBeInTheDocument();
-    });
-
-    it('provides placeholder context when no organisation', () => {
-      const TestPlaceholderComponent = () => {
-        const org = useOrganisations();
-        
-        return (
-          <div>
-            <div data-testid="hasPlaceholderOrg">{org.selectedOrganisation ? 'true' : 'false'}</div>
-            <div data-testid="hasValidContext">{org.hasValidOrganisationContext ? 'true' : 'false'}</div>
-          </div>
-        );
-      };
-
-      render(
-        <TestWrapper>
-          <TestPlaceholderComponent />
-        </TestWrapper>
-      );
-
-      // Should show placeholder values
-      expect(screen.getByTestId('hasPlaceholderOrg')).toHaveTextContent('true');
-      expect(screen.getByTestId('hasValidContext')).toHaveTextContent('false');
-    });
-  });
-
-  describe('Cleanup', () => {
-    it('handles component unmount gracefully', () => {
-      const { unmount } = render(
-        <TestWrapper supabaseClient={mockSupabaseClient}>
-          <div>Test content</div>
-        </TestWrapper>
-      );
-
-      // Should not throw errors on unmount
-      expect(() => unmount()).not.toThrow();
-    });
-
-    it('clears cached data on unmount', () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      // Set some cached data
-      localStorage.setItem('pace-core-selected-organisation', 'test-data');
-      
-      const { unmount } = render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <div>Test content</div>
-        </TestWrapper>
-      );
-
-      unmount();
-
-      // Cached data should be cleared
-      expect(localStorage.getItem('pace-core-selected-organisation')).toBeNull();
-    });
-  });
-
-  describe('Retry Logic', () => {
-    it('handles retry count limits', async () => {
-      const user = testDataGenerators.user({ id: 'user-1' });
-      const session = testDataGenerators.session();
-      
-      // Mock consistent failures
-      mockSupabaseClient.rpc.mockImplementation((functionName: string, params: any) => {
-        if (functionName === 'data_user_organisation_roles_get') {
-          return Promise.resolve({
-            data: null,
-            error: new Error('Consistent failure')
-          });
-        }
-        return Promise.resolve({ data: null, error: null });
-      });
-
-      render(
-        <TestWrapper supabaseClient={mockSupabaseClient} user={user} session={session}>
-          <TestComponent />
-        </TestWrapper>
-      );
-
-      // Should eventually show error state after retries
-      await waitFor(() => {
-        expect(screen.getByText('Organisation Access Required')).toBeInTheDocument();
-      }, { timeout: 10000 });
-    });
-  });
-});