Skip to content

Commit c389888

Browse files
committed
feat: Update uploadReport mock implementation to include AbortSignal parameter
1 parent 258e3d9 commit c389888

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

frontend/src/common/hooks/__tests__/useFileUpload.test.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ import { uploadReport } from '../../api/reportService';
3535
// Define mocked function types
3636
type MockedValidateFile = { mockReturnValue: (value: { isValid: boolean, errorKey?: string }) => void };
3737
type MockedUploadReport = {
38-
mockImplementation: (fn: (file: File, progressCallback?: UploadProgressCallback) => Promise<MedicalReport>) => void;
38+
mockImplementation: (fn: (file: File, progressCallback?: UploadProgressCallback, signal?: AbortSignal) => Promise<MedicalReport>) => void;
3939
mockRejectedValue: (err: Error) => void;
4040
};
4141
type MockedPermissionCheck = { mockResolvedValue: (value: boolean) => void };
@@ -48,7 +48,7 @@ describe('useFileUpload hook', () => {
4848
vi.resetAllMocks();
4949
// Default mock implementation with safer type casting
5050
(validateFile as unknown as MockedValidateFile).mockReturnValue({ isValid: true });
51-
(uploadReport as unknown as MockedUploadReport).mockImplementation((file: File, progressCallback?: UploadProgressCallback) => {
51+
(uploadReport as unknown as MockedUploadReport).mockImplementation((file: File, progressCallback?: UploadProgressCallback, _signal?: AbortSignal) => {
5252
if (progressCallback) progressCallback(1);
5353
return Promise.resolve(mockReport as MedicalReport);
5454
});
@@ -114,7 +114,7 @@ describe('useFileUpload hook', () => {
114114
});
115115

116116
expect(checkFilePermissions).toHaveBeenCalled();
117-
expect(uploadReport).toHaveBeenCalledWith(mockFile, expect.any(Function));
117+
expect(uploadReport).toHaveBeenCalledWith(mockFile, expect.any(Function), expect.any(AbortSignal));
118118
expect(result.current.status).toBe(UploadStatus.SUCCESS);
119119
expect(result.current.progress).toBe(1);
120120
expect(onUploadCompleteMock).toHaveBeenCalledWith(mockReport);
@@ -140,7 +140,7 @@ describe('useFileUpload hook', () => {
140140
}
141141
});
142142

143-
expect(uploadReport).toHaveBeenCalledWith(mockFile, expect.any(Function));
143+
expect(uploadReport).toHaveBeenCalledWith(mockFile, expect.any(Function), expect.any(AbortSignal));
144144
expect(result.current.status).toBe(UploadStatus.ERROR);
145145
expect(result.current.error).toBe('Upload failed');
146146
});
@@ -186,4 +186,4 @@ describe('useFileUpload hook', () => {
186186
expect(result.current.progress).toBe(0);
187187
expect(result.current.error).toBeNull();
188188
});
189-
});
189+
});

0 commit comments

Comments
 (0)