@jmruthers/pace-core 0.5.73 → 0.5.75

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

@@ -4,7 +4,7 @@
  */
 
 import React from 'react';
-import { render, screen } from '@testing-library/react';
+import { render, screen, fireEvent } from '@testing-library/react';
 import { vi, describe, it, expect, beforeEach } from 'vitest';
 import { OrganisationProvider, useOrganisations } from '../OrganisationProvider';
 import { TEST_FIXTURES } from '../../__tests__/fixtures/test-data';
@@ -77,7 +77,7 @@ const TestWrapper = ({
   );
 };
 
-describe('OrganisationProvider - Simple Tests', () => {
+describe('OrganisationProvider', () => {
   let mockSupabaseClient: ReturnType<typeof createMockSupabaseClient>;
 
   beforeEach(() => {
@@ -85,7 +85,7 @@ describe('OrganisationProvider - Simple Tests', () => {
     mockSupabaseClient = createMockSupabaseClient();
   });
 
-  describe('Basic Rendering', () => {
+  describe('Rendering', () => {
     it('renders without crashing', () => {
       render(
         <TestWrapper supabaseClient={mockSupabaseClient}>
@@ -181,4 +181,314 @@ describe('OrganisationProvider - Simple Tests', () => {
       expect(screen.getByTestId('test-component')).toBeInTheDocument();
     });
   });
+
+  describe('[integration] Organisation Loading and Data Management', () => {
+    it('loads user organisations', () => {
+      const TestLoadingComponent = () => {
+        const { organisations, isLoading, error } = useOrganisations();
+        
+        return (
+          <div>
+            <div data-testid="orgs-loaded">{organisations.length}</div>
+            <div data-testid="loading-status">{isLoading ? 'loading' : 'ready'}</div>
+            <div data-testid="error-status">{error ? 'error' : 'no-error'}</div>
+          </div>
+        );
+      };
+
+      render(
+        <TestWrapper supabaseClient={mockSupabaseClient}>
+          <TestLoadingComponent />
+        </TestWrapper>
+      );
+
+      expect(screen.getByTestId('orgs-loaded')).toBeInTheDocument();
+      expect(screen.getByTestId('loading-status')).toHaveTextContent('ready');
+      expect(screen.getByTestId('error-status')).toHaveTextContent('no-error');
+    });
+
+    it('provides organisation selection functionality', () => {
+      const TestSelectionComponent = () => {
+        const { organisations, setSelectedOrganisation, selectedOrganisation } = useOrganisations();
+        
+        return (
+          <div>
+            <div data-testid="selected-org">{selectedOrganisation?.name || selectedOrganisation?.display_name || 'none'}</div>
+            <button 
+              data-testid="select-org-btn"
+              onClick={() => setSelectedOrganisation(organisations[0] || null)}
+            >
+              Select First
+            </button>
+          </div>
+        );
+      };
+
+      render(
+        <TestWrapper supabaseClient={mockSupabaseClient}>
+          <TestSelectionComponent />
+        </TestWrapper>
+      );
+
+      expect(screen.getByTestId('selected-org')).toBeInTheDocument();
+      expect(screen.getByTestId('select-org-btn')).toBeInTheDocument();
+    });
+
+    it('handles organisation switching', () => {
+      const TestSwitchingComponent = () => {
+        const { organisations, setSelectedOrganisation, selectedOrganisation } = useOrganisations();
+        
+        return (
+          <div>
+            <div data-testid="current-org">{selectedOrganisation?.name || selectedOrganisation?.display_name || 'none'}</div>
+            <button 
+              data-testid="switch-org-btn"
+              onClick={() => {
+                if (organisations.length > 1) {
+                  setSelectedOrganisation(organisations[1] || null);
+                }
+              }}
+            >
+              Switch Org
+            </button>
+          </div>
+        );
+      };
+
+      render(
+        <TestWrapper supabaseClient={mockSupabaseClient}>
+          <TestSwitchingComponent />
+        </TestWrapper>
+      );
+
+      expect(screen.getByTestId('current-org')).toBeInTheDocument();
+      expect(screen.getByTestId('switch-org-btn')).toBeInTheDocument();
+    });
+  });
+
+  describe('[integration] Organisation Context and Security', () => {
+    it('validates organisation context availability', () => {
+      const TestContextValidationComponent = () => {
+        const { hasValidOrganisationContext, selectedOrganisation } = useOrganisations();
+        
+        return (
+          <div>
+            <div data-testid="valid-context">{hasValidOrganisationContext ? 'valid' : 'invalid'}</div>
+            <div data-testid="context-status">{selectedOrganisation ? 'set' : 'not-set'}</div>
+          </div>
+        );
+      };
+
+      render(
+        <TestWrapper supabaseClient={mockSupabaseClient}>
+          <TestContextValidationComponent />
+        </TestWrapper>
+      );
+
+      expect(screen.getByTestId('valid-context')).toBeInTheDocument();
+      expect(screen.getByTestId('context-status')).toBeInTheDocument();
+    });
+
+    it('provides organisation refresh functionality', () => {
+      const TestRefreshComponent = () => {
+        const { refreshOrganisations } = useOrganisations();
+        
+        return (
+          <div>
+            <button data-testid="refresh-org-btn" onClick={() => refreshOrganisations()}>
+              Refresh Organisations
+            </button>
+          </div>
+        );
+      };
+
+      render(
+        <TestWrapper supabaseClient={mockSupabaseClient}>
+          <TestRefreshComponent />
+        </TestWrapper>
+      );
+
+      const btn = screen.getByTestId('refresh-org-btn');
+      expect(btn).toBeInTheDocument();
+      
+      // Click refresh
+      fireEvent.click(btn);
+      // Should not throw
+    });
+
+    it('maintains organisation state across re-renders', () => {
+      const TestStateStabilityComponent = () => {
+        const { selectedOrganisation, organisations } = useOrganisations();
+        
+        return (
+          <div>
+            <div data-testid="org-state">
+              {selectedOrganisation ? (selectedOrganisation.name || selectedOrganisation.display_name) : 'none'}
+            </div>
+            <div data-testid="orgs-count">{organisations.length}</div>
+          </div>
+        );
+      };
+
+      const { rerender } = render(
+        <TestWrapper supabaseClient={mockSupabaseClient}>
+          <TestStateStabilityComponent />
+        </TestWrapper>
+      );
+
+      // Mock data returns organisations, so check for either initial state or existing orgs
+      const orgState = screen.getByTestId('org-state').textContent;
+      expect(['none', 'Test Organisation', 'Test Organisation 1']).toContain(orgState);
+      
+      // Re-render
+      rerender(
+        <TestWrapper supabaseClient={mockSupabaseClient}>
+          <TestStateStabilityComponent />
+        </TestWrapper>
+      );
+
+      // After re-render, state should remain stable
+      const orgStateAfter = screen.getByTestId('org-state').textContent;
+      expect(orgStateAfter).toBe(orgState); // State should be stable
+      expect(screen.getByTestId('orgs-count')).toBeInTheDocument();
+    });
+  });
+
+  describe('[integration] Organisation Providers and Integration', () => {
+    it('integrates with auth provider', () => {
+      const TestAuthIntegrationComponent = () => {
+        const { selectedOrganisation, organisations } = useOrganisations();
+        
+        return (
+          <div>
+            <div data-testid="auth-integrated">integrated</div>
+            <div data-testid="orgs-available">{organisations.length > 0 ? 'yes' : 'no'}</div>
+          </div>
+        );
+      };
+
+      const mockUser = {
+        id: 'user-123',
+        email: 'test@example.com',
+      };
+
+      const mockSession = {
+        access_token: 'token',
+        user: mockUser,
+      };
+
+      render(
+        <TestWrapper supabaseClient={mockSupabaseClient} user={mockUser} session={mockSession}>
+          <TestAuthIntegrationComponent />
+        </TestWrapper>
+      );
+
+      expect(screen.getByTestId('auth-integrated')).toBeInTheDocument();
+      expect(screen.getByTestId('orgs-available')).toBeInTheDocument();
+    });
+
+    it('handles missing auth gracefully', () => {
+      const TestNoAuthComponent = () => {
+        const { organisations, selectedOrganisation } = useOrganisations();
+        
+        return (
+          <div>
+            <div data-testid="orgs-count">{organisations.length}</div>
+            <div data-testid="has-selection">{selectedOrganisation ? 'yes' : 'no'}</div>
+            <div data-testid="orgs-array-exists">{Array.isArray(organisations) ? 'yes' : 'no'}</div>
+          </div>
+        );
+      };
+
+      render(
+        <TestWrapper supabaseClient={mockSupabaseClient} user={null} session={null}>
+          <TestNoAuthComponent />
+        </TestWrapper>
+      );
+
+      expect(screen.getByTestId('orgs-count')).toBeInTheDocument();
+      expect(screen.getByTestId('orgs-array-exists')).toHaveTextContent('yes');
+      // Mock might return data, so we just check the structure exists
+      expect(screen.getByTestId('has-selection')).toBeInTheDocument();
+    });
+
+    it('supports organisation hierarchy', () => {
+      const TestHierarchyComponent = () => {
+        const { organisations, selectedOrganisation } = useOrganisations();
+        
+        return (
+          <div>
+            <div data-testid="has-organisations">{organisations.length > 0 ? 'yes' : 'no'}</div>
+            <div data-testid="hierarchy-supported">supported</div>
+          </div>
+        );
+      };
+
+      render(
+        <TestWrapper supabaseClient={mockSupabaseClient}>
+          <TestHierarchyComponent />
+        </TestWrapper>
+      );
+
+      expect(screen.getByTestId('has-organisations')).toBeInTheDocument();
+      expect(screen.getByTestId('hierarchy-supported')).toHaveTextContent('supported');
+    });
+  });
+
+  describe('[integration] Error Handling and Edge Cases', () => {
+    it('handles organisation loading errors', () => {
+      const TestErrorComponent = () => {
+        const { error, organisations } = useOrganisations();
+        
+        return (
+          <div>
+            <div data-testid="error-state">{error ? 'has-error' : 'no-error'}</div>
+            <div data-testid="orgs-available">{organisations.length}</div>
+          </div>
+        );
+      };
+
+      render(
+        <TestWrapper supabaseClient={null}>
+          <TestErrorComponent />
+        </TestWrapper>
+      );
+
+      expect(screen.getByTestId('error-state')).toBeInTheDocument();
+      expect(screen.getByTestId('orgs-available')).toBeInTheDocument();
+    });
+
+    it('maintains stability with rapid operations', () => {
+      const TestRapidOpsComponent = () => {
+        const { setSelectedOrganisation, refreshOrganisations, organisations } = useOrganisations();
+        
+        return (
+          <div>
+            <button 
+              data-testid="rapid-ops-btn"
+              onClick={() => {
+                setSelectedOrganisation(organisations[0] || null);
+                refreshOrganisations();
+              }}
+            >
+              Rapid Ops
+            </button>
+          </div>
+        );
+      };
+
+      render(
+        <TestWrapper supabaseClient={mockSupabaseClient}>
+          <TestRapidOpsComponent />
+        </TestWrapper>
+      );
+
+      const btn = screen.getByTestId('rapid-ops-btn');
+      fireEvent.click(btn);
+      fireEvent.click(btn);
+      fireEvent.click(btn);
+      
+      expect(btn).toBeInTheDocument();
+    });
+  });
 });