#352 Add Transaction History

src/admin/admin.controller.ts

@@ -132,8 +132,9 @@ export class AdminController {
   updateTransactionStatus(
     @Param('id') transactionId: string,
     @Body() payload: UpdateTransactionStatusDto,
+    @CurrentUser() user: AuthUserPayload,
   ) {
-    return this.adminService.updateTransactionStatus(transactionId, payload);
+    return this.adminService.updateTransactionStatus(transactionId, payload, user.sub);
   }
 
   @Get('fraud/alerts')

src/admin/admin.service.ts

@@ -324,8 +324,12 @@ export class AdminService {
     };
   }
 
-  async updateTransactionStatus(transactionId: string, payload: UpdateTransactionStatusDto) {
-    return this.transactionsService.updateTransactionStatus(transactionId, payload.status);
+  async updateTransactionStatus(
+    transactionId: string,
+    payload: UpdateTransactionStatusDto,
+    actorId?: string,
+  ) {
+    return this.transactionsService.updateTransactionStatus(transactionId, payload.status, actorId);
   }
 
   async listFraudAlerts(query: FraudAlertsQueryDto) {

src/notifications/notifications.service.ts

@@ -97,7 +97,7 @@ export class NotificationsService {
     // FCM Push Integration
     const user = await this.prisma.user.findUnique({ where: { id: userId }, select: { fcmToken: true } });
     if (user?.fcmToken) {
-      console.log(Sending FCM notification to token: \);
+      console.log(`Sending FCM notification to token: ${user.fcmToken}`);
       // In production, use admin.messaging().send() here
     }
     const delivered = this.gateway.sendToUser(userId, 'notification', notification);

src/transactions/dto/timeline.dto.ts

@@ -14,6 +14,10 @@ export class CreateMilestoneDto {
 }
 
 export class UpdateMilestoneDto {
+  @IsOptional()
+  @IsString()
+  title?: string;
+
   @IsOptional()
   @IsEnum(MilestoneStatus)
   status?: MilestoneStatus;

src/transactions/transactions.service.spec.ts

@@ -23,6 +23,10 @@ describe('TransactionsService', () => {
           provide: PrismaService,
           useValue: mockPrismaService,
         },
+        {
+          provide: NotificationsService,
+          useValue: notificationsService,
+        },
       ],
     }).compile();
 
@@ -81,4 +85,31 @@ describe('TransactionsService', () => {
       expect(result).toBeNull();
     });
   });
+
+  describe('updateStatus', () => {
+    it('updates status and logs history in a transaction', async () => {
+      const mockTx = { id: 'tx-123', status: TransactionStatus.PENDING };
+      prisma.transaction.findUnique.mockResolvedValue(mockTx);
+      prisma.transaction.update.mockResolvedValue({ ...mockTx, status: TransactionStatus.COMPLETED });
+      prisma.transactionHistory.create.mockResolvedValue({ id: 'hist-1' });
+      prisma.$transaction.mockImplementation(async (cb) => cb(prisma));
+
+      const result = await service.updateStatus('tx-123', TransactionStatus.COMPLETED, 'actor-1');
+
+      expect(prisma.transaction.update).toHaveBeenCalledWith({
+        where: { id: 'tx-123' },
+        data: { status: TransactionStatus.COMPLETED },
+      });
+      expect(prisma.transactionHistory.create).toHaveBeenCalledWith({
+        data: {
+          transactionId: 'tx-123',
+          status: TransactionStatus.COMPLETED,
+          actorId: 'actor-1',
+          notes: 'Status updated from PENDING to COMPLETED',
+        },
+      });
+      expect(notificationsService.handleTransactionUpdate).toHaveBeenCalledWith('tx-123');
+      expect(result.status).toBe(TransactionStatus.COMPLETED);
+    });
+  });
 });

src/types/prisma.types.ts

@@ -47,6 +47,16 @@ export interface ApiKey {
   updatedAt: Date;
 }
 
+export interface TransactionHistory {
+  id: string;
+  transactionId: string;
+  status: string;
+  actorId: string | null;
+  notes: string | null;
+  metadata: any | null;
+  createdAt: Date;
+}
+
 export enum TokenType {
   ACCESS = 'ACCESS',
   REFRESH = 'REFRESH',

test/transactions/transactions.service.spec.ts

@@ -26,10 +26,16 @@ describe('TransactionsService', () => {
       findFirst: jest.fn(),
       update: jest.fn(),
     },
+    transactionHistory: {
+      create: jest.fn(),
+      findMany: jest.fn(),
+    },
+    $transaction: jest.fn().mockImplementation(async (cb) => cb(mockPrismaService)),
   } as any;
 
   const mockNotificationsService = {
     sendNotification: jest.fn(),
+    handleTransactionUpdate: jest.fn(),
   };
 
   beforeEach(async () => {