Issue 27 return empty object on 204 responses (#28)

Addresses #27 by returning
an empty object for 204 responses to avoid json parsing errors while
keeping with current conventions for backward compatability.

src/index.test.ts

@@ -249,6 +249,25 @@ describe('post()', () => {
       body: { foo: 'Bar' },
     });
   });
+
+  it('returns empty object on 204', async () => {
+    const client = createClient<paths>();
+    fetchMocker.mockResponseOnce(() => ({ status: 204, body: '' }));
+    const { data, error, response } = await client.post('/create-tag/{name}', {
+      params: { path: { name: 'New Tag' } },
+      body: { description: 'This is a new tag' },
+    });
+
+    // assert correct URL was called
+    expect(fetchMocker.mock.calls[0][0]).toBe('/create-tag/New%20Tag');
+
+    // assert correct data was returned
+    expect(data).toEqual({});
+    expect(response.status).toBe(204);
+
+    // assert error is empty
+    expect(error).toBe(undefined);
+  });
 });
 
 describe('delete()', () => {
@@ -258,6 +277,24 @@ describe('delete()', () => {
     await client.del('/' as any, {});
     expect(fetchMocker.mock.calls[0][1]?.method).toBe('DELETE');
   });
+
+  it('returns empty object on 204', async () => {
+    const client = createClient<paths>();
+    fetchMocker.mockResponseOnce(() => ({ status: 204, body: '' }));
+    const { data, error, response } = await client.del('/post/{post_id}', {
+      params: { path: { post_id: '123' } },
+    });
+
+    // assert correct URL was called
+    expect(fetchMocker.mock.calls[0][0]).toBe('/post/123');
+
+    // assert correct data was returned
+    expect(data).toEqual({});
+    expect(response.status).toBe(204);
+
+    // assert error is empty
+    expect(error).toBe(undefined);
+  });
 });
 
 describe('options()', () => {

src/index.ts

@@ -139,7 +139,7 @@ export default function createClient<T>(options?: ClientOptions) {
       type: res.type,
       url: res.url,
     };
-    return res.ok ? { data: await res.json(), response } : { error: await res.json(), response };
+    return res.ok ? { data: res.status === 204 ? {} : await res.json(), response } : { error: await res.json(), response };
   }
 
   return {