From 6917d944fe36addc79d4123aa792fe2c29746522 Mon Sep 17 00:00:00 2001
From: beatz174-bit <beatz174@gmail.com>
Date: Mon, 1 Dec 2025 22:00:27 +1000
Subject: [PATCH] Fix type issues in tests

---
 src/components/__tests__/AddProductDialog.test.tsx  |  2 +-
 src/components/__tests__/ProductRow.errors.test.tsx | 10 +++++++---
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/src/components/__tests__/AddProductDialog.test.tsx b/src/components/__tests__/AddProductDialog.test.tsx
index d10ed3a..51040a7 100644
--- a/src/components/__tests__/AddProductDialog.test.tsx
+++ b/src/components/__tests__/AddProductDialog.test.tsx
@@ -57,7 +57,7 @@ describe('AddProductDialog', () => {
   });
 
   it('applies initialBarcode and triggers lookup', async () => {
-    fetchProductSpy.mockResolvedValue({ name: 'From OFF' });
+    fetchProductSpy.mockResolvedValue({ name: 'From OFF', source: 'openfoodfacts' });
 
     render(
       <AddProductDialog
diff --git a/src/components/__tests__/ProductRow.errors.test.tsx b/src/components/__tests__/ProductRow.errors.test.tsx
index 50dcd94..9482b5a 100644
--- a/src/components/__tests__/ProductRow.errors.test.tsx
+++ b/src/components/__tests__/ProductRow.errors.test.tsx
@@ -2,6 +2,7 @@ import { render, screen, waitFor } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { describe, expect, it, vi, beforeEach } from 'vitest';
 import { ProductRow } from '../ProductRow';
+import type { Product } from '../../models/Product';
 import { makeNamedError } from '../../test/makeNamedError';
 
 vi.mock('../BarcodeScannerView', () => ({
@@ -18,20 +19,23 @@ const categoriesById = new Map([
   ['cat-2', 'Drinks'],
 ]);
 
-const baseProduct = {
+const baseProduct: Product = {
   id: 'p1',
   name: 'Product One',
   category: 'cat-1',
   unit_type: 'unit',
   bulk_name: 'carton',
+  barcode: undefined,
   archived: false,
   created_at: 0,
   updated_at: 0,
 };
 
 describe('ProductRow error handling', () => {
-  let onSave: ReturnType<typeof vi.fn>;
-  let onDelete: ReturnType<typeof vi.fn>;
+  let onSave: ReturnType<
+    typeof vi.fn<(productId: string, updates: { name: string; category: string; barcode?: string }) => void>
+  >;
+  let onDelete: ReturnType<typeof vi.fn<(productId: string) => void>>;
 
   beforeEach(() => {
     onSave = vi.fn();