fix: fix useNumber increse (#58)

Author: toshisugimo

src/array/index.test.ts

@@ -64,6 +64,21 @@ describe('useNumber array', () => {
     // then
     expect(result.current[0]).toBe(7);
   });
+  it('should return lowerLimit value if nextValue is greater than upperLimit and loop', () => {
+    // given
+    const { result } = renderHook(() =>
+      useNumber(5, {
+        upperLimit: 5,
+        lowerLimit: 1,
+        loop: true,
+      }),
+    );
+    const [, actions] = result.current;
+    // when
+    act(() => actions.increase());
+    // then
+    expect(result.current[0]).toBe(1);
+  });
   it('should decrease value with concrete value', () => {
     // given
     const { result } = renderHook(() => useNumber(5));

src/array/useNumber.ts

@@ -51,8 +51,8 @@ export function useNumber(
 
         if (upperLimit !== undefined) {
           if (nextValue > upperLimit) {
-            if (loop) {
-              return initial;
+            if (loop && lowerLimit) {
+              return lowerLimit;
             }
             return upperLimit;
           }
@@ -61,7 +61,7 @@ export function useNumber(
         return nextValue;
       });
     },
-    [initial, loop, step, upperLimit],
+    [loop, step, upperLimit, lowerLimit],
   );
   const actions = useMemo(
     () => ({