Incubator.Dialog - fix test after RN71 bug fix (#2575)

* Incubator.Dialog - fix test after RN71 bug fix

* Forgot this

Author: M-i-k-e-l

GestureDetectorMock.tsx renamed to jestSetup/GestureDetectorMock.tsx

@@ -97,6 +97,20 @@ jest.mock('react-native-gesture-handler',
 jest.mock('react-native', () => {
   const reactNative = jest.requireActual('react-native');
   reactNative.NativeModules.KeyboardTrackingViewTempManager = {};
+  const OriginalModal = reactNative.Modal;
+  const React = jest.requireActual('react');
+  const useDidUpdate = require('./useDidUpdate').default;
+  Object.defineProperty(reactNative, 'Modal', {
+    value: props => {
+      // eslint-disable-next-line react-hooks/rules-of-hooks
+      useDidUpdate(() => {
+        if (!props.visible) {
+          props.onDismiss?.();
+        }
+      }, [props.visible]);
+      return <OriginalModal {...props}/>;
+    }
+  });
   return reactNative;
 });
 

jest-setup.js renamed to jestSetup/jest-setup.js

@@ -0,0 +1,18 @@
+import {useEffect, useRef, DependencyList} from 'react';
+
+/**
+ * This hook avoid calling useEffect on the initial value of his dependency array
+ */
+const useDidUpdate = (callback: () => void, dep: DependencyList) => {
+  const isMounted = useRef<boolean>(false);
+
+  useEffect(() => {
+    if (isMounted.current) {
+      callback();
+    } else {
+      isMounted.current = true;
+    }
+  }, dep);
+};
+
+export default useDidUpdate;

jestSetup/useDidUpdate.ts

@@ -139,7 +139,7 @@
       "/uilib-docs/"
     ],
     "setupFiles": [
-      "./jest-setup.js"
+      "./jestSetup/jest-setup.js"
     ],
     "testMatch": [
       "**/*.spec.(js|ts|tsx)"

package.json

@@ -54,7 +54,6 @@ describe('Incubator.Dialog', () => {
     const closeButtonDriver = new ButtonDriver({component, testID: 'closeButton'});
     await closeButtonDriver.press();
     expect(await dialogDriver.exists()).toBeFalsy();
-    // TODO:
-    // expect(onDismiss).toHaveBeenCalledTimes(1);
+    expect(onDismiss).toHaveBeenCalledTimes(1);
   });
 });