feat: add product type to product mutations and queries, update user termsAccepted field

Author: Alejandro Caicedo

__tests__/modules/products/application/product.mutations.test.ts

@@ -56,11 +56,11 @@ describe('product.mutations', () => {
 
   describe('useProductCreate', () => {
     it('should create product and invalidate queries on success', async () => {
-      const mockProduct = { id: '1', name: 'New Product', description: 'Desc', price: 100, createdAt: new Date(), updatedAt: new Date() };
+      const mockProduct = { id: '1', name: 'New Product', description: 'Desc', price: 100, type: 'comida' as const, createdAt: new Date(), updatedAt: new Date() };
       mockProductService.create.mockResolvedValue(mockProduct);
 
       const { mutate } = useProductCreate();
-      await mutate({ name: 'New Product', description: 'Desc', price: 100 });
+      await mutate({ name: 'New Product', description: 'Desc', price: 100, type: { label: 'Comida', value: 'comida' } });
 
       expect(mockProductService.create).toHaveBeenCalled();
       expect(mockInvalidateQueries).toHaveBeenCalledWith({
@@ -69,11 +69,11 @@ describe('product.mutations', () => {
     });
 
     it('should show success toast on create', async () => {
-      const mockProduct = { id: '1', name: 'New Product', description: 'Desc', price: 100, createdAt: new Date(), updatedAt: new Date() };
+      const mockProduct = { id: '1', name: 'New Product', description: 'Desc', price: 100, type: 'comida' as const, createdAt: new Date(), updatedAt: new Date() };
       mockProductService.create.mockResolvedValue(mockProduct);
 
       const { mutate } = useProductCreate();
-      await mutate({ name: 'New Product', description: 'Desc', price: 100 });
+      await mutate({ name: 'New Product', description: 'Desc', price: 100, type: { label: 'Comida', value: 'comida' } });
 
       // Toast is called via useAppStorage mock
     });
@@ -85,17 +85,17 @@ describe('product.mutations', () => {
       const { mutate } = useProductCreate();
 
       // The mutation will throw because the service returns an Error
-      await expect(mutate({ name: 'Test', description: 'Desc', price: 100 })).rejects.toThrow();
+      await expect(mutate({ name: 'Test', description: 'Desc', price: 100, type: { label: 'Comida', value: 'comida' } })).rejects.toThrow();
     });
   });
 
   describe('useProductUpdate', () => {
     it('should update product and invalidate queries on success', async () => {
-      const mockProduct = { id: '1', name: 'Updated Product', description: 'Desc', price: 200, createdAt: new Date(), updatedAt: new Date() };
+      const mockProduct = { id: '1', name: 'Updated Product', description: 'Desc', price: 200, type: 'comida' as const, createdAt: new Date(), updatedAt: new Date() };
       mockProductService.update.mockResolvedValue(mockProduct);
 
       const { mutate } = useProductUpdate();
-      await mutate({ id: '1', form: { name: 'Updated Product', description: 'Desc', price: 200 } });
+      await mutate({ id: '1', form: { name: 'Updated Product', description: 'Desc', price: 200, type: { label: 'Comida', value: 'comida' } } });
 
       expect(mockProductService.update).toHaveBeenCalledWith('1', expect.any(Object));
       expect(mockInvalidateQueries).toHaveBeenCalledWith({
@@ -113,7 +113,7 @@ describe('product.mutations', () => {
       const { mutate } = useProductUpdate();
 
       await expect(
-        mutate({ id: '1', form: { name: 'Test', description: 'Desc', price: 100 } }),
+        mutate({ id: '1', form: { name: 'Test', description: 'Desc', price: 100, type: { label: 'Comida', value: 'comida' } } }),
       ).rejects.toThrow();
     });
   });

__tests__/modules/products/application/product.queries.test.ts

@@ -58,7 +58,7 @@ describe('product.queries', () => {
     });
 
     it('should call productService.getAll in queryFn', async () => {
-      const mockProducts = [{ id: '1', name: 'Product 1', description: 'Desc', price: 100, createdAt: new Date(), updatedAt: new Date() }];
+      const mockProducts = [{ id: '1', name: 'Product 1', description: 'Desc', price: 100, type: 'comida' as const, createdAt: new Date(), updatedAt: new Date() }];
       mockProductService.getAll.mockResolvedValue(mockProducts);
       mockUseQuery.mockImplementation(async ({ queryFn }) => {
         await queryFn();
@@ -114,7 +114,7 @@ describe('product.queries', () => {
     });
 
     it('should call productService.getById in queryFn', async () => {
-      const mockProduct = { id: '1', name: 'Product 1', description: 'Desc', price: 100, createdAt: new Date(), updatedAt: new Date() };
+      const mockProduct = { id: '1', name: 'Product 1', description: 'Desc', price: 100, type: 'comida' as const, createdAt: new Date(), updatedAt: new Date() };
       mockProductService.getById.mockResolvedValue(mockProduct);
       mockUseQuery.mockImplementation(async ({ queryFn }) => {
         const data = await queryFn();

src/modules/products/infrastructure/product.firebase.service.ts

@@ -18,6 +18,7 @@ interface ProductFirebaseDoc {
   name: string;
   description: string;
   price: number;
+  type: string;
   createdAt: Timestamp;
   updatedAt: Timestamp;
 }
@@ -29,6 +30,7 @@ interface ProductFirebaseEntity extends ProductFirebaseDoc {
 function toProduct(entity: ProductFirebaseEntity): Product {
   return {
     ...entity,
+    type: entity.type as Product['type'],
     createdAt: new Date(entity.createdAt.seconds * 1000),
     updatedAt: new Date(entity.updatedAt.seconds * 1000),
   };

src/modules/products/ui/components/ProductItem.tsx

@@ -34,7 +34,7 @@ export const ProductItem = React.memo(function ProductItem({
         <Text variant="caption" color="primary">
           ${product.price.toFixed(2)}
         </Text>
-        <Text variant="caption" color="muted">
+        <Text variant="caption" color="textSecondary">
           {product.type}
         </Text>
       </View>

src/modules/users/infrastructure/user.firebase.service.ts

@@ -21,6 +21,7 @@ interface UserFirebaseDoc {
   phone: string;
   role: string;
   avatar?: string;
+  termsAccepted: boolean;
   createdAt: Timestamp;
   updatedAt: Timestamp;
 }
@@ -33,6 +34,7 @@ function toUser(entity: UserFirebaseEntity): User {
   return {
     ...entity,
     avatar: entity.avatar ?? undefined,
+    termsAccepted: entity.termsAccepted ?? false,
     createdAt: new Date(entity.createdAt.seconds * 1000),
     updatedAt: new Date(entity.updatedAt.seconds * 1000),
   };
@@ -144,6 +146,7 @@ class UserFirebaseService implements UserRepository {
           email: data.email,
           phone: data.phone,
           role: data.role,
+          termsAccepted: data.termsAccepted,
           createdAt: Timestamp.fromDate(now),
           updatedAt: Timestamp.fromDate(now),
         },