test(core): fix deprecated act() warnings + await expect().rejects (#332)

Author: dev-dino22

packages/core/src/hooks/useBooleanState/useBooleanState.spec.ts

@@ -25,7 +25,7 @@ describe('useBooleanState', () => {
     const { result } = await renderHookSSR(() => useBooleanState(false));
     const [, setTrue] = result.current;
 
-    act(() => {
+    await act(async () => {
       setTrue();
     });
 
@@ -37,7 +37,7 @@ describe('useBooleanState', () => {
     const { result } = await renderHookSSR(() => useBooleanState(true));
     const [, , setFalse] = result.current;
 
-    act(() => {
+    await act(async () => {
       setFalse();
     });
 
@@ -49,14 +49,14 @@ describe('useBooleanState', () => {
     const { result } = await renderHookSSR(() => useBooleanState(false));
     const [, , , toggle] = result.current;
 
-    act(() => {
+    await act(async () => {
       toggle();
     });
 
     let [bool] = result.current;
     expect(bool).toBe(true);
 
-    act(() => {
+    await act(async () => {
       toggle();
     });
 

packages/core/src/hooks/useControlledState/useControlledState.spec.tsx

@@ -13,12 +13,12 @@ describe('useControlledState', () => {
     expect(value).toBe('testing');
   });
 
-  it('should be uncontrolled when defaultValue is passed', () => {
+  it('should be uncontrolled when defaultValue is passed', async () => {
     const { result } = renderHookSSR(() => useControlledState({ defaultValue: 'testing' }));
     const [value] = result.current;
     expect(value).toBe('testing');
 
-    act(() => {
+    await act(async () => {
       const [, setValue] = result.current;
       setValue('naruto');
     });
@@ -27,12 +27,12 @@ describe('useControlledState', () => {
     expect(next).toBe('naruto');
   });
 
-  it('should be controlled when value is passed', () => {
+  it('should be controlled when value is passed', async () => {
     const { result } = renderHookSSR(() => useControlledState({ value: 'testing' }));
     const [value] = result.current;
     expect(value).toBe('testing');
 
-    act(() => {
+    await act(async () => {
       const [, setValue] = result.current;
       setValue('naruto');
     });
@@ -71,12 +71,12 @@ describe('useControlledState', () => {
     expect(screen.getByTestId('value')).toHaveTextContent('0');
   });
 
-  it('should not change when the value is the same', () => {
+  it('should not change when the value is the same', async () => {
     const { result } = renderHookSSR(() => useControlledState({ value: 'testing' }));
     const [value] = result.current;
     expect(value).toBe('testing');
 
-    act(() => {
+    await act(async () => {
       const [, setValue] = result.current;
       setValue('testing');
     });
@@ -85,15 +85,15 @@ describe('useControlledState', () => {
     expect(next).toBe('testing');
   });
 
-  it('should handle controlled undefined value correctly', () => {
+  it('should handle controlled undefined value correctly', async () => {
     const { result } = renderHookSSR(() =>
       useControlledState({
         value: 'test',
         onChange: () => {},
       })
     );
 
-    act(() => {
+    await act(async () => {
       const [, setValue] = result.current;
       setValue(undefined as never);
     });
@@ -102,12 +102,12 @@ describe('useControlledState', () => {
     expect(value).toBe('test');
   });
 
-  it('should handle function setState action', () => {
+  it('should handle function setState action', async () => {
     const { result } = renderHookSSR(() => useControlledState({ defaultValue: 5 }));
     const [value] = result.current;
     expect(value).toBe(5);
 
-    act(() => {
+    await act(async () => {
       const [, setValue] = result.current;
       setValue(prev => prev + 3);
     });

packages/core/src/hooks/useCounter/useCounter.spec.ts

@@ -16,51 +16,51 @@ describe('useCounter', () => {
     expect(result.current.count).toBe(10);
   });
 
-  it('should increment the counter', () => {
+  it('should increment the counter', async () => {
     const { result } = renderHook(() => useCounter(5));
 
-    act(() => {
+    await act(async () => {
       result.current.increment();
     });
 
     expect(result.current.count).toBe(6);
   });
 
-  it('should decrement the counter', () => {
+  it('should decrement the counter', async () => {
     const { result } = renderHook(() => useCounter(5));
 
-    act(() => {
+    await act(async () => {
       result.current.decrement();
     });
 
     expect(result.current.count).toBe(4);
   });
 
-  it('should reset the counter to initial value', () => {
+  it('should reset the counter to initial value', async () => {
     const { result } = renderHook(() => useCounter(5));
 
-    act(() => {
+    await act(async () => {
       result.current.increment();
       result.current.increment();
     });
 
     expect(result.current.count).toBe(7);
 
-    act(() => {
+    await act(async () => {
       result.current.reset();
     });
 
     expect(result.current.count).toBe(5);
   });
 
-  it('should not go below minimum value', () => {
+  it('should not go below minimum value', async () => {
     const { result } = renderHook(() =>
       useCounter(5, {
         min: 3,
       })
     );
 
-    act(() => {
+    await act(async () => {
       result.current.decrement();
       result.current.decrement();
       result.current.decrement();
@@ -69,14 +69,14 @@ describe('useCounter', () => {
     expect(result.current.count).toBe(3);
   });
 
-  it('should not go above maximum value', () => {
+  it('should not go above maximum value', async () => {
     const { result } = renderHook(() =>
       useCounter(5, {
         max: 7,
       })
     );
 
-    act(() => {
+    await act(async () => {
       result.current.increment();
       result.current.increment();
       result.current.increment();
@@ -85,20 +85,20 @@ describe('useCounter', () => {
     expect(result.current.count).toBe(7);
   });
 
-  it('should use the provided step value for increment and decrement', () => {
+  it('should use the provided step value for increment and decrement', async () => {
     const { result } = renderHook(() =>
       useCounter(5, {
         step: 2,
       })
     );
 
-    act(() => {
+    await act(async () => {
       result.current.increment();
     });
 
     expect(result.current.count).toBe(7);
 
-    act(() => {
+    await act(async () => {
       result.current.decrement();
     });
 
@@ -123,37 +123,37 @@ describe('useCounter', () => {
     expect(result2.current.count).toBe(8);
   });
 
-  it('should allow setting arbitrary value within constraints', () => {
+  it('should allow setting arbitrary value within constraints', async () => {
     const { result } = renderHook(() =>
       useCounter(0, {
         min: 3,
         max: 8,
       })
     );
 
-    act(() => {
+    await act(async () => {
       result.current.setCount(6);
     });
 
     expect(result.current.count).toBe(6);
 
-    act(() => {
+    await act(async () => {
       result.current.setCount(1);
     });
 
     expect(result.current.count).toBe(3);
 
-    act(() => {
+    await act(async () => {
       result.current.setCount(10);
     });
 
     expect(result.current.count).toBe(8);
   });
 
-  it('should work with updater function for setCount', () => {
+  it('should work with updater function for setCount', async () => {
     const { result } = renderHook(() => useCounter(5));
 
-    act(() => {
+    await act(async () => {
       result.current.setCount(prev => prev + 3);
     });