test(fetcher): cover JSON parsing error catch block

- Add test case for when error response JSON parsing fails
- Ensures the catch block fallback to default error message is covered
- Now at 77 total tests with complete line coverage
- Fixes missing coverage on line 59 in src/api/fetcher.ts

This should finally achieve true 100% coverage on Codecov! 🎯

Author: marklearst

tests/api/fetcher.test.ts

@@ -1,48 +1,59 @@
-import { describe, it, expect, vi } from 'vitest';
-import { fetcher } from '../../src/api/fetcher';
+import { describe, it, expect, vi } from 'vitest'
+import { fetcher } from '../../src/api/fetcher'
 
-const DUMMY_URL = 'https://api.example.com/test';
-const DUMMY_TOKEN = 'dummy-token';
+const DUMMY_URL = 'https://api.example.com/test'
+const DUMMY_TOKEN = 'dummy-token'
 
 // Helper to mock fetch
-function mockFetch(response: Partial<Response> & { json?: () => any }, ok = true) {
+function mockFetch(
+  response: Partial<Response> & { json?: () => any },
+  ok = true
+) {
   global.fetch = vi.fn().mockResolvedValue({
     ok,
     json: response.json || (() => Promise.resolve(response)),
     ...response,
-  }) as any;
+  }) as any
 }
 
 describe('fetcher', () => {
   afterEach(() => {
-    vi.resetAllMocks();
-  });
+    vi.resetAllMocks()
+  })
 
   it('throws if token is missing', async () => {
-    await expect(fetcher(DUMMY_URL, ''))
-      .rejects.toThrow(/token/i);
-  });
+    await expect(fetcher(DUMMY_URL, '')).rejects.toThrow(/token/i)
+  })
 
   it('returns JSON on success', async () => {
-    const data = { foo: 'bar' };
-    mockFetch({ json: () => Promise.resolve(data) });
-    const result = await fetcher(DUMMY_URL, DUMMY_TOKEN);
-    expect(result).toEqual(data);
-    expect(global.fetch).toHaveBeenCalledWith(DUMMY_URL, expect.objectContaining({
-      headers: expect.objectContaining({
-        'X-FIGMA-TOKEN': DUMMY_TOKEN,
-        'Content-Type': 'application/json',
-      }),
-    }));
-  });
+    const data = { foo: 'bar' }
+    mockFetch({ json: () => Promise.resolve(data) })
+    const result = await fetcher(DUMMY_URL, DUMMY_TOKEN)
+    expect(result).toEqual(data)
+    expect(global.fetch).toHaveBeenCalledWith(
+      DUMMY_URL,
+      expect.objectContaining({
+        headers: expect.objectContaining({
+          'X-FIGMA-TOKEN': DUMMY_TOKEN,
+          'Content-Type': 'application/json',
+        }),
+      })
+    )
+  })
 
   it('throws with error message if response is not ok and error message exists', async () => {
-    mockFetch({ json: () => Promise.resolve({ message: 'fail!' }) }, false);
-    await expect(fetcher(DUMMY_URL, DUMMY_TOKEN)).rejects.toThrow('fail!');
-  });
+    mockFetch({ json: () => Promise.resolve({ message: 'fail!' }) }, false)
+    await expect(fetcher(DUMMY_URL, DUMMY_TOKEN)).rejects.toThrow('fail!')
+  })
 
   it('throws with fallback error if response is not ok and no message', async () => {
-    mockFetch({ json: () => Promise.resolve({}) }, false);
-    await expect(fetcher(DUMMY_URL, DUMMY_TOKEN)).rejects.toThrow(/fetch/i);
-  });
-});
+    mockFetch({ json: () => Promise.resolve({}) }, false)
+    await expect(fetcher(DUMMY_URL, DUMMY_TOKEN)).rejects.toThrow(/fetch/i)
+  })
+
+  it('throws with fallback error if response is not ok and JSON parsing fails', async () => {
+    // Mock a response that fails JSON parsing
+    mockFetch({ json: () => Promise.reject(new Error('Invalid JSON')) }, false)
+    await expect(fetcher(DUMMY_URL, DUMMY_TOKEN)).rejects.toThrow(/fetch/i)
+  })
+})