Fix next-auth mock configuration in API route tests (#56)

Author: rezwana-karim

src/app/api/orders/[id]/__tests__/route.test.ts

@@ -13,6 +13,7 @@ import { OrderStatus, PaymentStatus, ShippingStatus } from '@prisma/client';
 
 // Mock next-auth
 vi.mock('next-auth', () => ({
+  default: vi.fn(),
   getServerSession: vi.fn(),
 }));
 
@@ -333,7 +334,7 @@ describe('GET /api/orders/[id]', () => {
       const request = new NextRequest('http://localhost:3000/api/orders/order-1');
       await GET(request, { params: Promise.resolve({ id: 'order-1' }) });
 
-      expect(orderService.getOrderById).toHaveBeenCalledWith('order-1', null);
+      expect(orderService.getOrderById).toHaveBeenCalledWith('order-1', undefined);
     });
 
     it('should return 404 when order belongs to different store', async () => {

src/app/api/orders/[id]/invoice/__tests__/route.test.ts

@@ -13,6 +13,7 @@ import { PaymentStatus } from '@prisma/client';
 
 // Mock next-auth
 vi.mock('next-auth', () => ({
+  default: vi.fn(),
   getServerSession: vi.fn(),
 }));
 

src/app/api/orders/[id]/status/__tests__/route.test.ts

@@ -13,6 +13,7 @@ import { OrderStatus } from '@prisma/client';
 
 // Mock next-auth
 vi.mock('next-auth', () => ({
+  default: vi.fn(),
   getServerSession: vi.fn(),
 } as any));
 
@@ -512,7 +513,7 @@ describe('PUT /api/orders/[id]/status', () => {
         expect.objectContaining({
           orderId: 'order-1',
           newStatus: OrderStatus.PROCESSING,
-          storeId: null,
+          storeId: undefined,
         } as any)
       );
     });

src/app/api/orders/__tests__/route.test.ts

@@ -13,6 +13,7 @@ import { OrderStatus } from '@prisma/client';
 
 // Mock next-auth
 vi.mock('next-auth', () => ({
+  default: vi.fn(),
   getServerSession: vi.fn(),
 }));
 
@@ -577,7 +578,7 @@ describe('GET /api/orders', () => {
 
       expect(orderService.listOrders).toHaveBeenCalledWith(
         expect.objectContaining({
-          storeId: null,
+          storeId: undefined,
         })
       );
     });