fix some uploading flyer issues + more unit tests

src/hooks/useFlyerUploader.ts

@@ -44,11 +44,16 @@ export const useFlyerUploader = () => {
     enabled: !!jobId,
     // Polling logic: react-query handles the interval
     refetchInterval: (query) => {
-      const data = query.state.data;
+      const data = query.state.data as JobStatus | undefined;
       // Stop polling if the job is completed or has failed
       if (data?.state === 'completed' || data?.state === 'failed') {
         return false;
       }
+      // Also stop polling if the query itself has errored (e.g. network error, or JobFailedError thrown from getJobStatus)
+      if (query.state.status === 'error') {
+        logger.warn('[useFlyerUploader] Polling stopped due to query error state.');
+        return false;
+      }
       // Otherwise, poll every 3 seconds
       return 3000;
     },

src/hooks/useShoppingLists.test.tsx

@@ -495,6 +495,22 @@ describe('useShoppingLists Hook', () => {
       expect(currentLists[0].items).toHaveLength(1); // Length should remain 1
       console.log('  LOG: SUCCESS! Duplicate was not added and API was not called.');
     });
+
+    it('should log an error and not call the API if the listId does not exist', async () => {
+      const consoleErrorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+      const { result } = renderHook(() => useShoppingLists());
+
+      await act(async () => {
+        // Call with a non-existent list ID (mock lists have IDs 1 and 2)
+        await result.current.addItemToList(999, { customItemName: 'Wont be added' });
+      });
+
+      // The API should not have been called because the list was not found.
+      expect(mockAddItemApi).not.toHaveBeenCalled();
+      expect(consoleErrorSpy).toHaveBeenCalledWith('useShoppingLists: List with ID 999 not found.');
+
+      consoleErrorSpy.mockRestore();
+    });
   });
 
   describe('updateItemInList', () => {
@@ -689,11 +705,21 @@ describe('useShoppingLists Hook', () => {
         setupApiMocks(apiMocksWithError);
         apiMock.mockRejectedValue(new Error(errorMessage));
 
+        // Spy on console.error to ensure the catch block is executed for logging
+        const consoleErrorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+
         const { result } = renderHook(() => useShoppingLists());
         await act(async () => {
           await action(result.current);
         });
-        await waitFor(() => expect(result.current.error).toBe(errorMessage));
+
+        await waitFor(() => {
+          expect(result.current.error).toBe(errorMessage);
+          // Verify that our custom logging within the catch block was called
+          expect(consoleErrorSpy).toHaveBeenCalled();
+        });
+
+        consoleErrorSpy.mockRestore();
       },
     );
   });