fix: fetchJsonWithAuthToken missing options param (#4)

Co-authored-by: koo6666 <gujixian@xingfeiinc.com> Co-authored-by: Bojan Mihelac <bmihelac@mihelac.org>
bmihelac · Jul 1, 2020 · e51f17e · e51f17e
1 parent 60cc95d
commit e51f17e
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/src/tokenAuthProvider.ts b/src/tokenAuthProvider.ts
@@ -62,8 +62,12 @@ export function createOptionsFromToken() {
   };
 }
 
-export function fetchJsonWithAuthToken(url: string) {
-  return fetchUtils.fetchJson(url, createOptionsFromToken());
+export function fetchJsonWithAuthToken(url: string, options: object) {
+  console.log(options);
+  return fetchUtils.fetchJson(
+    url,
+    Object.assign(createOptionsFromToken(), options)
+  );
 }
 
 export default tokenAuthProvider;
diff --git a/test/tokenAuthProvider.test.ts b/test/tokenAuthProvider.test.ts
@@ -3,6 +3,7 @@ import fetchMock from 'fetch-mock-jest';
 
 import tokenAuthProvider, {
   createOptionsFromToken,
+  fetchJsonWithAuthToken,
 } from '../src/tokenAuthProvider';
 
 fetchMock.config.overwriteRoutes = true;
@@ -96,3 +97,12 @@ describe('createOptionsFromToken', () => {
     expect(createOptionsFromToken()).toEqual({});
   });
 });
+
+describe('fetchJsonWithAuthToken', function() {
+  fetchMock.patch('/', 200);
+  test('with options', () => {
+    fetchJsonWithAuthToken('/', {
+      method: 'PATCH',
+    });
+  });
+});