Bug/response interceptor cache (#236)

* v0.4.4

* v0.4.5

* v1.0.0

* should fix, need to add tests

* adding test

README.md

@@ -1041,6 +1041,10 @@ Todos
     retryOnError: false,
     refreshWhenHidden: false,
   })
+
+
+  // potential for causing a rerender after clearing cache if needed
+  request.cache.clear(true)
   ```
 
 - [ ] potential option ideas for `GraphQL`

package.json

@@ -1,6 +1,6 @@
 {
   "name": "use-http",
-  "version": "0.4.5",
+  "version": "1.0.0",
   "homepage": "http://use-http.com",
   "main": "dist/index.js",
   "license": "MIT",

src/__tests__/useFetch.test.tsx

@@ -602,6 +602,29 @@ describe('useFetch - BROWSER - interceptors', (): void => {
     await act(result.current.get)
     expect((fetch.mock.calls[0][1] as any).data).toEqual('url')
   })
+
+  it('should still call both interceptors when using cache', async (): Promise<void> => {
+    let requestCalled = 0
+    let responseCalled = 0
+    const { result, waitForNextUpdate } = renderHook(
+      () => useFetch('url', {
+        interceptors: {
+          async request(options) {
+            requestCalled++
+            return options
+          },
+          async response(response) {
+            responseCalled++
+            return response
+          }
+        }
+      }, []),
+    )
+    await waitForNextUpdate()
+    await act(result.current.get)
+    expect(requestCalled).toBe(2)
+    expect(responseCalled).toBe(2)
+  })
 })
 
 describe('useFetch - BROWSER - Overwrite Global Options set in Provider', (): void => {
@@ -813,7 +836,7 @@ describe('useFetch - BROWSER - retryOn & retryDelay', (): void => {
 
     it('should retryOn specific error codes', async (): Promise<void> => {
       const { result, waitForNextUpdate } = renderHook(
-        () => useFetch('url', {
+        () => useFetch('some-url-1234124', {
           retries: 2,
           retryOn: [401]
         }, [])

src/useFetch.ts

@@ -94,9 +94,11 @@ function useFetch<TData = any>(...args: UseFetchArgs): UseFetch<TData> {
       if (response.isCached && cachePolicy === CACHE_FIRST) {
         try {
           res.current = response.cached as Res<TData>
-          const d = await tryGetData(response.cached, defaults.data)
-          res.current.data = d
-          data.current = d
+          const theData = await tryGetData(response.cached, defaults.data)
+          res.current.data = theData
+          res.current = interceptors.response ? await interceptors.response(res.current) : res.current
+          invariant('data' in res.current, 'You must have `data` field on the Response returned from your `interceptors.response`')
+          data.current = res.current.data as TData
           if (!suspense && mounted.current) forceUpdate()
           return data.current
         } catch (err) {