@jmruthers/pace-core 0.6.10 → 0.6.11

package/src/components/FileDisplay/FileDisplay.test.tsx

@@ -37,11 +37,11 @@ vi.mock('../../hooks/useFileReference', () => ({
 }));
 
 // Mock useFileDisplay hook which is used by FileDisplayAuthenticated
-vi.mock('../../hooks/useFileDisplay', () => ({
+vi.mock('./useFileDisplay', () => ({
   useFileDisplay: vi.fn()
 }));
 
-vi.mock('../../hooks/public/usePublicFileDisplay', () => ({
+vi.mock('./usePublicFileDisplay', () => ({
   usePublicFileDisplay: vi.fn()
 }));
 
@@ -51,7 +51,7 @@ vi.mock('../../hooks/useFileUrl', () => ({
 
 vi.mock('../../utils/storage/helpers', () => ({
   getPublicUrl: vi.fn((_, path: string) => `https://example.com/${path}`),
-  getSignedUrl: vi.fn(async (_: unknown, path: string) => ({ url: `https://signed.example.com/${path}` })),
+  getSignedUrl: vi.fn(async (_: unknown, path: string) => ({ ok: true, data: { url: `https://signed.example.com/${path}`, expiresAt: new Date().toISOString() } })),
 }));
 
 // Helper to render with UnifiedAuthProvider
@@ -102,7 +102,7 @@ describe('[component] FileDisplay', () => {
     });
     
     // Set up default mock for useFileDisplay
-    const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+    const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
     useFileDisplay.mockReturnValue({
       isLoading: false,
       error: null,
@@ -114,7 +114,7 @@ describe('[component] FileDisplay', () => {
       refetch: vi.fn(),
     });
 
-    const usePublicFileDisplay = (await import('../../hooks/public/usePublicFileDisplay')).usePublicFileDisplay as unknown as vi.Mock;
+    const usePublicFileDisplay = (await import('./usePublicFileDisplay')).usePublicFileDisplay as unknown as vi.Mock;
     usePublicFileDisplay.mockReturnValue({
       isLoading: false,
       error: null,
@@ -138,7 +138,7 @@ describe('[component] FileDisplay', () => {
 
   it('renders error state and clears error on retry', async () => {
     const refetch = vi.fn();
-    const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+    const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
     useFileDisplay.mockReturnValue({
       isLoading: false,
       error: new Error('boom'),
@@ -159,7 +159,7 @@ describe('[component] FileDisplay', () => {
   });
 
   it('renders empty state when no files are found', async () => {
-    const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+    const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
     useFileDisplay.mockReturnValue({
       isLoading: false,
       error: null,
@@ -189,7 +189,7 @@ describe('[component] FileDisplay', () => {
     const fileUrlsMap = new Map<string, string>();
     fileUrlsMap.set('fr-1', 'https://example.com/file.png');
 
-    const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+    const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
     useFileDisplay.mockReturnValue({
       isLoading: false,
       error: null,
@@ -247,7 +247,7 @@ describe('[component] FileDisplay', () => {
       file_metadata: { fileName: 'report.pdf', fileSize: 4096, fileType: 'application/pdf' },
     };
 
-    const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+    const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
     useFileDisplay.mockReturnValue({
       isLoading: false,
       error: null,
@@ -291,7 +291,7 @@ describe('[component] FileDisplay', () => {
     fileUrlsMap.set('f1', 'https://example.com/bucket/path/a.png');
     fileUrlsMap.set('f2', 'https://signed.example.com/bucket/path/b.pdf');
 
-    const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+    const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
     useFileDisplay.mockReturnValue({
       isLoading: false,
       error: null,
@@ -344,7 +344,7 @@ describe('[component] FileDisplay', () => {
     const fileUrlsMap = new Map<string, string>();
     fileUrlsMap.set('fr-3', 'https://example.com/logo.png');
 
-    const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+    const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
     useFileDisplay.mockReturnValue({
       isLoading: false,
       error: null,
@@ -385,7 +385,7 @@ describe('[component] FileDisplay', () => {
     const fileUrlsMap = new Map<string, string>();
     fileUrlsMap.set('fr-4', 'https://example.com/logo.png');
 
-    const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+    const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
     useFileDisplay.mockReturnValue({
       isLoading: false,
       error: null,
@@ -454,7 +454,7 @@ describe('[component] FileDisplay', () => {
     const fileUrlsMap = new Map<string, string>();
     fileUrlsMap.set('fr-img', 'https://example.com/image.png');
 
-    const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+    const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
     useFileDisplay.mockReturnValue({
       isLoading: false,
       error: null,
@@ -478,7 +478,7 @@ describe('[component] FileDisplay', () => {
   });
 
   it('renders fallback UI when showFallback is enabled and an error occurs', async () => {
-    const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+    const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
     useFileDisplay.mockReturnValue({
       isLoading: false,
       error: new Error('explode'),
@@ -509,7 +509,7 @@ describe('[component] FileDisplay', () => {
       file_metadata: { fileName: 'doc.pdf', fileSize: 1024, fileType: 'application/pdf' },
     };
 
-    const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+    const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
     useFileDisplay.mockReturnValue({
       isLoading: false,
       error: null,
@@ -570,7 +570,7 @@ describe('[component] FileDisplay', () => {
     };
 
     const fileUrlsMap = new Map<string, string>([['pub-1', 'https://public.example.com/hero.png']]);
-    const usePublicFileDisplay = (await import('../../hooks/public/usePublicFileDisplay')).usePublicFileDisplay as unknown as vi.Mock;
+    const usePublicFileDisplay = (await import('./usePublicFileDisplay')).usePublicFileDisplay as unknown as vi.Mock;
     usePublicFileDisplay.mockReturnValue({
       isLoading: false,
       error: null,
@@ -607,7 +607,7 @@ describe('[component] FileDisplay', () => {
       const pdfFile = createNonImageFile('fr-pdf', 'document.pdf', 'application/pdf');
       const fileUrlsMap = new Map([['fr-pdf', 'https://signed.example.com/document.pdf']]);
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -632,7 +632,7 @@ describe('[component] FileDisplay', () => {
       const pdfFile = createNonImageFile('fr-pdf', 'doc.pdf', 'application/pdf');
       const fileUrlsMap = new Map([['fr-pdf', 'https://signed.example.com/doc.pdf']]);
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -656,7 +656,7 @@ describe('[component] FileDisplay', () => {
     it('falls back to fallback UI when fileUrl is null', async () => {
       const pdfFile = createNonImageFile('fr-pdf', 'doc.pdf', 'application/pdf');
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -693,7 +693,7 @@ describe('[component] FileDisplay', () => {
         file_metadata: { fileName: 'image.png', fileSize: 1024, fileType: 'image/png' },
       };
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -715,7 +715,7 @@ describe('[component] FileDisplay', () => {
     it('uses wrapper behavior when showDelete is true', async () => {
       const pdfFile = createNonImageFile('fr-pdf', 'doc.pdf', 'application/pdf');
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -748,7 +748,7 @@ describe('[component] FileDisplay', () => {
       };
 
       const fileUrlsMap = new Map<string, string>([['pub-pdf', 'https://public.example.com/public-doc.pdf']]);
-      const usePublicFileDisplay = (await import('../../hooks/public/usePublicFileDisplay')).usePublicFileDisplay as unknown as vi.Mock;
+      const usePublicFileDisplay = (await import('./usePublicFileDisplay')).usePublicFileDisplay as unknown as vi.Mock;
       usePublicFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -775,7 +775,7 @@ describe('[component] FileDisplay', () => {
       const pdfFile = createNonImageFile('fr-pdf', '', 'application/pdf');
       pdfFile.file_metadata.fileName = '';
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -796,7 +796,7 @@ describe('[component] FileDisplay', () => {
 
   describe('Loading States', () => {
     it('renders loading spinner when isLoading is true', async () => {
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: true,
         error: null,
@@ -818,7 +818,7 @@ describe('[component] FileDisplay', () => {
     it('uses custom loading component when provided', async () => {
       const CustomLoader = () => <div data-testid="custom-loader">Custom Loading...</div>;
       
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: true,
         error: null,
@@ -837,7 +837,7 @@ describe('[component] FileDisplay', () => {
     });
 
     it('shows fallback during loading when showFallback is enabled and fileCount is 0', async () => {
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: true,
         error: null,
@@ -857,7 +857,7 @@ describe('[component] FileDisplay', () => {
 
   describe('Fallback UI', () => {
     it('generates fallback text from filename when generateFallbackText is provided', async () => {
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -883,7 +883,7 @@ describe('[component] FileDisplay', () => {
     });
 
     it('uses fallbackSourceText when provided', async () => {
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -913,7 +913,7 @@ describe('[component] FileDisplay', () => {
       const sizes: Array<'xs' | 'sm' | 'md' | 'lg' | 'xl' | '2xl'> = ['xs', 'sm', 'md', 'lg', 'xl', '2xl'];
       
       for (const size of sizes) {
-        const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+        const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
         useFileDisplay.mockReturnValue({
           isLoading: false,
           error: null,
@@ -949,7 +949,7 @@ describe('[component] FileDisplay', () => {
         file_metadata: { fileName: 'image.png', fileSize: 1024, fileType: 'image/png' },
       };
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -992,7 +992,7 @@ describe('[component] FileDisplay', () => {
         file_metadata: { fileName: 'file.pdf', fileSize: 2048, fileType: 'application/pdf' },
       };
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -1021,7 +1021,7 @@ describe('[component] FileDisplay', () => {
         file_metadata: { fileName: 'file.pdf', fileSize: 2048, fileType: 'application/pdf' },
       };
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -1043,7 +1043,7 @@ describe('[component] FileDisplay', () => {
 
   describe('Children Rendering', () => {
     it('renders children in empty state', async () => {
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -1075,7 +1075,7 @@ describe('[component] FileDisplay', () => {
         file_metadata: { fileName: 'file.png', fileSize: 1024, fileType: 'image/png' },
       };
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -1108,7 +1108,7 @@ describe('[component] FileDisplay', () => {
         file_metadata: { fileName: 'file.png', fileSize: 1024, fileType: 'image/png' },
       };
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -1141,7 +1141,7 @@ describe('[component] FileDisplay', () => {
       );
 
       const refetch = vi.fn();
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: new Error('Test error'),
@@ -1164,7 +1164,7 @@ describe('[component] FileDisplay', () => {
     });
 
     it('handles string error messages', async () => {
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: 'String error message',
@@ -1194,7 +1194,7 @@ describe('[component] FileDisplay', () => {
         file_metadata: { fileName: 'image.png', fileSize: 1024, fileType: 'image/png', category: FileCategory.IMAGES },
       };
 
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -1214,7 +1214,7 @@ describe('[component] FileDisplay', () => {
 
   describe('Organisation ID Handling', () => {
     it('handles undefined organisation_id', async () => {
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,
@@ -1232,7 +1232,7 @@ describe('[component] FileDisplay', () => {
     });
 
     it('handles null organisation_id', async () => {
-      const useFileDisplay = (await import('../../hooks/useFileDisplay')).useFileDisplay as unknown as vi.Mock;
+      const useFileDisplay = (await import('./useFileDisplay')).useFileDisplay as unknown as vi.Mock;
       useFileDisplay.mockReturnValue({
         isLoading: false,
         error: null,