refactor: remove throwOnError option

it duplicate of the onError and the error state

Author: John60676

src/__tests__/index.test.tsx

@@ -244,7 +244,7 @@ describe('useRequest', () => {
       defineComponent({
         setup() {
           const { run } = useRequest(failedRequest, { manual: true });
-          const handleClick = () => run().catch(() => {}); // catch is needed or the node.js will be crash
+          const handleClick = () => run();
           return () => <button onClick={handleClick}></button>;
         },
       }),
@@ -254,31 +254,6 @@ describe('useRequest', () => {
     expect(console.error).toHaveBeenCalledWith(new Error('fail'));
   });
 
-  test('request error can be handle by user', async () => {
-    let errorText = '';
-
-    const wrapper = shallowMount(
-      defineComponent({
-        setup() {
-          const { run } = useRequest(failedRequest, { manual: true, throwOnError: true });
-
-          return () => (
-            <button
-              onClick={() =>
-                run().catch((err: Error) => {
-                  errorText = err.message;
-                })
-              }
-            ></button>
-          );
-        },
-      }),
-    );
-    await wrapper.find('button').trigger('click');
-    await waitForAll();
-    expect(errorText).toBe('fail');
-  });
-
   test('onSuccess should work', async () => {
     const mockSuccessCallback = jest.fn();
 
@@ -289,7 +264,7 @@ describe('useRequest', () => {
             manual: true,
             onSuccess: mockSuccessCallback,
           });
-          const handleClick = () => run().catch(() => {}); // catch is needed or the node.js will be crash
+          const handleClick = () => run();
           return () => <button onClick={handleClick}></button>;
         },
       }),
@@ -310,7 +285,7 @@ describe('useRequest', () => {
             manual: true,
             onError: mockErrorCallback,
           });
-          const handleClick = () => run().catch(() => {}); // catch is needed or the node.js will be crash
+          const handleClick = () => run();
           return () => <button onClick={handleClick}></button>;
         },
       }),

src/core/config.ts

@@ -29,7 +29,6 @@ export type GlobalOptions = {
   cacheTime?: number;
   // -1 means the cache is always valid
   staleTime?: number;
-  throwOnError?: boolean;
   manual?: boolean;
 };
 

src/core/createQuery.ts

@@ -56,7 +56,6 @@ const createQuery = <R, P extends unknown[]>(
 ): InnerQueryState<R, P> => {
   const {
     initialAutoRunFlag,
-    throwOnError,
     initialData,
     loadingDelay,
     pollingInterval,
@@ -159,17 +158,14 @@ const createQuery = <R, P extends unknown[]>(
             onError(error, args);
           }
 
-          if (throwOnError) {
-            throw error;
-          }
           console.error(error);
-          return Promise.reject('已处理的错误');
         }
         return resolvedPromise;
       })
       .finally(() => {
         if (currentCount === count.value) {
           cb?.();
+
           // clear delayLoadingTimer
           delayLoadingTimer.value();
           // run for polling

src/core/useAsyncQuery.ts

@@ -44,7 +44,6 @@ function useAsyncQuery<R, P extends unknown[], FR>(
     manual = false,
     ready = ref(true),
     refreshDeps = [],
-    throwOnError = false,
     loadingDelay = 0,
     pollingWhenHidden = false,
     refreshOnWindowFocus = false,
@@ -89,7 +88,6 @@ function useAsyncQuery<R, P extends unknown[], FR>(
     initialAutoRunFlag,
     initialData,
     loadingDelay,
-    throwOnError,
     pollingInterval,
     debounceInterval,
     throttleInterval,