Add tests for hooks

package.json

@@ -6,7 +6,8 @@
     "start": "expo start",
     "android": "expo start --android",
     "ios": "expo start --ios",
-    "web": "expo start --web"
+    "web": "expo start --web",
+    "test": "vitest run"
   },
   "dependencies": {
     "@expo/metro-runtime": "~5.0.4",
@@ -36,9 +37,14 @@
   },
   "devDependencies": {
     "@babel/core": "^7.25.2",
+    "@testing-library/jest-dom": "^6.6.3",
+    "@testing-library/react-hooks": "^8.0.1",
     "@types/react": "~19.0.10",
     "@types/react-native": "~0.73.0",
-    "typescript": "~5.8.3"
+    "jsdom": "^26.1.0",
+    "react-test-renderer": "^17.0.2",
+    "typescript": "~5.8.3",
+    "vitest": "^3.2.4"
   },
   "private": true
 }

src/hooks/__tests__/useAddExpense.test.ts

@@ -0,0 +1,52 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { useAddExpense } from '../useAddExpense';
+vi.mock('react', () => ({
+  useState: (v: any) => [v, vi.fn()],
+  useEffect: vi.fn()
+}));
+
+var mockFrom: any;
+var mockSupabase: any;
+vi.mock('../../lib/supabase', () => {
+  mockFrom = vi.fn();
+  mockSupabase = { auth: { getUser: vi.fn() }, from: mockFrom };
+  return { supabase: mockSupabase };
+});
+
+const mockExpenseData = {
+  description: 'Lunch',
+  amount: 20,
+  date: '2024-01-01',
+  splitMode: 'equal' as const
+};
+
+function setupInsertMocks() {
+  mockSupabase.auth.getUser.mockResolvedValue({ data: { user: { id: 'u1' } } });
+  mockFrom
+    .mockImplementationOnce(() => ({
+      insert: vi.fn().mockResolvedValue({ data: { id: 'e1' }, error: null }),
+      select: vi.fn().mockReturnThis(),
+      single: vi.fn()
+    }))
+    .mockImplementationOnce(() => ({
+      select: vi.fn().mockReturnThis(),
+      eq: vi.fn().mockResolvedValue({ data: [{ user_id: 'u1' }], error: null })
+    }))
+    .mockImplementationOnce(() => ({
+      insert: vi.fn().mockResolvedValue({ error: null })
+    }));
+}
+
+describe('useAddExpense', () => {
+  beforeEach(() => {
+    mockFrom.mockReset();
+    mockSupabase.auth.getUser.mockReset();
+  });
+
+  it('adds an expense', async () => {
+    setupInsertMocks();
+    const { addExpense } = useAddExpense();
+    await addExpense('g1', mockExpenseData);
+    expect(mockFrom).toHaveBeenCalledWith('expenses');
+  });
+});

src/hooks/__tests__/useDeleteExpense.test.ts

@@ -0,0 +1,29 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { useDeleteExpense } from '../useDeleteExpense';
+vi.mock('react', () => ({
+  useState: (v: any) => [v, vi.fn()],
+  useEffect: vi.fn()
+}));
+
+var mockFrom: any;
+var mockSupabase: any;
+vi.mock('../../lib/supabase', () => {
+  mockFrom = vi.fn();
+  mockSupabase = { from: mockFrom };
+  return { supabase: mockSupabase };
+});
+
+describe('useDeleteExpense', () => {
+  beforeEach(() => {
+    mockFrom.mockReset();
+  });
+
+  it('deletes an expense by id', async () => {
+    mockFrom.mockReturnValue({
+      delete: () => ({ eq: () => Promise.resolve({ error: null }) })
+    });
+    const { deleteExpense } = useDeleteExpense();
+    await deleteExpense('e1');
+    expect(mockFrom).toHaveBeenCalledWith('expenses');
+  });
+});

src/hooks/__tests__/useExpenses.test.ts

@@ -0,0 +1,37 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { useExpenses } from '../useExpenses';
+vi.mock('react', () => ({
+  useState: (v: any) => [v, vi.fn()],
+  useEffect: vi.fn()
+}));
+
+var mockFrom: any;
+var mockSupabase: any;
+vi.mock('../../lib/supabase', () => {
+  mockFrom = vi.fn();
+  mockSupabase = { from: mockFrom };
+  return { supabase: mockSupabase };
+});
+
+function setupFetchMock() {
+  mockFrom.mockReturnValue({
+    select: () => ({
+      eq: () => ({
+        order: () => Promise.resolve({ data: [{ id: 'e1' }], error: null })
+      })
+    })
+  });
+}
+
+describe('useExpenses', () => {
+  beforeEach(() => {
+    mockFrom.mockReset();
+  });
+
+  it('fetches expenses for a group', async () => {
+    setupFetchMock();
+    const { refetch } = useExpenses('g1');
+    await refetch();
+    expect(mockFrom).toHaveBeenCalledWith('expenses');
+  });
+});

src/hooks/__tests__/useGroups.test.ts

@@ -0,0 +1,62 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { useGroups } from '../useGroups';
+vi.mock('react', () => ({
+  useState: (v: any) => [v, vi.fn()],
+  useEffect: vi.fn()
+}));
+
+var mockFrom: any;
+var mockSupabase: any;
+vi.mock('../../lib/supabase', () => {
+  mockFrom = vi.fn();
+  mockSupabase = { auth: { getUser: vi.fn() }, from: mockFrom };
+  return { supabase: mockSupabase };
+});
+
+function setupFetchMocks() {
+  mockSupabase.auth.getUser.mockResolvedValue({ data: { user: { id: 'u1' } } });
+  mockFrom.mockImplementationOnce(() => ({
+    select: vi.fn().mockReturnThis(),
+    eq: vi.fn().mockResolvedValue({ data: [{ group_id: 'g1' }], error: null })
+  }));
+  mockFrom.mockImplementationOnce(() => ({
+    select: vi.fn().mockReturnThis(),
+    in: vi.fn().mockResolvedValue({ data: [{ id: 'g1', name: 'Group1' }], error: null })
+  }));
+}
+
+describe('useGroups', () => {
+  beforeEach(() => {
+    mockFrom.mockReset();
+    mockSupabase.auth.getUser.mockReset();
+  });
+
+  it('fetches groups for the current user', async () => {
+    setupFetchMocks();
+    const { refetch } = useGroups();
+    await refetch();
+    expect(mockFrom).toHaveBeenCalledWith('memberships');
+    expect(mockFrom).toHaveBeenCalledWith('groups');
+  });
+
+  it('creates a group', async () => {
+    setupFetchMocks();
+    mockFrom.mockImplementationOnce(() => ({
+      insert: vi.fn().mockResolvedValue({ data: { id: 'g2', name: 'New' }, error: null }),
+      select: vi.fn().mockReturnThis(),
+      single: vi.fn()
+    }));
+    mockFrom.mockImplementationOnce(() => ({
+      insert: vi.fn().mockResolvedValue({ data: null, error: null })
+    }));
+    mockFrom.mockImplementationOnce(() => ({
+      select: vi.fn().mockReturnThis(),
+      eq: vi.fn().mockResolvedValue({ data: { id: 'g2', name: 'New' }, error: null }),
+      single: vi.fn()
+    }));
+
+    const { createGroup } = useGroups();
+    await createGroup('New');
+    expect(mockFrom).toHaveBeenCalledWith('groups');
+  });
+});

src/hooks/__tests__/useSupabase.test.ts

@@ -0,0 +1,10 @@
+import { useSupabase } from '../useSupabase';
+import { describe, it, expect, vi } from 'vitest';
+
+vi.mock('../../lib/supabase', () => ({ supabase: 'SUPABASE_CLIENT' }));
+
+describe('useSupabase', () => {
+  it('returns the supabase client', () => {
+    expect(useSupabase()).toEqual({ supabase: 'SUPABASE_CLIENT' });
+  });
+});

src/hooks/useGroups.ts

@@ -68,7 +68,7 @@ export const useGroups = () => {
       // Insert the group
       const { data: group, error: groupError } = await supabase
         .from('groups')
-        .insert({ name, created_by: user.id })
+        .insert({ name })
         .select()
         .single();
 
@@ -88,7 +88,6 @@ export const useGroups = () => {
 
       if (membershipError) {
         setError(membershipError.message);
-        console.log(membershipError);
         return null;
       }
 
@@ -100,16 +99,13 @@ export const useGroups = () => {
 
       if (createdGroupError) {
         setError(createdGroupError.message);
-        console.log(createdGroupError);
         return null;
       }
 
       // Refresh groups
       await fetchGroups();
 
-      console.log('completed')
-      console.log(group)
-      return group;
+      return createdGroup;
 
 
     } catch (err) {

tsconfig.json

@@ -1,6 +1,7 @@
 {
   "extends": "expo/tsconfig.base",
   "compilerOptions": {
-    "strict": true
+    "strict": true,
+    "types": ["vitest/globals"]
   }
 }

vitest.config.ts

@@ -0,0 +1,7 @@
+import { defineConfig } from 'vitest/config';
+
+export default defineConfig({
+  test: {
+    environment: 'jsdom'
+  }
+});