fix(useasync): unmount condition

src/useAsync.ts

@@ -23,12 +23,10 @@ export default function useAsync(
   }
 
   const run = useCallback(async (...args) => {
-    console.log(...args)
     const uid = ++ref.current.count
     try {
       setLoading(true)
       const value = await ref.current.fn(...args)
-      console.log('uid ======> ', uid, ref.current.count)
       if (uid === ref.current.count) {
         setValue(value)
         setError(null)
@@ -53,6 +51,10 @@ export default function useAsync(
       // tslint:disable-next-line:no-floating-promises
       run(...(deps as DependencyList))
     }
+    // ignore request after unmount
+    return () => {
+      ++ref.current.count
+    }
   }, [run])
 
   return { loading, value, error, run, cancel }

test/useAsync.spec.ts

@@ -157,7 +157,7 @@ describe('useAsync', () => {
         }, deley || 100)
       )
     }
-    const { result, rerender, waitForNextUpdate } = renderHook(
+    const { result, unmount, waitForNextUpdate } = renderHook(
       ({ fn, deps, immediate }: params) => useAsync(fn, deps, immediate),
       {
         initialProps: {
@@ -207,6 +207,19 @@ describe('useAsync', () => {
     expect(result.current.value).toBe('reject6')
     expect(result.current.error).toBe(null)
 
+    // unmount condition
+    act(() => {
+      result.current.run('unmount')
+    })
+    unmount()
+
+    jest.runAllTimers()
+    expect(fn).toBeCalledTimes(6)
+    expect(fn1).toBeCalledTimes(6)
+    expect(result.current.loading).toBe(true)
+    expect(result.current.value).toBe('reject6')
+    expect(result.current.error).toBe(null)
+
     jest.useRealTimers()
   })
 })