feat: Refactor report processing status to use enum and update related components

Author: adamrefaey

backend/src/document-processor/controllers/document-processor.controller.ts

@@ -25,6 +25,7 @@ import { RequestWithUser } from '../../auth/auth.middleware';
 import { Readable } from 'stream';
 import { S3Client, GetObjectCommand } from '@aws-sdk/client-s3';
 import { ConfigService } from '@nestjs/config';
+import { ProcessingStatus } from '../../reports/models/report.model';
 
 @Controller('document-processor')
 export class DocumentProcessorController {
@@ -146,7 +147,7 @@ export class DocumentProcessorController {
       // Update the report with analysis results
       report.title = result.analysis.title || 'Untitled Report';
       report.category = result.analysis.category || 'general';
-      report.isProcessed = true;
+      report.processingStatus = ProcessingStatus.PROCESSED;
 
       // Extract lab values
       report.labValues = result.analysis.labValues || [];

backend/src/reports/models/report.model.ts

@@ -5,6 +5,12 @@ export enum ReportStatus {
   READ = 'READ',
 }
 
+export enum ProcessingStatus {
+  PROCESSED = 'processed',
+  UNPROCESSED = 'unprocessed',
+  IN_PROGRESS = 'in_progress',
+}
+
 export class Report {
   @ApiProperty({ description: 'Unique identifier for the report' })
   id: string;
@@ -21,8 +27,12 @@ export class Report {
   @ApiProperty({ description: 'Category of the report' })
   category: string;
 
-  @ApiProperty({ description: 'Whether the report has been processed' })
-  isProcessed: boolean;
+  @ApiProperty({
+    description: 'Processing status of the report',
+    enum: ProcessingStatus,
+    default: ProcessingStatus.UNPROCESSED,
+  })
+  processingStatus: ProcessingStatus;
 
   @ApiProperty({ description: 'List of lab values' })
   labValues: Array<{

backend/src/reports/reports.service.ts

@@ -15,7 +15,7 @@ import {
   QueryCommand,
 } from '@aws-sdk/client-dynamodb';
 import { marshall, unmarshall } from '@aws-sdk/util-dynamodb';
-import { Report, ReportStatus } from './models/report.model';
+import { Report, ReportStatus, ProcessingStatus } from './models/report.model';
 import { GetReportsQueryDto } from './dto/get-reports.dto';
 import { UpdateReportStatusDto } from './dto/update-report-status.dto';
 import { v4 as uuidv4 } from 'uuid';
@@ -299,7 +299,7 @@ export class ReportsService {
         title: 'New Report',
         bookmarked: false,
         category: '',
-        isProcessed: false,
+        processingStatus: ProcessingStatus.UNPROCESSED,
         labValues: [],
         summary: '',
         status: ReportStatus.UNREAD,

frontend/src/common/api/reportService.ts

@@ -1,5 +1,5 @@
 import axios, { AxiosProgressEvent } from 'axios';
-import { MedicalReport, ReportCategory, ReportStatus } from '../models/medicalReport';
+import { MedicalReport, ReportCategory, ReportStatus, ProcessingStatus } from '../models/medicalReport';
 import { fetchAuthSession } from '@aws-amplify/auth';
 // Get the API URL from environment variables
 const API_URL = import.meta.env.VITE_BASE_URL_API || '';
@@ -12,7 +12,7 @@ const mockReports: MedicalReport[] = [
     title: 'Blood Test Report',
     category: ReportCategory.GENERAL,
     bookmarked: false,
-    isProcessed: true,
+    processingStatus: ProcessingStatus.PROCESSED,
     labValues: [],
     summary: 'Blood test results within normal range',
     status: ReportStatus.UNREAD,
@@ -26,7 +26,7 @@ const mockReports: MedicalReport[] = [
     title: 'Heart Checkup',
     category: ReportCategory.HEART,
     bookmarked: true,
-    isProcessed: true,
+    processingStatus: ProcessingStatus.PROCESSED,
     labValues: [],
     summary: 'Heart functioning normally',
     status: ReportStatus.READ,

frontend/src/common/models/medicalReport.ts

@@ -15,8 +15,17 @@ export enum ReportCategory {
  * Status of a medical report.
  */
 export enum ReportStatus {
+  UNREAD = 'UNREAD',
   READ = 'READ',
-  UNREAD = 'UNREAD'
+}
+
+/**
+ * Processing status of a medical report.
+ */
+export enum ProcessingStatus {
+  PROCESSED = 'processed',
+  UNPROCESSED = 'unprocessed',
+  IN_PROGRESS = 'in_progress',
 }
 
 /**
@@ -42,7 +51,7 @@ export interface MedicalReport {
   title: string;
   category: ReportCategory | string;
   bookmarked: boolean;
-  isProcessed: boolean;
+  processingStatus: ProcessingStatus;
   labValues: LabValue[];
   summary: string;
   status: ReportStatus;

frontend/src/pages/Upload/__tests__/UploadPage.test.tsx

@@ -2,7 +2,7 @@ import { vi, describe, test, expect, beforeEach } from 'vitest';
 import { render, screen, fireEvent } from '@testing-library/react';
 import UploadPage from '../UploadPage';
 import { MemoryRouter } from 'react-router-dom';
-import { MedicalReport, ReportCategory, ReportStatus } from 'common/models/medicalReport';
+import { MedicalReport, ReportCategory, ReportStatus, ProcessingStatus } from 'common/models/medicalReport';
 import '@testing-library/jest-dom';
 
 // Mock the dependencies
@@ -48,7 +48,7 @@ vi.mock('common/components/Upload/UploadModal', () => {
       createdAt: '2023-01-01',
       status: ReportStatus.UNREAD,
       bookmarked: false,
-      isProcessed: true,
+      processingStatus: ProcessingStatus.PROCESSED,
       labValues: [],
       summary: 'Test report summary',
       filePath: '/reports/test-report.pdf',