@jmruthers/pace-core 0.6.10 → 0.6.11

package/src/hooks/{__tests__/useStorage.unit.test.ts → useStorage.unit.test.ts}

@@ -1,7 +1,7 @@
 /**
  * @file useStorage Hook Unit Tests
  * @package @jmruthers/pace-core
- * @module Hooks/__tests__
+ * @module Hooks
  * @since 0.1.0
  *
  * Tests focus on behavior: file upload, URL generation, file management, listing, and error handling.
@@ -10,11 +10,11 @@
 
 import { renderHook, act } from '@testing-library/react';
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { useStorage, useFileUpload } from '../useStorage';
+import { useStorage, useFileUpload } from './useStorage';
 import { SupabaseClient } from '@supabase/supabase-js';
 
-// Mock the storage utilities - mock individual file since barrel was removed
-vi.mock('../../utils/storage/helpers', () => ({
+// Mock the storage utilities - same path as useStorage.ts imports (../utils/storage/helpers from src/hooks/)
+vi.mock('../utils/storage/helpers', () => ({
   uploadFile: vi.fn(),
   getPublicUrl: vi.fn(),
   getSignedUrl: vi.fn(),
@@ -30,7 +30,7 @@ import {
   deleteFile,
   listFiles,
   archiveFile,
-} from '../../utils/storage/helpers';
+} from '../utils/storage/helpers';
 
 // Mock Supabase client
 const mockSupabase = {
@@ -74,8 +74,7 @@ describe('useStorage Hook', () => {
 
   describe('File Upload', () => {
     it('handles successful file upload and refreshes file list', async () => {
-      const mockUploadResult = {
-        success: true,
+      const mockUploadData = {
         path: 'test-org/files/test-file.txt',
         publicUrl: 'https://example.com/test-file.txt',
         metadata: {
@@ -88,7 +87,6 @@ describe('useStorage Hook', () => {
           isPublic: false,
         },
       };
-
       const mockListResult = {
         files: [
           {
@@ -97,15 +95,15 @@ describe('useStorage Hook', () => {
             size: 1024,
             mimeType: 'text/plain',
             lastModified: '2024-01-01T00:00:00Z',
-            metadata: mockUploadResult.metadata,
+            metadata: mockUploadData.metadata,
           },
         ],
         totalCount: 1,
         hasMore: false,
       };
 
-      vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
-      vi.mocked(listFiles).mockResolvedValue(mockListResult);
+      vi.mocked(uploadFile).mockResolvedValue({ ok: true, data: mockUploadData });
+      vi.mocked(listFiles).mockResolvedValue({ ok: true, data: mockListResult });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -114,7 +112,7 @@ describe('useStorage Hook', () => {
         uploadResult = await result.current.uploadFile(mockFile);
       });
 
-      expect(uploadResult).toEqual(mockUploadResult);
+      expect(uploadResult).toEqual({ success: true, ...mockUploadData });
       expect(result.current.isUploading).toBe(false);
       expect(result.current.uploadError).toBe(null);
       expect(result.current.files).toEqual(mockListResult.files);
@@ -126,12 +124,7 @@ describe('useStorage Hook', () => {
     });
 
     it('handles upload failure', async () => {
-      const mockUploadResult = {
-        success: false,
-        error: 'Upload failed: File too large',
-      };
-
-      vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
+      vi.mocked(uploadFile).mockResolvedValue({ ok: false, error: { code: 'UPLOAD_FAILED', message: 'Upload failed: File too large' } });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -140,19 +133,14 @@ describe('useStorage Hook', () => {
         uploadResult = await result.current.uploadFile(mockFile);
       });
 
-      expect(uploadResult).toEqual(mockUploadResult);
+      expect(uploadResult).toEqual({ success: false, error: 'Upload failed: File too large' });
       expect(result.current.isUploading).toBe(false);
       expect(result.current.uploadError).toBe('Upload failed: File too large');
     });
 
     it('handles upload with custom options', async () => {
-      const mockUploadResult = {
-        success: true,
-        path: 'test-org/custom/test-file.txt',
-      };
-
-      vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
-      vi.mocked(listFiles).mockResolvedValue({ files: [], totalCount: 0, hasMore: false });
+      vi.mocked(uploadFile).mockResolvedValue({ ok: true, data: { path: 'test-org/custom/test-file.txt', publicUrl: undefined, metadata: undefined } });
+      vi.mocked(listFiles).mockResolvedValue({ ok: true, data: { files: [], totalCount: 0, hasMore: false } });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -252,8 +240,7 @@ describe('useStorage Hook', () => {
         url: 'https://example.com/signed/test-file.txt',
         expiresAt: '2024-01-01T01:00:00Z',
       };
-
-      vi.mocked(getSignedUrl).mockResolvedValue(mockSignedUrl);
+      vi.mocked(getSignedUrl).mockResolvedValue({ ok: true, data: mockSignedUrl });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -283,8 +270,8 @@ describe('useStorage Hook', () => {
       expect(signedUrl).toBe(null);
     });
 
-    it('handles null signed URL result', async () => {
-      vi.mocked(getSignedUrl).mockResolvedValue(null);
+    it('handles err signed URL result', async () => {
+      vi.mocked(getSignedUrl).mockResolvedValue({ ok: false, error: { code: 'SIGNED_URL_FAILED', message: 'Failed' } });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -299,11 +286,9 @@ describe('useStorage Hook', () => {
 
   describe('File Management', () => {
     it('deletes file successfully and refreshes file list', async () => {
-      const mockDeleteResult = { success: true };
       const mockListResult = { files: [], totalCount: 0, hasMore: false };
-
-      vi.mocked(deleteFile).mockResolvedValue(mockDeleteResult);
-      vi.mocked(listFiles).mockResolvedValue(mockListResult);
+      vi.mocked(deleteFile).mockResolvedValue({ ok: true, data: undefined });
+      vi.mocked(listFiles).mockResolvedValue({ ok: true, data: mockListResult });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -312,14 +297,12 @@ describe('useStorage Hook', () => {
         deleteResult = await result.current.deleteFile('test-org/files/test-file.txt');
       });
 
-      expect(deleteResult).toEqual(mockDeleteResult);
+      expect(deleteResult).toEqual({ success: true });
       expect(deleteFile).toHaveBeenCalledWith(mockSupabase, 'test-org/files/test-file.txt');
     });
 
     it('handles delete file failures', async () => {
-      const mockDeleteResult = { success: false, error: 'Delete failed' };
-
-      vi.mocked(deleteFile).mockResolvedValue(mockDeleteResult);
+      vi.mocked(deleteFile).mockResolvedValue({ ok: false, error: { code: 'DELETE_FAILED', message: 'Delete failed' } });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -328,7 +311,7 @@ describe('useStorage Hook', () => {
         deleteResult = await result.current.deleteFile('test-org/files/test-file.txt');
       });
 
-      expect(deleteResult).toEqual(mockDeleteResult);
+      expect(deleteResult).toEqual({ success: false, error: 'Delete failed' });
     });
 
     it('handles delete file exceptions', async () => {
@@ -348,11 +331,9 @@ describe('useStorage Hook', () => {
     });
 
     it('archives file successfully and refreshes file list', async () => {
-      const mockArchiveResult = { success: true };
       const mockListResult = { files: [], totalCount: 0, hasMore: false };
-
-      vi.mocked(archiveFile).mockResolvedValue(mockArchiveResult);
-      vi.mocked(listFiles).mockResolvedValue(mockListResult);
+      vi.mocked(archiveFile).mockResolvedValue({ ok: true, data: undefined });
+      vi.mocked(listFiles).mockResolvedValue({ ok: true, data: mockListResult });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -361,7 +342,7 @@ describe('useStorage Hook', () => {
         archiveResult = await result.current.archiveFile('test-org/files/test-file.txt');
       });
 
-      expect(archiveResult).toEqual(mockArchiveResult);
+      expect(archiveResult).toEqual({ success: true });
       expect(archiveFile).toHaveBeenCalledWith(mockSupabase, 'test-org/files/test-file.txt', {
         appName: 'test-app',
         orgId: 'test-org',
@@ -369,9 +350,7 @@ describe('useStorage Hook', () => {
     });
 
     it('handles archive file failures', async () => {
-      const mockArchiveResult = { success: false, error: 'Archive failed' };
-
-      vi.mocked(archiveFile).mockResolvedValue(mockArchiveResult);
+      vi.mocked(archiveFile).mockResolvedValue({ ok: false, error: { code: 'ARCHIVE_FAILED', message: 'Archive failed' } });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -380,7 +359,7 @@ describe('useStorage Hook', () => {
         archiveResult = await result.current.archiveFile('test-org/files/test-file.txt');
       });
 
-      expect(archiveResult).toEqual(mockArchiveResult);
+      expect(archiveResult).toEqual({ success: false, error: 'Archive failed' });
     });
 
     it('handles archive file exceptions', async () => {
@@ -425,7 +404,7 @@ describe('useStorage Hook', () => {
         hasMore: false,
       };
 
-      vi.mocked(listFiles).mockResolvedValue(mockListResult);
+      vi.mocked(listFiles).mockResolvedValue({ ok: true, data: mockListResult });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -446,8 +425,7 @@ describe('useStorage Hook', () => {
 
     it('lists files with custom options', async () => {
       const mockListResult = { files: [], totalCount: 0, hasMore: false };
-
-      vi.mocked(listFiles).mockResolvedValue(mockListResult);
+      vi.mocked(listFiles).mockResolvedValue({ ok: true, data: mockListResult });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -490,8 +468,7 @@ describe('useStorage Hook', () => {
 
     it('refreshes files list', async () => {
       const mockListResult = { files: [], totalCount: 0, hasMore: false };
-
-      vi.mocked(listFiles).mockResolvedValue(mockListResult);
+      vi.mocked(listFiles).mockResolvedValue({ ok: true, data: mockListResult });
 
       const { result } = renderHook(() => useStorage(baseProps));
 
@@ -524,7 +501,7 @@ describe('useStorage Hook', () => {
       expect(result.current.isListing).toBe(true);
 
       await act(async () => {
-        resolveList!({ files: [], totalCount: 0, hasMore: false });
+        resolveList!({ ok: true, data: { files: [], totalCount: 0, hasMore: false } });
         await listPromise;
       });
 
@@ -570,12 +547,8 @@ describe('useFileUpload Hook', () => {
 
   describe('Upload with Progress', () => {
     it('handles successful upload with progress tracking', async () => {
-      const mockUploadResult = {
-        success: true,
-        path: 'test-org/files/test-file.txt',
-      };
-
-      vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
+      const mockUploadData = { path: 'test-org/files/test-file.txt', publicUrl: undefined, metadata: undefined };
+      vi.mocked(uploadFile).mockResolvedValue({ ok: true, data: mockUploadData });
 
       const { result } = renderHook(() => useFileUpload(baseProps));
 
@@ -584,7 +557,7 @@ describe('useFileUpload Hook', () => {
         uploadResult = await result.current.uploadWithProgress(mockFile);
       });
 
-      expect(uploadResult).toEqual(mockUploadResult);
+      expect(uploadResult).toEqual({ success: true, ...mockUploadData });
       expect(result.current.uploadProgress).toBe(100);
       expect(result.current.isUploading).toBe(false);
       expect(result.current.uploadError).toBe(null);
@@ -633,12 +606,7 @@ describe('useFileUpload Hook', () => {
     });
 
     it('handles upload failure', async () => {
-      const mockUploadResult = {
-        success: false,
-        error: 'Upload failed',
-      };
-
-      vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
+      vi.mocked(uploadFile).mockResolvedValue({ ok: false, error: { code: 'UPLOAD_FAILED', message: 'Upload failed' } });
 
       const { result } = renderHook(() => useFileUpload(baseProps));
 
@@ -647,7 +615,7 @@ describe('useFileUpload Hook', () => {
         uploadResult = await result.current.uploadWithProgress(mockFile);
       });
 
-      expect(uploadResult).toEqual(mockUploadResult);
+      expect(uploadResult).toEqual({ success: false, error: 'Upload failed' });
       expect(result.current.uploadProgress).toBe(100);
       expect(result.current.isUploading).toBe(false);
       expect(result.current.uploadError).toBe('Upload failed');
@@ -672,10 +640,7 @@ describe('useFileUpload Hook', () => {
 
     it('resets progress after delay', async () => {
       vi.useFakeTimers();
-      
-      const mockUploadResult = { success: true };
-
-      vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
+      vi.mocked(uploadFile).mockResolvedValue({ ok: true, data: { path: 'test-org/files/test-file.txt', publicUrl: undefined, metadata: undefined } });
 
       const { result } = renderHook(() => useFileUpload(baseProps));
 
@@ -695,9 +660,7 @@ describe('useFileUpload Hook', () => {
     });
 
     it('handles upload with custom options', async () => {
-      const mockUploadResult = { success: true };
-
-      vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
+      vi.mocked(uploadFile).mockResolvedValue({ ok: true, data: { path: 'test-org/files/test-file.txt', publicUrl: undefined, metadata: undefined } });
 
       const { result } = renderHook(() => useFileUpload(baseProps));
 

package/src/hooks/{__tests__/useToast.test.ts → useToast.test.ts}

@@ -1,13 +1,13 @@
 /**
  * @file useToast Hook Tests
  * @package @jmruthers/pace-core
- * @module Hooks/__tests__
+ * @module Hooks
  */
 
 import { renderHook, act } from '@testing-library/react';
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import React from 'react';
-import { useToast, toast, reset } from '../useToast';
+import { useToast, toast, reset } from './useToast';
 
 describe('useToast', () => {
   beforeEach(() => {

package/src/hooks/{__tests__/useToast.unit.test.tsx → useToast.unit.test.tsx}

@@ -1,7 +1,7 @@
 /**
  * @file useToast Hook Unit Tests
  * @package @jmruthers/pace-core
- * @module Hooks/__tests__/useToast
+ * @module Hooks/useToast
  * @since 0.1.0
  * 
  * Comprehensive tests for the useToast hook covering all critical functionality.
@@ -9,7 +9,7 @@
 
 import { renderHook, act } from '@testing-library/react';
 import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { useToast, reset, toast } from '../useToast';
+import { useToast, reset, toast } from './useToast';
 
 const TOAST_REMOVE_DELAY = 1000;
 const DEFAULT_TOAST_DURATION = 5000;

package/src/hooks/{__tests__/useZodForm.unit.test.tsx → useZodForm.unit.test.tsx}

@@ -1,7 +1,7 @@
 /**
  * @file useZodForm Hook Unit Tests
  * @package @jmruthers/pace-core
- * @module Hooks/__tests__
+ * @module Hooks
  * @since 0.1.0
  *
  * Tests focus on behavior: form initialization, validation, and integration with react-hook-form.
@@ -9,7 +9,7 @@
 
 import { renderHook, act } from '@testing-library/react';
 import { describe, it, expect } from 'vitest';
-import { useZodForm } from '../useZodForm';
+import { useZodForm } from './useZodForm';
 import { z } from 'zod';
 
 const testSchema = z.object({

package/src/icons/{__tests__/index.test.ts → index.test.ts}

@@ -1,7 +1,7 @@
 /**
  * @file Icon Exports Tests
  * @package @jmruthers/pace-core
- * @module Icons/__tests__
+ * @module Icons
  * @since 0.6.6
  *
  * Comprehensive tests for icon exports covering:
@@ -11,7 +11,7 @@
  */
 
 import { describe, it, expect } from 'vitest';
-import * as Icons from '../index';
+import * as Icons from './index';
 
 // Following testing standards: use timeout parameter to prevent hanging
 const TEST_TIMEOUT = 10000; // 10 seconds per test

package/src/icons/index.ts

@@ -24,4 +24,6 @@
 // This ensures consuming apps have access to all icons while enforcing
 // consistent usage through pace-core's icon export system
 export * from 'lucide-react';
+// Explicit re-export for X so bundlers (esbuild) resolve it when building from source
+export { X } from 'lucide-react';
 

package/src/{__tests__/index.test.ts → index.test.ts}

@@ -1,7 +1,7 @@
 /**
  * @file Main Index Barrel File Tests
  * @package @jmruthers/pace-core
- * @module Core/__tests__
+ * @module Core
  * @since 0.1.0
  *
  * Comprehensive tests for the main barrel file (src/index.ts) ensuring all expected
@@ -9,7 +9,7 @@
  */
 
 import { describe, it, expect } from 'vitest';
-import * as MainExports from '../index';
+import * as MainExports from './index';
 
 // Following testing standards: use timeout parameter to prevent hanging
 const TEST_TIMEOUT = 10000; // 10 seconds per test
@@ -41,10 +41,6 @@ describe('Main Index Barrel File Exports', () => {
       expect(typeof MainExports.InactivityServiceProvider).toBe('function');
     }, TEST_TIMEOUT);
 
-    it('should export OrganisationProvider', () => {
-      expect(MainExports.OrganisationProvider).toBeDefined();
-      expect(typeof MainExports.OrganisationProvider).toBe('function');
-    }, TEST_TIMEOUT);
   });
 
   describe('Hook Exports', () => {
@@ -519,7 +515,7 @@ describe('Main Index Barrel File Exports', () => {
         'cn',
         'formatDate'
       ];
-      
+
       functionExports.forEach(exportName => {
         if (MainExports[exportName as keyof typeof MainExports]) {
           // React components can be functions or objects (memoized/forwardRef)

package/src/index.ts

@@ -28,12 +28,9 @@ export { EventServiceProvider } from './providers/services/EventServiceProvider'
 export { OrganisationServiceProvider } from './providers/services/OrganisationServiceProvider';
 export { InactivityServiceProvider } from './providers/services/InactivityServiceProvider';
 
-// Convenience provider wrapper (backward compatibility)
-export { OrganisationProvider } from './providers/OrganisationProvider';
-export type { OrganisationProviderProps } from './providers/OrganisationProvider';
 
 // Convenience hooks
-export { useEvents } from './hooks/useEvents';
+export { useEvents, useOptionalEvents } from './hooks/useEvents';
 export { useOrganisations } from './hooks/useOrganisations';
 
 // Service hooks for advanced usage (better performance)
@@ -50,6 +47,11 @@ export type {
   OrganisationSecurityError 
 } from './types/organisation';
 
+// API result shape (standard for async APIs)
+export type { ApiResult, ApiError } from './types/api-result';
+export { ok, err, isOk, isErr } from './types/api-result';
+export type { AuthSessionData } from './services/interfaces/IAuthService';
+
 // INACTIVITY TRACKING
 export { InactivityWarningModal } from './components/InactivityWarningModal/InactivityWarningModal';
 export type { InactivityWarningModalProps } from './components/InactivityWarningModal/InactivityWarningModal';
@@ -182,6 +184,7 @@ export {
 
 // Re-export types from DataTable types
 export type { DataRecord, ExportOptions } from './components/DataTable/types';
+export type { ImportSummary, ImportHandlerResult } from './components/DataTable/ui/modals/importModalPersistence';
 
 // Export DataTable utility functions
 export {
@@ -234,7 +237,12 @@ export type { UserMenuProps } from './components/UserMenu/UserMenu';
 
 // Reusable Page/Layout Components
 export { PaceAppLayout } from './components/PaceAppLayout/PaceAppLayout';
-export type { PaceAppLayoutProps } from './components/PaceAppLayout/PaceAppLayout';
+export type {
+  PaceAppLayoutProps,
+  PaceAppLayoutRouteConfigItem,
+  PaceAppLayoutPermissionConfig,
+  PaceAppLayoutRoutingConfig,
+} from './components/PaceAppLayout/PaceAppLayout';
 export { PaceLoginPage } from './components/PaceLoginPage/PaceLoginPage';
 export type { PaceLoginPageProps } from './components/PaceLoginPage/PaceLoginPage';
 
@@ -349,7 +357,7 @@ export type { PaletteData, ColorPalette, ColorShade } from './theming/runtime';
 
 // PUBLIC PAGES
 export { usePublicEvent, clearPublicEventCache, getPublicEventCacheStats } from './hooks/public/usePublicEvent';
-export { usePublicFileDisplay, clearPublicFileDisplayCache, getPublicFileDisplayCacheStats } from './hooks/public/usePublicFileDisplay';
+export { usePublicFileDisplay, clearPublicFileDisplayCache, getPublicFileDisplayCacheStats } from './components/FileDisplay/usePublicFileDisplay';
 export { usePublicEventLogo, clearPublicLogoCache, getPublicLogoCacheStats } from './hooks/public/usePublicEventLogo';
 export { 
   usePublicRouteParams, 
@@ -358,7 +366,7 @@ export {
   extractEventCodeFromPath
 } from './hooks/public/usePublicRouteParams';
 export type { UsePublicEventReturn, UsePublicEventOptions } from './hooks/public/usePublicEvent';
-export type { UsePublicFileDisplayReturn, UsePublicFileDisplayOptions } from './hooks/public/usePublicFileDisplay';
+export type { UsePublicFileDisplayReturn, UsePublicFileDisplayOptions } from './components/FileDisplay/usePublicFileDisplay';
 export type { UsePublicEventLogoReturn, UsePublicEventLogoOptions } from './hooks/public/usePublicEventLogo';
 export type { UsePublicRouteParamsReturn } from './hooks/public/usePublicRouteParams';
 

package/src/providers/{__tests__/AuthProvider.test.tsx → AuthProvider.test.tsx}

@@ -6,9 +6,9 @@
 import React from 'react';
 import { render, screen, fireEvent } from '@testing-library/react';
 import { vi, describe, it, expect, beforeEach } from 'vitest';
-import { AuthServiceProvider as AuthProvider } from '../services/AuthServiceProvider';
-import { useAuthService as useAuth } from '../../hooks/services/useAuthService';
-import { TEST_FIXTURES } from '../../__tests__/fixtures/test-data';
+import { AuthServiceProvider as AuthProvider } from './services/AuthServiceProvider';
+import { useAuthService as useAuth } from '../hooks/services/useAuthService';
+import { TEST_FIXTURES } from '../__tests__/fixtures/test-data';
 
 // Mock the debug logger
 vi.mock('../../utils/debugLogger', () => ({

package/src/providers/{__tests__/EventProvider.test.tsx → EventProvider.test.tsx}

@@ -6,8 +6,8 @@
 import React from 'react';
 import { render, screen, fireEvent } from '@testing-library/react';
 import { vi, describe, it, expect, beforeEach } from 'vitest';
-import { EventServiceProvider } from '../services/EventServiceProvider';
-import { useEvents } from '../../hooks/useEvents';
+import { EventServiceProvider } from './services/EventServiceProvider';
+import { useEvents } from '../hooks/useEvents';
 
 // Mock the debug logger
 vi.mock('../../utils/debugLogger', () => ({
@@ -18,7 +18,7 @@ vi.mock('../../utils/debugLogger', () => ({
 
 // Mock the organisation context utility
 vi.mock('../../utils/organisationContext', () => ({
-  setOrganisationContext: vi.fn().mockResolvedValue(undefined),
+  setOrganisationContext: vi.fn().mockResolvedValue({ ok: true, data: undefined }),
 }));
 
 // Mock UnifiedAuthProvider

package/src/providers/InactivityProvider.test-helper.tsx

@@ -0,0 +1,40 @@
+/**
+ * Test helper for InactivityProvider tests.
+ * Exposes a wrapper component that provides InactivityService for tests.
+ */
+
+import React from 'react';
+import { InactivityServiceProvider } from './services/InactivityServiceProvider';
+import type { SupabaseClient, User, Session } from '@supabase/supabase-js';
+
+export interface InactivityTestWrapperProps {
+  children: React.ReactNode;
+  user?: User | null;
+  session?: Session | null;
+  supabaseClient: SupabaseClient;
+  idleTimeoutMs?: number;
+  warnBeforeMs?: number;
+  onIdleLogout?: (reason: 'inactivity') => void;
+}
+
+export const TestWrapper = ({
+  children,
+  user = null,
+  session = null,
+  supabaseClient,
+  idleTimeoutMs = 30 * 60 * 1000,
+  warnBeforeMs = 60 * 1000,
+  onIdleLogout = () => {},
+}: InactivityTestWrapperProps) => (
+  <InactivityServiceProvider
+    user={user}
+    session={session}
+    supabaseClient={supabaseClient}
+    idleTimeoutMs={idleTimeoutMs}
+    warnBeforeMs={warnBeforeMs}
+    onIdleLogout={onIdleLogout}
+  >
+    {children}
+  </InactivityServiceProvider>
+);
+