main 🧊 add hook tests

Author: debabin

packages/core/src/bundle/hooks/useEventSource/useEventSource.js

@@ -14,24 +14,28 @@ import { getRetry } from '@/utils/helpers';
  * @returns {UseEventSourceReturn<Data>} The EventSource state and controls
  *
  * @example
- * const { instance, data, isConnecting, isOpen, isError, close, open } = useEventSource('url', ['message']);
+ * const { instance, data, connecting, opened, isError, close, open } = useEventSource('url', ['message']);
  */
 export const useEventSource = (url, events = [], options = {}) => {
   const [isConnecting, setIsConnecting] = useState(false);
-  const [isOpen, setIsOpen] = useState(false);
+  const [opened, setOpened] = useState(false);
   const [isError, setIsError] = useState(false);
   const retryCountRef = useRef(options?.retry ? getRetry(options.retry) : 0);
   const [error, setError] = useState(undefined);
   const [data, setData] = useState(options?.placeholderData);
   const eventSourceRef = useRef(undefined);
   const immediately = options.immediately ?? true;
+  const onEventRef = useRef((event) => setData(event.data));
   const close = () => {
     if (!eventSourceRef.current) return;
-    eventSourceRef.current.close();
-    eventSourceRef.current = undefined;
-    setIsOpen(false);
+    setOpened(false);
     setIsConnecting(false);
     setIsError(false);
+    events.forEach((eventName) =>
+      eventSourceRef.current.removeEventListener(eventName, onEventRef.current)
+    );
+    eventSourceRef.current.close();
+    eventSourceRef.current = undefined;
   };
   const open = () => {
     close();
@@ -41,13 +45,13 @@ export const useEventSource = (url, events = [], options = {}) => {
     eventSourceRef.current = eventSource;
     setIsConnecting(true);
     eventSource.onopen = () => {
-      setIsOpen(true);
+      setOpened(true);
       setIsConnecting(false);
       setError(undefined);
       options?.onOpen?.();
     };
     eventSource.onerror = (event) => {
-      setIsOpen(false);
+      setOpened(false);
       setIsConnecting(false);
       setIsError(true);
       setError(event);
@@ -62,6 +66,7 @@ export const useEventSource = (url, events = [], options = {}) => {
           setTimeout(open, retryDelay);
           return;
         }
+        return open();
       }
       retryCountRef.current = options?.retry ? getRetry(options.retry) : 0;
     };
@@ -70,11 +75,7 @@ export const useEventSource = (url, events = [], options = {}) => {
       setData(data);
       options?.onMessage?.(event);
     };
-    events.forEach((eventName) => {
-      eventSource.addEventListener(eventName, (event) => {
-        setData(event.data);
-      });
-    });
+    events.forEach((eventName) => eventSource.addEventListener(eventName, onEventRef.current));
   };
   useEffect(() => {
     if (!immediately) return;
@@ -88,7 +89,7 @@ export const useEventSource = (url, events = [], options = {}) => {
     data,
     error,
     isConnecting,
-    isOpen,
+    opened,
     isError,
     close,
     open

packages/core/src/bundle/hooks/useField/useField.js

@@ -118,6 +118,7 @@ export const useField = (params) => {
     setValue,
     reset,
     watch,
-    focus
+    focus,
+    ref: inputRef
   };
 };

packages/core/src/bundle/hooks/useLongPress/useLongPress.js

@@ -57,23 +57,19 @@ export const useLongPress = (...params) => {
         } else if (isPressedRef.current) {
           internalOptionsRef.current?.onCancel?.(event);
         }
+        isPressedRef.current = false;
+        if (timeoutIdRef.current) clearTimeout(timeoutIdRef.current);
         return false;
       });
-      isPressedRef.current = false;
-      if (timeoutIdRef.current) clearTimeout(timeoutIdRef.current);
     };
     element.addEventListener('mousedown', onStart);
-    element.addEventListener('touchstart', onStart);
-    element.addEventListener('mouseup', onCancel);
-    element.addEventListener('touchend', onCancel);
     window.addEventListener('mouseup', onCancel);
+    element.addEventListener('touchstart', onStart);
     window.addEventListener('touchend', onCancel);
     return () => {
       element.removeEventListener('mousedown', onStart);
-      element.removeEventListener('touchstart', onStart);
-      element.removeEventListener('mouseup', onCancel);
-      element.removeEventListener('touchend', onCancel);
       window.removeEventListener('mouseup', onCancel);
+      element.removeEventListener('touchstart', onStart);
       window.removeEventListener('touchend', onCancel);
       if (timeoutIdRef.current) clearTimeout(timeoutIdRef.current);
     };

packages/core/src/helpers/createContext/createContext.demo.tsx

@@ -17,7 +17,7 @@ const App = () => {
   });
 
   return (
-    <div className='rounded-lg p-4 shadow-md'>
+    <div className='rounded-lg p-4'>
       <div className='mb-6'>
         <h3 className='font-semibold'>Current Profile:</h3>
         <div>

packages/core/src/helpers/createEventEmitter/createEventEmitter.demo.tsx

@@ -35,7 +35,7 @@ const Demo = () => {
   });
 
   return (
-    <div className='rounded-lg p-4 shadow-md'>
+    <div className='rounded-lg p-4'>
       <div className='mb-6'>
         <h3 className='font-semibold'>Current Profile:</h3>
         <NameFieldInfo />

packages/core/src/helpers/createStore/createStore.demo.tsx

@@ -61,7 +61,7 @@ const Demo = () => {
   });
 
   return (
-    <div className='rounded-lg p-4 shadow-md'>
+    <div className='rounded-lg p-4'>
       <div className='mb-6'>
         <h3 className='font-semibold'>Current Profile:</h3>
         <NameFieldInfo />

packages/core/src/hooks/useBattery/useBattery.test.ts

@@ -5,13 +5,17 @@ import { createTrigger, renderHookServer } from '@/tests';
 import { useBattery } from './useBattery';
 
 const trigger = createTrigger<string, () => void>();
+const mockAddEventListener = vi.fn();
 const mockRemoveEventListener = vi.fn();
 const mockBatteryManager = {
   charging: true,
   chargingTime: 0,
   dischargingTime: 5,
   level: 1,
-  addEventListener: (type: string, callback: () => void) => trigger.add(type, callback),
+  addEventListener: (type: string, callback: () => void) => {
+    mockAddEventListener(type, callback);
+    trigger.add(type, callback);
+  },
   removeEventListener: (type: string, callback: () => void) => {
     mockRemoveEventListener(type, callback);
     if (trigger.get(type) === callback) trigger.delete(type);

packages/core/src/hooks/useBroadcastChannel/useBroadcastChannel.test.ts

@@ -18,7 +18,7 @@ class MockBroadcastChannel {
   postMessage = mockBroadcastChannelPostMessage;
   close = mockBroadcastChannelClose;
   addEventListener = (type: string, callback: (event: MessageEvent) => void) => {
-    mockAddEventListener();
+    mockAddEventListener(type, callback);
     trigger.add(type, callback);
   };
   removeEventListener = (type: string, callback: (event: MessageEvent) => void) => {