use-http-request: allow to perform non-abortable requests

Author: nebarf

src/__tests__/use-http-request.spec.ts

@@ -191,7 +191,7 @@ describe('use-http-request', () => {
     expect(fetchHeaders).toEqual(defaultHttpReqConfig.reqOptions.headers);
     expect(fetchCredentials).toBeUndefined();
     expect(fetchBody).toBeNull();
-    expect(fetchSignal).toBeInstanceOf(AbortSignal);
+    expect(fetchSignal).toBeUndefined();
   });
 
   test('should merge params provided on hook declaration with the ones provided on request run', async () => {

src/request/types.ts

@@ -19,5 +19,6 @@ export interface UseHttpAbortableRequestReturn<HttpResponseT> {
 
 export type UseHttpRequestReturn<HttpResponseT> = [
   HttpRequestState<HttpResponseT>,
-  () => UseHttpAbortableRequestReturn<HttpResponseT>
+  () => UseHttpAbortableRequestReturn<HttpResponseT>,
+  () => Promise<HttpResponseT>
 ];

src/request/use-http-request.ts

@@ -12,7 +12,8 @@ export const useHttpRequest = <HttpResponseT, HttpRequestBodyT = unknown>(
   /**
    * Grabs the "request" function from the http client.
    */
-  const { abortableRequest: httpClientAbortableRequest } = useHttpClient();
+  const { abortableRequest: httpClientAbortableRequest, request: httpClientRequest } =
+    useHttpClient();
 
   // The state of the request.
   const [state, dispatch] = useReducer<Reducer<HttpRequestState<HttpResponseT>, HttpReqActionType>>(
@@ -89,7 +90,7 @@ export const useHttpRequest = <HttpResponseT, HttpRequestBodyT = unknown>(
   /**
    * Performs the http request allowing to abort it.
    */
-  const request = useCompareCallback(
+  const abortableRequest = useCompareCallback(
     (
       paramsOverride?: Partial<PerformHttpRequestParams<HttpRequestBodyT, HttpResponseT>>
     ): UseHttpAbortableRequestReturn<HttpResponseT> => {
@@ -119,6 +120,36 @@ export const useHttpRequest = <HttpResponseT, HttpRequestBodyT = unknown>(
     fastCompare
   );
 
+  /**
+   * Performs the http request.
+   */
+  const request = useCompareCallback(
+    (
+      paramsOverride?: Partial<PerformHttpRequestParams<HttpRequestBodyT, HttpResponseT>>
+    ): Promise<HttpResponseT> => {
+      safelyDispatch(requestInit());
+
+      const mergedParams = paramsOverride
+        ? mergeParams(performHttpRequestParams, paramsOverride)
+        : performHttpRequestParams;
+      const reqResult = httpClientRequest<HttpResponseT, HttpRequestBodyT>(mergedParams);
+
+      // Listen request to be successfully resolved or reject and
+      // update the state accordingly.
+      reqResult
+        .then((response) => {
+          safelyDispatch(requestSuccess(response));
+        })
+        .catch((error) => {
+          safelyDispatch(requestError(error));
+        });
+
+      return reqResult;
+    },
+    [httpClientRequest, performHttpRequestParams, safelyDispatch],
+    fastCompare
+  );
+
   /**
    * Keeps track of the mounting state of the component.
    */
@@ -139,5 +170,5 @@ export const useHttpRequest = <HttpResponseT, HttpRequestBodyT = unknown>(
     fastCompare
   );
 
-  return [state, request];
+  return [state, abortableRequest, request];
 };