main 🧊 add use device orientation test, fix use async

Author: babin

src/hooks/useAsync/useAsync.ts

@@ -45,7 +45,6 @@ export const useAsync = <Data>(
       .catch((error: Error) => {
         setError(error);
         setIsError(true);
-        setIsLoading(false);
       })
       .finally(() => {
         setIsLoading(false);

src/hooks/useDeviceOrientation/useDeviceOrientation.test.ts

@@ -2,40 +2,41 @@ import { act, renderHook } from '@testing-library/react';
 
 import { useDeviceOrientation } from './useDeviceOrientation';
 
-it('Should use onDeviceOrientation', () => {
-  const { result } = renderHook(useDeviceOrientation);
-
-  expect(result.current.supported).toBe(true);
-  expect(typeof result.current.onDeviceOrientation).toBe('function');
+beforeAll(() => {
+  Object.defineProperty(window, 'DeviceOrientationEvent', {
+    configurable: true,
+    value: {},
+    writable: true
+  });
 });
 
-it('Should use initial values', () => {
+it('Should use on device orientation', () => {
   const { result } = renderHook(useDeviceOrientation);
 
+  expect(result.current.supported).toBe(true);
   expect(result.current.value.alpha).toBeNull();
   expect(result.current.value.beta).toBeNull();
   expect(result.current.value.gamma).toBeNull();
   expect(result.current.value.absolute).toBeFalsy();
 });
 
-it('Should set new values', () => {
+it('Should set new values when device orientation change', () => {
   const { result } = renderHook(useDeviceOrientation);
 
-  const mockEvent = {
-    alpha: 30,
-    beta: 60,
-    gamma: 90,
-    absolute: true
-  };
-
   act(() => {
     const event = new Event('deviceorientation');
-    Object.assign(event, mockEvent);
+    Object.assign(event, {
+      alpha: 30,
+      beta: 60,
+      gamma: 90,
+      absolute: true
+    });
+
     window.dispatchEvent(event);
   });
 
-  expect(result.current.value.alpha).toContain(mockEvent.alpha);
-  expect(result.current.value.beta).toContain(mockEvent.beta);
-  expect(result.current.value.gamma).toContain(mockEvent.gamma);
-  expect(result.current.value.absolute).toContain(mockEvent.absolute);
+  expect(result.current.value.alpha).toBe(30);
+  expect(result.current.value.beta).toBe(60);
+  expect(result.current.value.gamma).toBe(90);
+  expect(result.current.value.absolute).toBe(true);
 });

src/hooks/useDeviceOrientation/useDeviceOrientation.ts

@@ -41,6 +41,7 @@ export const useDeviceOrientation = (): UseDeviceOrientationReturn => {
   });
 
   useEffect(() => {
+    console.log('@@@@@@', window, 'DeviceOrientationEvent' in window);
     if (!supported) return;
 
     const onDeviceOrientation = (event: DeviceOrientationEvent) =>

src/utils/helpers/debounce.ts

@@ -1,11 +1,11 @@
 export function debounce<Params extends unknown[]>(
   callback: (...args: Params) => void,
   delay: number
-): ((...args: Params) => void) {
+): (...args: Params) => void {
   let timer: ReturnType<typeof setTimeout>;
 
   return function (this: any, ...args: Params) {
     clearTimeout(timer);
     timer = setTimeout(() => callback.apply(this, args), delay);
   };
-};
+}