Infra/use did update (#1119)

* created useDIdUpdate hook

* added unittests to cover useDidUpdate hook

* useDidUpdate export

* typescript

* remove unused

* remove unused

* naming

* Avoid re run isEqual again by using useMemo + match dependencies type

* PR Comment - logic is based on another useEffect check

* Fix dependencies array to be flat

* Add supported test to validate useDidUpdate not getting called when same vaule passed

* naming

Author: mendyEdri

src/hooks/useDidUpdate/__tests__/useDidUpdate.spec.js

@@ -66,4 +66,15 @@ describe('useDidUpdate hook tests', () => {
     rerender();
     expect(action.mock.calls.length).toBe(1);
   });
+
+  it('Expect action not to be called when value in array dependencies array has not change value just reference', () => {
+    const action = jest.fn();
+    let dependencies = [false];
+    const {rerender} = renderHook(() => useDidUpdate(action, dependencies));
+
+    dependencies = [false];
+    rerender();
+    expect(action.mock.calls.length).toBe(0);
+  });
+
 });

src/hooks/useDidUpdate/index.ts

@@ -7,15 +7,12 @@ const useDidUpdate = (callback: () => void, dep: [any]) => {
   const isMounted = useRef<boolean>(false);
 
   useEffect(() => {
-    if (!isMounted.current) {
-      return;
+    if (isMounted.current) {
+      callback();
+    } else {
+      isMounted.current = true;
     }
-    callback();
-  }, [dep]);
-
-  useEffect(() => {
-    isMounted.current = true;
-  }, []);
+  }, dep);
 };
 
 export default useDidUpdate;