Fix: Make config optional on decorated Axios

src/api/api-client.js

@@ -13,7 +13,7 @@ export const ApiClient = (apiFactory) => (axiosInstance, options = {}) => (req,
   const controller = new AbortController();
   res.on('close', () => controller.abort());
 
-  const decorateWithoutData = (axiosMethod) => async (url, config) => axiosMethod(url, {
+  const decorateWithoutData = (axiosMethod) => async (url, config = {}) => axiosMethod(url, {
     ...config,
     signal: controller.signal,
     headers: {
@@ -22,7 +22,7 @@ export const ApiClient = (apiFactory) => (axiosInstance, options = {}) => (req,
     },
   });
 
-  const decorateWithData = (axiosMethod) => async (url, data, config) => axiosMethod(url, data, {
+  const decorateWithData = (axiosMethod) => async (url, data, config = {}) => axiosMethod(url, data, {
     ...config,
     signal: controller.signal,
     headers: {

src/api/api-client.test.js

@@ -29,11 +29,13 @@ const TestApiClient = ApiClient((axiosInstance) => ({
       'accept': 'application/samples+json;charset=UTF-8',
     },
   }),
+  listNoConfig: () => axiosInstance.get('/samples'),
   createSample: (sample) => axiosInstance.post('/samples', sample, {
     headers: {
       'accept': 'application/samples+json;charset=UTF-8',
     }
   }),
+  createNoConfig: (sample) => axiosInstance.post('/samples', sample),
 }));
 
 test('should make GET API calls as configured', async () => {
@@ -71,6 +73,37 @@ test('should make GET API calls as configured', async () => {
   scope.done();
 });
 
+test('should make GET API calls without config', async () => {
+  const req = MockRequest();
+  const res = MockResponse();
+
+  const axiosInstance = axios.create({
+    baseURL: 'https://api.quickcase.app',
+  });
+  const client = TestApiClient(axiosInstance)(req, res);
+
+  const scope = nock('https://api.quickcase.app')
+    .get('/samples')
+    .reply(200, {
+      samples: [
+        {id: 1},
+        {id: 2},
+      ],
+    });
+
+  const listResponse = await client.listNoConfig();
+
+  expect(listResponse.status).toBe(200);
+  expect(listResponse.data).toEqual({
+    samples: [
+      {id: 1},
+      {id: 2},
+    ],
+  });
+
+  scope.done();
+});
+
 test('should authorize GET API call with provided access token', async () => {
   const req = MockRequest();
   const res = MockResponse();
@@ -142,6 +175,35 @@ test('should make POST API calls as configured', async () => {
   scope.done();
 });
 
+test('should make POST API calls without config', async () => {
+  const req = MockRequest();
+  const res = MockResponse();
+
+  const axiosInstance = axios.create({
+    baseURL: 'https://api.quickcase.app',
+  });
+  const client = TestApiClient(axiosInstance)(req, res);
+
+  const sample = {name: 'Some sample'};
+
+  const scope = nock('https://api.quickcase.app')
+    .post('/samples', sample)
+    .reply(201, {
+      id: '1',
+      ...sample,
+    });
+
+  const createResponse = await client.createNoConfig(sample);
+
+  expect(createResponse.status).toBe(201);
+  expect(createResponse.data).toEqual({
+    id: '1',
+    name: 'Some sample',
+  });
+
+  scope.done();
+});
+
 test('should authorize POST API call with provided access token', async () => {
   const req = MockRequest();
   const res = MockResponse();