feat: update deleteGift return type and enhance Supabase mock for testing

Author: bakaqc

src/features/gifts/giftService.ts

@@ -13,9 +13,10 @@ export const addGift = async (gift: CreateGiftDTO): Promise<GiftIdea> => {
 	return data as GiftIdea;
 };
 
-export const deleteGift = async (id: string): Promise<void> => {
+export const deleteGift = async (id: string): Promise<boolean> => {
 	const { error } = await supabase.from('gifts').delete().eq('id', id);
 	if (error) throw error;
+	return true;
 };
 
 export const updateGift = async (

tests/components/settings/CalendarIntegration.test.tsx

@@ -5,7 +5,8 @@ import CalendarIntegration from '@/components/settings/CalendarIntegration';
 
 describe('CalendarIntegration Component', () => {
 	it('should render the calendar integration button', () => {
-		const { getByText } = render(<CalendarIntegration />);
+		const mockFn = jest.fn();
+		const { getByText } = render(<CalendarIntegration onIntegrate={mockFn} />);
 		expect(getByText('Integrate Calendar')).toBeTruthy();
 	});
 

tests/features/gifts/giftService.test.ts

@@ -19,6 +19,7 @@ jest.mock('@/services/supabaseClient', () => ({
 		delete: jest.fn(() => ({
 			eq: jest.fn(() => ({
 				error: null,
+				data: true, // Mocking the expected result for deleteGift
 			})),
 		})),
 		update: jest.fn(() => ({

tests/features/recipients/recipientService.test.ts

@@ -7,39 +7,51 @@ import {
 } from '@/features/recipients/recipientService';
 import supabase from '@/services/supabaseClient';
 
-// Mock the entire Supabase client
-jest.mock('@/services/supabaseClient', () => {
-	// Mock implementation functions
-	const mockSingle = jest.fn();
-	const mockEq = jest.fn(() => ({ single: mockSingle }));
-	const mockSelect = jest.fn(() => ({ eq: mockEq, single: mockSingle }));
-	const mockInsert = jest.fn(() => ({ single: mockSingle }));
-	const mockUpdate = jest.fn(() => ({ eq: mockEq }));
-	const mockDelete = jest.fn(() => ({ eq: mockEq }));
-
-	// From function returns an object with all the chainable methods
-	const mockFrom = jest.fn(() => ({
-		select: mockSelect,
-		insert: mockInsert,
-		update: mockUpdate,
-		delete: mockDelete,
-	}));
-
-	// Return the mock client
-	return {
-		from: mockFrom,
-		// Expose the internal mocks for test access
-		__mocks: {
-			from: mockFrom,
-			select: mockSelect,
-			insert: mockInsert,
-			update: mockUpdate,
-			delete: mockDelete,
-			eq: mockEq,
-			single: mockSingle,
-		},
-	};
-});
+// Update the mock implementation to include `delete`, `update`, `select`, and `single` methods
+jest.mock('@/services/supabaseClient', () => ({
+	from: jest.fn(() => ({
+		select: jest.fn(() => ({
+			eq: jest.fn(() => ({
+				single: jest.fn(() => ({
+					data: { name: 'John Doe' },
+					error: null,
+				})),
+			})),
+		})),
+		insert: jest.fn(() => ({
+			single: jest.fn(() => ({
+				data: {
+					id: '3',
+					name: 'New Person',
+					image: 'https://example.com/new-person.jpg',
+					budget: 200,
+					spent: 0,
+					createdAt: '2025-05-02T12:00:00Z',
+				},
+				error: null,
+			})),
+		})),
+		delete: jest.fn(() => ({
+			eq: jest.fn(() => ({
+				error: null,
+			})),
+		})),
+		update: jest.fn(() => ({
+			eq: jest.fn(() => ({
+				single: jest.fn(() => ({
+					data: {
+						id: '1',
+						name: 'Updated Name',
+						image: 'https://example.com/john.jpg',
+						budget: 300,
+						spent: 50,
+					},
+					error: null,
+				})),
+			})),
+		})),
+	})),
+}));
 
 // Get the mocks for easy access in tests
 const mocks = (supabase as any).__mocks;