@eohjsc/react-native-smart-city 0.7.26 → 0.7.30

package/src/screens/Automate/AddNewAction/SetupScriptReceiverSms.js

@@ -1,41 +1,20 @@
-import React, { useCallback, useMemo, useState, useEffect, memo } from 'react';
-import { FlatList, View } from 'react-native';
+import React, { useCallback, useState } from 'react';
 import { useNavigation } from '@react-navigation/native';
-import styles from './Styles/SetupScriptReceiverEmailStyles';
-import { CircleView, HeaderCustom, Text } from '../../../commons';
 import { useTranslations } from '../../../hooks/Common/useTranslations';
 
-import BottomButtonView from '../../../commons/BottomButtonView';
-import { axiosPost, axiosGet } from '../../../utils/Apis/axios';
-import { API, Colors } from '../../../configs';
+import { axiosPost } from '../../../utils/Apis/axios';
+import { API } from '../../../configs';
 import { ToastBottomHelper } from '../../../utils/Utils';
 import Routes from '../../../utils/Route';
+import ReceiverSelect from './ReceiverSelect';
 import moment from 'moment';
-import CheckBox from '@react-native-community/checkbox';
-import { useSCContextSelector } from '../../../context';
-import { Image } from 'react-native';
 
 const SetupScriptReceiverSms = ({ route }) => {
   const t = useTranslations();
-  const { goBack, navigate } = useNavigation();
-  const { automate = {}, unitId, formData } = route.params || {};
+  const { navigate } = useNavigation();
+  const { automate = {}, unitId, formData } = route.params;
   const { id: automateId } = automate;
-  const [members, setMembers] = useState([]);
   const [listUser, setListUser] = useState([]);
-  const currentUserId = useSCContextSelector(
-    (state) => state.auth.account.user.id
-  );
-
-  const loadMembers = useCallback(async () => {
-    const { success, data } = await axiosGet(API.SHARE.UNITS_MEMBERS(unitId));
-    if (success) {
-      setMembers(data);
-    }
-  }, [unitId]);
-
-  useEffect(() => {
-    loadMembers();
-  }, [loadMembers]);
 
   const onNext = useCallback(async () => {
     formData.receiver = listUser;
@@ -55,112 +34,26 @@ const SetupScriptReceiverSms = ({ route }) => {
     }
   }, [automateId, formData, listUser, navigate, t]);
 
-  const canSave = useMemo(() => {
-    const { message } = formData;
-    return !!message && !!listUser.length;
-  }, [formData, listUser.length]);
-
-  const arrColor = useMemo(
-    () => [
-      Colors.GeekBlue3,
-      Colors.Purple3,
-      Colors.Orange3,
-      Colors.Volcano3,
-      Colors.Blue9,
-      Colors.Green3,
-      Colors.Cyan2,
-    ],
-    []
-  );
-
-  const onChecked = useCallback(
-    (id) => (checked) => {
-      setListUser((prevListUser) =>
-        checked
-          ? [...prevListUser, id]
-          : prevListUser.filter((userId) => userId !== id)
-      );
+  const onLoadOption = useCallback(
+    (item) => {
+      return {
+        label: item.phone_number,
+        invalidLabel: t('no_phone_number'),
+        disabled: !item.phone_number,
+      };
     },
-    []
+    [t]
   );
 
-  const RowMember = memo(({ member, index, onValueChange }) => {
-    const { id, name, avatar, share_id, phone_number } = member;
-    const [role, roleColor] = useMemo(() => {
-      if (!share_id) {
-        return [t('owner'), Colors.Primary];
-      }
-      if (id === currentUserId) {
-        return [t('me'), Colors.Primary];
-      }
-      return [t('member'), Colors.Gray6];
-    }, [share_id, id]);
-
-    const firstWordsInName = useMemo(() => {
-      return name.charAt();
-    }, [name]);
-
-    const circleColor = arrColor[index % arrColor.length];
-
-    return (
-      <View style={styles.rowContainer}>
-        <View style={styles.border}>
-          <CheckBox
-            disabled={!phone_number}
-            lineWidth={4}
-            value={listUser.includes(id)}
-            onValueChange={onValueChange(id)}
-            style={styles.checkbox}
-          />
-          <View style={styles.paddingLeft16}>
-            {avatar ? (
-              <Image source={{ uri: avatar }} style={styles.avatar} />
-            ) : (
-              <CircleView size={40} backgroundColor={circleColor} center>
-                <Text color={Colors.White}>{firstWordsInName}</Text>
-              </CircleView>
-            )}
-          </View>
-          <View style={styles.paddingLeft16}>
-            <Text style={styles.titleName}>{name}</Text>
-            {phone_number ? (
-              <Text style={styles.status}>{phone_number}</Text>
-            ) : (
-              <Text style={styles.invalid}>{t('no_phone_number')}</Text>
-            )}
-          </View>
-          <View style={styles.endFlex}>
-            <Text style={[styles.textRole, { color: roleColor }]}>{role}</Text>
-          </View>
-        </View>
-      </View>
-    );
-  });
-
   return (
-    <View style={styles.wrap}>
-      <HeaderCustom isShowClose onClose={goBack} title={t('sms_to')} />
-      <FlatList
-        data={members}
-        renderItem={({ item, index }) => (
-          <RowMember member={item} index={index} onValueChange={onChecked} />
-        )}
-        keyExtractor={(item) => item.id.toString()}
-        ListEmptyComponent={
-          <View style={styles.viewEmpty}>
-            <Text style={styles.textCenter}>{t('no_member')}</Text>
-          </View>
-        }
-      />
-      <View style={styles.container}>
-        <BottomButtonView
-          style={styles.bottomButtonView}
-          mainTitle={t('done')}
-          onPressMain={onNext}
-          typeMain={canSave ? 'primary' : 'disabled'}
-        />
-      </View>
-    </View>
+    <ReceiverSelect
+      listUser={listUser}
+      unitId={unitId}
+      title={t('notify_to')}
+      setListUser={setListUser}
+      onLoadOption={onLoadOption}
+      onNext={onNext}
+    />
   );
 };
 

package/src/screens/Automate/AddNewAction/SetupScriptSms.js

@@ -37,7 +37,7 @@ const SetupScriptSms = ({ route }) => {
   }, [navigate, automate, initialUnitId, formData]);
 
   const canSave = useMemo(() => {
-    const { message } = formData || {};
+    const { message } = formData;
     return !!message;
   }, [formData]);
 

package/src/screens/Automate/AddNewAction/Styles/{SetupScriptReceiverEmailStyles.js → ReceiverSelectStyles.js}

@@ -1,6 +1,6 @@
 import { StyleSheet } from 'react-native';
-import { Colors, Constants } from '../../../../configs';
 import { getBottomSpace } from 'react-native-iphone-x-helper';
+import { Colors, Constants } from '../../../../configs';
 
 export default StyleSheet.create({
   wrap: {
@@ -82,4 +82,21 @@ export default StyleSheet.create({
     width: 24,
     height: 24,
   },
+
+  wrapSelectAll: {
+    paddingHorizontal: 16,
+    paddingVertical: 8,
+  },
+  wrapCheckboxSelectAll: {
+    flexDirection: 'row',
+    alignItems: 'center',
+  },
+  checkboxSelectAll: {
+    marginRight: 8,
+  },
+  borderSelectAll: {
+    height: 1,
+    backgroundColor: Colors.Gray4,
+    marginHorizontal: 16,
+  },
 });

package/src/screens/Automate/AddNewAction/__test__/SetupScriptNotify.test.js

@@ -1,20 +1,14 @@
 import { useNavigation, useRoute } from '@react-navigation/native';
-import MockAdapter from 'axios-mock-adapter';
 import React from 'react';
 import { TouchableWithoutFeedback } from 'react-native';
 import renderer, { act } from 'react-test-renderer';
 import BottomButtonView from '../../../../commons/BottomButtonView';
 import _TextInput from '../../../../commons/Form/TextInput';
-import API from '../../../../configs/API';
 import { SCProvider } from '../../../../context';
 import { mockSCStore } from '../../../../context/mockStore';
-import api from '../../../../utils/Apis/axios';
 import Routes from '../../../../utils/Route';
-import { ToastBottomHelper } from '../../../../utils/Utils';
 import SetupScriptNotify from '../SetupScriptNotify';
 
-const mock = new MockAdapter(api.axiosInstance);
-
 const wrapComponent = (route) => (
   <SCProvider initState={mockSCStore({})}>
     <SetupScriptNotify route={route} />
@@ -43,8 +37,6 @@ describe('Test SetupScriptNotify', () => {
   });
 
   it('SetupScriptNotify onPress create script notify success', async () => {
-    const spyToast = jest.spyOn(ToastBottomHelper, 'success');
-    mock.onPost(API.AUTOMATE.ADD_SCRIPT_NOTIFY(1)).reply(200);
     await act(async () => {
       tree = await renderer.create(wrapComponent(route));
     });
@@ -66,14 +58,19 @@ describe('Test SetupScriptNotify', () => {
     await act(async () => {
       button.props.onPressMain();
     });
-    expect(spyToast).toHaveBeenCalled();
+    expect(mockedNavigate).toHaveBeenCalledWith(
+      Routes.SetupScriptReceiverNotify,
+      {
+        automate: route.params.automate,
+        unitId: 1,
+        formData: { message: 'Message', title: 'Title', unit: 1 },
+      }
+    );
   });
 
   it('SetupScriptNotify onPress create script notify success with multi unit', async () => {
     route.params.unitId = null;
     route.params.multiUnit = { id: 1 };
-    const spyToast = jest.spyOn(ToastBottomHelper, 'success');
-    mock.onPost(API.AUTOMATE.ADD_SCRIPT_NOTIFY(1)).reply(200);
     await act(async () => {
       tree = await renderer.create(wrapComponent(route));
     });
@@ -95,27 +92,13 @@ describe('Test SetupScriptNotify', () => {
     await act(async () => {
       button.props.onPressMain();
     });
-    expect(spyToast).toHaveBeenCalled();
-  });
-
-  it('SetupScriptNotify onPress create script notify error', async () => {
-    const spyToast = jest.spyOn(ToastBottomHelper, 'error');
-    mock.onPost(API.AUTOMATE.ADD_SCRIPT_NOTIFY(1)).reply(400);
-    await act(async () => {
-      tree = await renderer.create(wrapComponent(route));
-    });
-    const instance = tree.root;
-    const inputs = instance.findAllByType(_TextInput);
-    expect(inputs).toHaveLength(2);
-
-    await act(async () => {
-      inputs[0].props.onChange('Title');
-      inputs[1].props.onChange('Message');
-    });
-    const button = instance.findByType(BottomButtonView);
-    await act(async () => {
-      button.props.onPressMain();
-    });
-    expect(spyToast).toHaveBeenCalled();
+    expect(mockedNavigate).toHaveBeenCalledWith(
+      Routes.SetupScriptReceiverNotify,
+      {
+        automate: route.params.automate,
+        unitId: 1,
+        formData: { message: 'Message', title: 'Title', unit: 1 },
+      }
+    );
   });
 });

package/src/screens/Automate/AddNewAction/__test__/SetupScriptReceiverEmail.test.js

@@ -66,11 +66,12 @@ describe('Test SetupScriptReceiverEmail', () => {
     });
     const instance = tree.root;
     const checkboxs = instance.findAllByType(CheckBox);
-    expect(checkboxs).toHaveLength(2);
-    expect(checkboxs[0].props.disabled).toBeFalsy();
-    expect(checkboxs[1].props.disabled).toBeTruthy();
+    expect(checkboxs).toHaveLength(3);
+    expect(checkboxs[0].props.value).toBeFalsy(); // Select All
+    expect(checkboxs[1].props.disabled).toBeFalsy();
+    expect(checkboxs[2].props.disabled).toBeTruthy();
     await act(async () => {
-      checkboxs[0].props.onValueChange(true);
+      checkboxs[1].props.onValueChange(true);
     });
 
     const button = instance.findByType(BottomButtonView);
@@ -89,11 +90,12 @@ describe('Test SetupScriptReceiverEmail', () => {
     });
     const instance = tree.root;
     const checkboxs = instance.findAllByType(CheckBox);
-    expect(checkboxs).toHaveLength(2);
-    expect(checkboxs[0].props.disabled).toBeFalsy();
-    expect(checkboxs[1].props.disabled).toBeTruthy();
+    expect(checkboxs).toHaveLength(3);
+    expect(checkboxs[0].props.value).toBeFalsy(); // Select All
+    expect(checkboxs[1].props.disabled).toBeFalsy();
+    expect(checkboxs[2].props.disabled).toBeTruthy();
     await act(async () => {
-      checkboxs[0].props.onValueChange(true);
+      checkboxs[1].props.onValueChange(true);
     });
 
     const button = instance.findByType(BottomButtonView);

package/src/screens/Automate/AddNewAction/__test__/SetupScriptReceiverNotify.test.js

@@ -0,0 +1,217 @@
+import { useNavigation, useRoute } from '@react-navigation/native';
+import { FlatList } from 'react-native';
+import MockAdapter from 'axios-mock-adapter';
+import React from 'react';
+import renderer, { act } from 'react-test-renderer';
+import BottomButtonView from '../../../../commons/BottomButtonView';
+import API from '../../../../configs/API';
+import { SCProvider } from '../../../../context';
+import { mockSCStore } from '../../../../context/mockStore';
+import { Search } from '../../../../commons/DevMode';
+import api from '../../../../utils/Apis/axios';
+import Routes from '../../../../utils/Route';
+import { getTranslate } from '../../../../utils/I18n';
+import { ToastBottomHelper } from '../../../../utils/Utils';
+import ReceiverSelect from '../ReceiverSelect';
+import SetupScriptReceiverNotify from '../SetupScriptReceiverNotify';
+import CheckBox from '@react-native-community/checkbox';
+
+const mock = new MockAdapter(api.axiosInstance);
+
+const wrapComponent = (route) => (
+  <SCProvider initState={mockSCStore({})}>
+    <SetupScriptReceiverNotify route={route} />
+  </SCProvider>
+);
+
+describe('Test SetupScriptReceiverNotify', () => {
+  const mockedNavigate = useNavigation().navigate;
+  const spyToastSuccess = jest.spyOn(ToastBottomHelper, 'success');
+  const spyToastError = jest.spyOn(ToastBottomHelper, 'error');
+
+  let tree;
+  const route = {
+    params: {
+      unitId: 1,
+      automateId: 1,
+      automate: {
+        id: 1,
+        sensor_id: 1,
+      },
+      closeScreen: Routes.ScriptDetail,
+      formData: { title: 'title', message: 'message' },
+    },
+  };
+
+  const listUser = [
+    {
+      id: 1,
+      avatar: 'https://image.com',
+      email: 'user1@eoh.io',
+      name: 'User 1',
+      phone_number: null,
+    },
+    {
+      id: 2,
+      avatar: null,
+      email: null,
+      name: 'User 2',
+      phone_number: '090xxx',
+      share_id: 3,
+    },
+  ];
+  beforeEach(() => {
+    mockedNavigate.mockClear();
+    useRoute.mockImplementation(() => route);
+    spyToastSuccess.mockClear();
+    spyToastError.mockClear();
+  });
+
+  it('test create script notify success', async () => {
+    mock.onGet(API.SHARE.UNITS_MEMBERS(1)).reply(200, listUser);
+    mock.onPost(API.AUTOMATE.ADD_SCRIPT_NOTIFY(1)).reply(200);
+    await act(async () => {
+      tree = await renderer.create(wrapComponent(route));
+    });
+
+    const instance = tree.root;
+    let checkboxs = instance.findAllByType(CheckBox);
+    expect(checkboxs).toHaveLength(3);
+    expect(checkboxs[0].props.value).toBeFalsy(); // Select All
+    expect(checkboxs[1].props.disabled).toBeFalsy();
+    expect(checkboxs[2].props.disabled).toBeFalsy();
+
+    const receiverSelect = instance.findByType(ReceiverSelect);
+    expect(receiverSelect.props.listUser).toEqual([]);
+
+    await act(async () => {
+      checkboxs[1].props.onValueChange(true);
+    });
+    expect(checkboxs[0].props.value).toBeFalsy();
+    expect(receiverSelect.props.listUser).toEqual([1]);
+
+    checkboxs = instance.findAllByType(CheckBox);
+    await act(async () => {
+      checkboxs[2].props.onValueChange(true);
+    });
+    expect(checkboxs[0].props.value).toBeTruthy();
+
+    checkboxs = instance.findAllByType(CheckBox);
+    await act(async () => {
+      checkboxs[2].props.onValueChange(false);
+    });
+    expect(checkboxs[0].props.value).toBeFalsy();
+
+    const button = instance.findByType(BottomButtonView);
+    await act(async () => {
+      button.props.onPressMain();
+    });
+    expect(mock.history.post).toHaveLength(1);
+    expect(mock.history.post[0].url).toEqual(API.AUTOMATE.ADD_SCRIPT_NOTIFY(1));
+    expect(mock.history.post[0].data).toEqual(
+      JSON.stringify({
+        title: 'title',
+        message: 'message',
+        receiver: [1],
+      })
+    );
+    expect(spyToastSuccess).toHaveBeenCalledWith(
+      getTranslate('en', 'text_done')
+    );
+  });
+
+  it('test filter select all and deselect all', async () => {
+    mock.onGet(API.SHARE.UNITS_MEMBERS(1)).reply(200, listUser);
+    await act(async () => {
+      tree = await renderer.create(wrapComponent(route));
+    });
+
+    const instance = tree.root;
+    let checkboxs = instance.findAllByType(CheckBox);
+    expect(checkboxs).toHaveLength(3);
+    expect(checkboxs[0].props.value).toBeFalsy(); // Select All
+    expect(checkboxs[1].props.disabled).toBeFalsy();
+    expect(checkboxs[2].props.disabled).toBeFalsy();
+
+    const flatList = instance.findByType(FlatList);
+    expect(flatList.props.data).toEqual([
+      {
+        ...listUser[0],
+        label: null,
+        invalidLabel: getTranslate('en', 'no_phone_number'),
+      },
+      {
+        ...listUser[1],
+        label: '090xxx',
+        invalidLabel: getTranslate('en', 'no_phone_number'),
+      },
+    ]);
+
+    const search = instance.findByType(Search);
+    await act(async () => {
+      search.props.onSearch('User 1');
+    });
+    expect(flatList.props.data).toEqual([
+      {
+        ...listUser[0],
+        label: null,
+        invalidLabel: getTranslate('en', 'no_phone_number'),
+      },
+    ]);
+
+    await act(async () => {
+      search.props.onSearch('090');
+    });
+    expect(flatList.props.data).toEqual([
+      {
+        ...listUser[1],
+        label: '090xxx',
+        invalidLabel: getTranslate('en', 'no_phone_number'),
+      },
+    ]);
+
+    await act(async () => {
+      search.props.onSearch('');
+    });
+    await act(async () => {
+      checkboxs[0].props.onValueChange(true); // Select All
+    });
+    checkboxs = instance.findAllByType(CheckBox);
+    expect(checkboxs[0].props.value).toBeTruthy();
+    expect(checkboxs[1].props.value).toBeTruthy();
+    expect(checkboxs[2].props.value).toBeTruthy();
+
+    await act(async () => {
+      checkboxs[0].props.onValueChange(false); // Deselect All
+    });
+    checkboxs = instance.findAllByType(CheckBox);
+    expect(checkboxs[0].props.value).toBeFalsy();
+    expect(checkboxs[1].props.value).toBeFalsy();
+    expect(checkboxs[2].props.value).toBeFalsy();
+  });
+
+  it('test create script notify fail', async () => {
+    mock.onGet(API.SHARE.UNITS_MEMBERS(1)).reply(200, listUser);
+    mock.onPost(API.AUTOMATE.ADD_SCRIPT_NOTIFY(1)).reply(400);
+    await act(async () => {
+      tree = await renderer.create(wrapComponent(route));
+    });
+    const instance = tree.root;
+    const checkboxs = instance.findAllByType(CheckBox);
+    expect(checkboxs).toHaveLength(3);
+    expect(checkboxs[0].props.value).toBeFalsy(); // Select All
+    expect(checkboxs[1].props.disabled).toBeFalsy();
+    expect(checkboxs[2].props.disabled).toBeFalsy();
+    await act(async () => {
+      checkboxs[1].props.onValueChange(true);
+    });
+
+    const button = instance.findByType(BottomButtonView);
+    await act(async () => {
+      button.props.onPressMain();
+    });
+    expect(spyToastError).toHaveBeenCalledWith(
+      getTranslate('en', 'error_please_try_later')
+    );
+  });
+});

package/src/screens/Automate/AddNewAction/__test__/SetupScriptReceiverSms.test.js

@@ -66,11 +66,12 @@ describe('Test SetupScriptReceiverSms', () => {
     });
     const instance = tree.root;
     const checkboxs = instance.findAllByType(CheckBox);
-    expect(checkboxs).toHaveLength(2);
-    expect(checkboxs[0].props.disabled).toBeFalsy();
-    expect(checkboxs[1].props.disabled).toBeTruthy();
+    expect(checkboxs).toHaveLength(3);
+    expect(checkboxs[0].props.value).toBeFalsy(); // Select All
+    expect(checkboxs[1].props.disabled).toBeFalsy();
+    expect(checkboxs[2].props.disabled).toBeTruthy();
     await act(async () => {
-      checkboxs[0].props.onValueChange(true);
+      checkboxs[1].props.onValueChange(true);
     });
 
     const button = instance.findByType(BottomButtonView);
@@ -89,11 +90,12 @@ describe('Test SetupScriptReceiverSms', () => {
     });
     const instance = tree.root;
     const checkboxs = instance.findAllByType(CheckBox);
-    expect(checkboxs).toHaveLength(2);
-    expect(checkboxs[0].props.disabled).toBeFalsy();
-    expect(checkboxs[1].props.disabled).toBeTruthy();
+    expect(checkboxs).toHaveLength(3);
+    expect(checkboxs[0].props.value).toBeFalsy(); // Select All
+    expect(checkboxs[1].props.disabled).toBeFalsy();
+    expect(checkboxs[2].props.disabled).toBeTruthy();
     await act(async () => {
-      checkboxs[0].props.onValueChange(true);
+      checkboxs[1].props.onValueChange(true);
     });
 
     const button = instance.findByType(BottomButtonView);

package/src/screens/Automate/Components/InputName.js

@@ -6,6 +6,7 @@ import { API } from '../../../configs';
 import { AccessibilityLabel } from '../../../configs/Constants';
 import _TextInput from '../../../commons/Form/TextInput';
 import styles from './InputNameStyles';
+import useInAppReview from '../../../hooks/Review/useInAppReview';
 import { useTranslations } from '../../../hooks/Common/useTranslations';
 import { axiosPost } from '../../../utils/Apis/axios';
 import NewActionWrapper from '../../Automate/AddNewAction/NewActionWrapper';
@@ -18,6 +19,8 @@ const InputName = ({ title, placeholder }) => {
   const { navigate } = useNavigation();
   const [name, setName] = useState(automateName);
   const [processing, setProcessing] = useState(false);
+  const { allowInAppReview } = useInAppReview(false);
+
   const handleContinue = useCallback(async () => {
     if (processing) {
       /* istanbul ignore next */
@@ -37,6 +40,7 @@ const InputName = ({ title, placeholder }) => {
     );
 
     if (success) {
+      allowInAppReview();
       navigate({
         name: Routes.ScriptDetail,
         merge: true,
@@ -47,7 +51,7 @@ const InputName = ({ title, placeholder }) => {
       });
     }
     setProcessing(false);
-  }, [processing, automate, name, navigate, closeScreen]);
+  }, [processing, automate, name, navigate, closeScreen, allowInAppReview]);
 
   return (
     <NewActionWrapper

package/src/screens/Automate/OneTap/__test__/AddNewOneTap.test.js

@@ -12,6 +12,18 @@ import api from '../../../../utils/Apis/axios';
 import Routes from '../../../../utils/Route';
 import AddNewOneTap from '../index';
 
+const mockAllowInAppReview = jest.fn();
+const mockAskReview = jest.fn();
+jest.mock('../../../../hooks/Review/useInAppReview', () => {
+  return {
+    __esModule: true,
+    default: jest.fn(() => ({
+      allowInAppReview: mockAllowInAppReview,
+      askReview: mockAskReview,
+    })),
+  };
+});
+
 const wrapComponent = (route) => {
   useRoute.mockReturnValue(route);
   return (
@@ -28,6 +40,8 @@ describe('test OneTap', () => {
   const mockedNavigate = useNavigation().navigate;
   beforeEach(() => {
     mockedNavigate.mockClear();
+    mockAllowInAppReview.mockClear();
+    mockAskReview.mockClear();
   });
 
   it('create OneTap success', async () => {
@@ -91,6 +105,8 @@ describe('test OneTap', () => {
         },
       },
     });
+    expect(mockAllowInAppReview).toHaveBeenCalled();
+    expect(mockAskReview).not.toHaveBeenCalled();
   });
 
   it('create OneTap fail', async () => {
@@ -116,6 +132,8 @@ describe('test OneTap', () => {
       await item[0].props.onPress();
     });
     expect(global.mockedNavigate).not.toHaveBeenCalled();
+    expect(mockAllowInAppReview).not.toHaveBeenCalled();
+    expect(mockAskReview).not.toHaveBeenCalled();
   });
 
   it('test onClose have automateId', async () => {