Enhance API client tests with improved error handling and timeout settings

- Updated test cases in `apiClient.test.js` and `criticalFlows.test.js` to include timeout settings for better reliability during asynchronous operations.
- Modified error handling assertions to provide more descriptive messages for fetch errors, ensuring clarity in test outcomes.

Author: angoca

tests/api/apiClient.test.js

@@ -24,7 +24,7 @@ describe('APIClient', () => {
             expect(result[0]).toHaveProperty('username');
         });
 
-        it('should handle fetch errors', async () => {
+        it('should handle fetch errors', { timeout: 10000 }, async () => {
             global.fetch = vi.fn(() =>
                 Promise.reject(new Error('Network error'))
             );
@@ -98,7 +98,8 @@ describe('APIClient', () => {
                 })
             );
 
-            await expect(apiClient.getUserIndex()).rejects.toThrow('HTTP error');
+            // The API client transforms 404 errors to a more descriptive message
+            await expect(apiClient.getUserIndex()).rejects.toThrow('Resource not found');
         });
     });
 });

tests/integration/criticalFlows.test.js

@@ -75,7 +75,7 @@ describe('Critical User Flows', () => {
             expect(filtered.length).toBeGreaterThan(0);
         });
 
-        it('should handle search errors gracefully', async () => {
+        it('should handle search errors gracefully', { timeout: 10000 }, async () => {
             // Clear cache first
             apiClient.clearCache();
 
@@ -253,7 +253,7 @@ describe('Critical User Flows', () => {
     });
 
     describe('Flow 9: Error Handling', () => {
-        it('should handle network errors', async () => {
+        it('should handle network errors', { timeout: 10000 }, async () => {
             global.fetch.mockRejectedValue(new Error('Network error'));
 
             await expect(apiClient.getMetadata()).rejects.toThrow();