test: update recipe update test for new behavior

- Test now validates that only specified relations are deleted
- First test: updating only title doesn't delete any relations
- Second test: updating tags and ingredients only deletes those
- Reflects new patch-like behavior of PUT endpoint

packages/api/src/routes/recipes.routes.real.test.ts

@@ -390,7 +390,7 @@ describe('Recipes Routes - Real Integration Tests', () => {
       expect(prisma.recipe.update).toHaveBeenCalled();
     });
 
-    it('should delete old relations before update', async () => {
+    it('should only delete relations that are being updated', async () => {
       (prisma.recipeSection.deleteMany as any).mockResolvedValue({});
       (prisma.ingredient.deleteMany as any).mockResolvedValue({});
       (prisma.instruction.deleteMany as any).mockResolvedValue({});
@@ -406,22 +406,46 @@ describe('Recipes Routes - Real Integration Tests', () => {
       });
       (prisma.cookbook.findMany as any).mockResolvedValue([]);
 
+      // Test 1: Only updating title - should not delete any relations
       await request(app)
         .put('/api/recipes/1')
         .send({ title: 'Updated' });
 
-      expect(prisma.recipeSection.deleteMany).toHaveBeenCalledWith({
-        where: { recipeId: '1' },
+      expect(prisma.recipeSection.deleteMany).not.toHaveBeenCalled();
+      expect(prisma.ingredient.deleteMany).not.toHaveBeenCalled();
+      expect(prisma.instruction.deleteMany).not.toHaveBeenCalled();
+      expect(prisma.recipeTag.deleteMany).not.toHaveBeenCalled();
+
+      // Reset mocks
+      vi.clearAllMocks();
+      (prisma.ingredient.deleteMany as any).mockResolvedValue({});
+      (prisma.recipeTag.deleteMany as any).mockResolvedValue({});
+      (prisma.recipe.update as any).mockResolvedValue({
+        id: '1',
+        title: 'Updated',
+        ingredients: [],
+        tags: [],
       });
+      (prisma.cookbook.findMany as any).mockResolvedValue([]);
+
+      // Test 2: Updating tags and ingredients - should only delete those
+      await request(app)
+        .put('/api/recipes/1')
+        .send({
+          title: 'Updated',
+          ingredients: [],
+          tags: []
+        });
+
       expect(prisma.ingredient.deleteMany).toHaveBeenCalledWith({
         where: { recipeId: '1' },
       });
-      expect(prisma.instruction.deleteMany).toHaveBeenCalledWith({
-        where: { recipeId: '1' },
-      });
       expect(prisma.recipeTag.deleteMany).toHaveBeenCalledWith({
         where: { recipeId: '1' },
       });
+      // These should NOT be called since we didn't send them
+      expect(prisma.recipeSection.deleteMany).not.toHaveBeenCalled();
+      expect(prisma.instruction.deleteMany).not.toHaveBeenCalled();
     });
 
     it('should return 500 on update error', async () => {