Remove old report service model

Author: GuidoBR

frontend/src/common/api/__tests__/reportService.test.ts

@@ -19,12 +19,12 @@ vi.mock('axios', () => ({
 // Mock dynamic imports to handle the service functions
 vi.mock('../reportService', async (importOriginal) => {
   const actual = await importOriginal() as typeof ReportServiceModule;
-  
+
   // Create a new object with the same properties as the original
   return {
     // Keep the ReportError class
     ReportError: actual.ReportError,
-    
+
     // Mock the API functions
     uploadReport: async (file: File, onProgress?: (progress: number) => void) => {
       try {
@@ -38,36 +38,36 @@ vi.mock('../reportService', async (importOriginal) => {
         return response.data;
       } catch (error) {
         // Properly wrap the error in a ReportError
-        throw new actual.ReportError(error instanceof Error 
+        throw new actual.ReportError(error instanceof Error
           ? `Failed to upload report: ${error.message}`
           : 'Failed to upload report');
       }
     },
-    
+
     // Mock fetchLatestReports
     fetchLatestReports: async (limit = 3) => {
       try {
         const response = await axios.get(`/api/reports/latest?limit=${limit}`);
         return response.data;
       } catch (error) {
-        throw new actual.ReportError(error instanceof Error 
+        throw new actual.ReportError(error instanceof Error
           ? `Failed to fetch latest reports: ${error.message}`
           : 'Failed to fetch latest reports');
       }
     },
-    
+
     // Mock fetchAllReports
     fetchAllReports: async () => {
       try {
         const response = await axios.get(`/api/reports`);
         return response.data;
       } catch (error) {
-        throw new actual.ReportError(error instanceof Error 
+        throw new actual.ReportError(error instanceof Error
           ? `Failed to fetch all reports: ${error.message}`
           : 'Failed to fetch all reports');
       }
     },
-    
+
     // Keep other functions as is
     markReportAsRead: actual.markReportAsRead,
     getAuthConfig: actual.getAuthConfig,
@@ -92,28 +92,26 @@ const mockReports = [
     title: 'heart-report',
     status: ReportStatus.UNREAD,
     category: ReportCategory.HEART,
-    documentUrl: 'http://example.com/heart-report.pdf',
     date: '2024-03-24',
   },
   {
     id: '2',
     title: 'brain-scan',
     status: ReportStatus.UNREAD,
     category: ReportCategory.NEUROLOGICAL,
-    documentUrl: 'http://example.com/brain-scan.pdf',
     date: '2024-03-24',
   }
 ];
 
 describe('reportService', () => {
   const mockFile = new File(['test content'], 'test-report.pdf', { type: 'application/pdf' });
   let progressCallback: (progress: number) => void;
-  
+
   beforeEach(() => {
     vi.resetAllMocks();
     progressCallback = vi.fn();
   });
-  
+
   describe('uploadReport', () => {
     beforeEach(() => {
       // Mock axios.post for successful response
@@ -124,23 +122,22 @@ describe('reportService', () => {
           status: ReportStatus.UNREAD,
           category: ReportCategory.GENERAL,
           date: '2024-05-10',
-          documentUrl: 'http://example.com/test-report.pdf'
         }
       });
     });
-    
+
     test('should upload file successfully', async () => {
       const report = await uploadReport(mockFile, progressCallback);
-      
+
       // Check the returned data matches our expectations
       expect(report).toBeDefined();
       expect(report.title).toBe('test-report');
       expect(report.status).toBe(ReportStatus.UNREAD);
-      
+
       // Check the progress callback was called
       expect(progressCallback).toHaveBeenCalled();
     });
-    
+
     test('should determine category based on filename', async () => {
       // Mock response for heart file
       (axios.post as ReturnType<typeof vi.fn>).mockResolvedValueOnce({
@@ -150,14 +147,13 @@ describe('reportService', () => {
           status: ReportStatus.UNREAD,
           category: ReportCategory.HEART,
           date: '2024-05-10',
-          documentUrl: 'http://example.com/heart-report.pdf'
         }
       });
-      
+
       const heartFile = new File(['test'], 'heart-report.pdf', { type: 'application/pdf' });
       const heartReport = await uploadReport(heartFile);
       expect(heartReport.category).toBe(ReportCategory.HEART);
-      
+
       // Mock response for neurological file
       (axios.post as ReturnType<typeof vi.fn>).mockResolvedValueOnce({
         data: {
@@ -166,27 +162,26 @@ describe('reportService', () => {
           status: ReportStatus.UNREAD,
           category: ReportCategory.NEUROLOGICAL,
           date: '2024-05-10',
-          documentUrl: 'http://example.com/brain-scan.pdf'
         }
       });
-      
+
       const neuroFile = new File(['test'], 'brain-scan.pdf', { type: 'application/pdf' });
       const neuroReport = await uploadReport(neuroFile);
       expect(neuroReport.category).toBe(ReportCategory.NEUROLOGICAL);
     });
-    
+
     test('should handle upload without progress callback', async () => {
       const report = await uploadReport(mockFile);
       expect(report).toBeDefined();
       expect(report.title).toBe('test-report');
     });
-    
+
     test('should throw ReportError on upload failure', async () => {
       // Mock axios.post to fail
       (axios.post as ReturnType<typeof vi.fn>).mockRejectedValueOnce(
         new Error('API request failed')
       );
-      
+
       await expect(uploadReport(mockFile, progressCallback))
         .rejects
         .toThrow(ReportError);
@@ -203,30 +198,30 @@ describe('reportService', () => {
 
     test('should fetch latest reports with default limit', async () => {
       const reports = await fetchLatestReports();
-      
+
       expect(axios.get).toHaveBeenCalled();
       expect(reports).toHaveLength(2);
       expect(reports[0]).toEqual(expect.objectContaining({
         id: expect.any(String),
         title: expect.any(String)
       }));
     });
-    
+
     test('should fetch latest reports with custom limit', async () => {
       const limit = 1;
       (axios.get as ReturnType<typeof vi.fn>).mockResolvedValue({
         data: mockReports.slice(0, 1)
       });
-      
+
       const reports = await fetchLatestReports(limit);
-      
+
       expect(axios.get).toHaveBeenCalled();
       expect(reports).toHaveLength(1);
     });
-    
+
     test('should throw ReportError on fetch failure', async () => {
       (axios.get as ReturnType<typeof vi.fn>).mockRejectedValue(new Error('Network error'));
-      
+
       await expect(fetchLatestReports())
         .rejects
         .toThrow(ReportError);
@@ -242,14 +237,14 @@ describe('reportService', () => {
 
     test('should fetch all reports', async () => {
       const reports = await fetchAllReports();
-      
+
       expect(axios.get).toHaveBeenCalled();
       expect(reports).toEqual(mockReports);
     });
-    
+
     test('should throw ReportError on fetch failure', async () => {
       (axios.get as ReturnType<typeof vi.fn>).mockRejectedValue(new Error('Network error'));
-      
+
       await expect(fetchAllReports())
         .rejects
         .toThrow(ReportError);
@@ -262,25 +257,25 @@ describe('reportService', () => {
         ...mockReports[0],
         status: ReportStatus.READ
       };
-      
+
       (axios.patch as ReturnType<typeof vi.fn>).mockResolvedValue({
         data: updatedReport
       });
     });
 
     test('should mark a report as read', async () => {
       const updatedReport = await markReportAsRead('1');
-      
+
       expect(axios.patch).toHaveBeenCalled();
       expect(updatedReport.status).toBe(ReportStatus.READ);
     });
-    
+
     test('should throw error when report not found', async () => {
       (axios.patch as ReturnType<typeof vi.fn>).mockRejectedValue(new Error('Report not found'));
-      
+
       await expect(markReportAsRead('non-existent-id'))
         .rejects
         .toThrow(ReportError);
     });
   });
-}); 
+});

frontend/src/common/models/medicalReport.ts

@@ -49,10 +49,4 @@ export interface MedicalReport {
   filePath: string;
   createdAt: string; // ISO date string
   updatedAt: string; // ISO date string
-
-  // Optional fields for backward compatibility
-  documentUrl?: string;
-  content?: string;
-  doctor?: string;
-  facility?: string;
 }

frontend/src/common/utils/i18n/resources/en/report.json

@@ -8,8 +8,6 @@
     "results": "Results",
     "refRange": "Ref. Range",
     "reportDate": "Report Date",
-    "doctor": "Doctor",
-    "facility": "Facility",
     "medicalComments": "Medical Comments",
     "hemoglobin": "Hemoglobin",
     "ldl": "LDL Cholesterol",

frontend/src/common/utils/i18n/resources/es/report.json

@@ -8,8 +8,6 @@
     "results": "Resultados",
     "refRange": "Rango Ref.",
     "reportDate": "Fecha del Informe",
-    "doctor": "Médico",
-    "facility": "Centro Médico",
     "medicalComments": "Comentarios Médicos",
     "hemoglobin": "Hemoglobina",
     "ldl": "Colesterol LDL",

frontend/src/common/utils/i18n/resources/fr/report.json

@@ -8,8 +8,6 @@
     "results": "Résultats",
     "refRange": "Plage Réf.",
     "reportDate": "Date du Rapport",
-    "doctor": "Médecin",
-    "facility": "Établissement",
     "medicalComments": "Commentaires Médicaux",
     "hemoglobin": "Hémoglobine",
     "ldl": "Cholestérol LDL",

frontend/src/pages/Home/components/ReportItem/__tests__/ReportItem.test.tsx

@@ -78,7 +78,6 @@ describe('ReportItem', () => {
 
     // ASSERT
     expect(screen.getByText('Blood Test')).toBeInTheDocument();
-    expect(screen.getByText('general')).toBeInTheDocument();
     expect(screen.getByText(/Upload Date • 01\/27\/2025/)).toBeInTheDocument();
     expect(screen.getByTestId('mocked-icon-user')).toBeInTheDocument();