@eohjsc/react-native-smart-city 0.7.14 → 0.7.16

package/src/screens/Automate/ScriptDetail/Components/AddActionScript.js

@@ -1,6 +1,6 @@
 import React, { memo, useMemo } from 'react';
 
-import { useNavigation, useRoute } from '@react-navigation/native';
+import { useRoute } from '@react-navigation/native';
 
 import { ModalCustom } from '../../../../commons/Modal';
 import { StyleSheet, View } from 'react-native';
@@ -11,17 +11,25 @@ import Event from '../../../../../assets/images/Event.svg';
 import Notify from '../../../../../assets/images/Notify.svg';
 import Delay from '../../../../../assets/images/Delay.svg';
 import Email from '../../../../../assets/images/Email.svg';
+import Sms from '../../../../../assets/images/Sms.svg';
 import { TouchableOpacity } from 'react-native';
 import { IconOutline } from '@ant-design/icons-react-native';
 import { Text } from '../../../../commons';
 import Routes from '../../../../utils/Route';
 import AccessibilityLabel from '../../../../configs/AccessibilityLabel';
+import { AUTOMATE_TYPE } from '../../../../configs/Constants';
 
 const AddActionScript = memo(
-  ({ automate, isVisible, setIsVisible, numberActionAdded }) => {
+  ({
+    automate,
+    isVisible,
+    setIsVisible,
+    numberActionAdded,
+    type,
+    navigate,
+  }) => {
     const t = useTranslations();
     const { id, unit } = automate;
-    const { navigate } = useNavigation();
 
     const { name: currentScreenName } = useRoute();
     const permissions = useBackendPermission();
@@ -99,6 +107,26 @@ const AddActionScript = memo(
             }
           },
         },
+        {
+          id: 'sms_alarm',
+          text: t('sms_alarm'),
+          image: <Sms />,
+          describe: t('only_in_local_control'),
+          onClick: () => {
+            setIsVisible(false);
+            if (unit) {
+              navigate(Routes.SetupScriptSms, {
+                automate,
+                unitId: unit,
+              });
+            } else {
+              navigate(Routes.SelectUnit, {
+                automate,
+                routeName: Routes.SetupScriptSms,
+              });
+            }
+          },
+        },
       ];
     }, [
       automate,
@@ -126,22 +154,27 @@ const AddActionScript = memo(
             <View style={styles.modalHeader}>
               <Text style={styles.modalHeaderText}>{t('add_action')}</Text>
             </View>
-            {listItem.map((item, index) => (
-              <View style={styles.rowItem} key={`rowItem-${index}`}>
-                <TouchableOpacity
-                  onPress={item.onClick}
-                  accessibilityLabel={
-                    AccessibilityLabel.AUTOMATE_LIST_SCRIPT_ACTION
-                  }
-                >
-                  <View style={styles.wapItem}>
-                    <View style={styles.imageItem}>{item.image}</View>
-                    <Text style={styles.textItem}>{item.text}</Text>
-                    <IconOutline name="right" />
-                  </View>
-                </TouchableOpacity>
-              </View>
-            ))}
+            {listItem.map((item, index) => {
+              if (item.id === 'sms_alarm' && type === AUTOMATE_TYPE.ONE_TAP) {
+                return;
+              }
+              return (
+                <View style={styles.rowItem} key={`rowItem-${index}`}>
+                  <TouchableOpacity
+                    onPress={item.onClick}
+                    accessibilityLabel={
+                      AccessibilityLabel.AUTOMATE_LIST_SCRIPT_ACTION
+                    }
+                  >
+                    <View style={styles.wapItem}>
+                      <View style={styles.imageItem}>{item.image}</View>
+                      <Text style={styles.textItem}>{item.text}</Text>
+                      <IconOutline name="right" />
+                    </View>
+                  </TouchableOpacity>
+                </View>
+              );
+            })}
           </View>
         </View>
       </ModalCustom>

package/src/screens/Automate/ScriptDetail/Styles/indexStyles.js

@@ -53,8 +53,7 @@ export default StyleSheet.create({
     flexDirection: 'row',
     alignItems: 'center',
     justifyContent: 'space-between',
-    marginTop: 16,
-    marginBottom: 16,
+    marginBottom: 7,
   },
   editButton: {
     height: 40,
@@ -184,4 +183,42 @@ export default StyleSheet.create({
     justifyContent: 'center',
     alignItems: 'center',
   },
+  width40: {
+    width: '40%',
+  },
+  popoverStyle: {
+    width: '100%',
+    backgroundColor: Colors.White,
+    borderRadius: 5,
+  },
+  localControl: {
+    flexDirection: 'row',
+    alignItems: 'center',
+    maxWidth: 190,
+    marginRight: 5,
+  },
+  card: {
+    marginRight: 0,
+    padding: 10,
+  },
+  marginLeft5: {
+    marginLeft: 5,
+  },
+  textDisable: {
+    marginLeft: 15,
+    marginTop: 15,
+    marginBottom: 15,
+    flexDirection: 'row',
+    justifyContent: 'space-between',
+  },
+  listChip: {
+    marginLeft: 15,
+    marginBottom: 15,
+    flexDirection: 'row',
+    justifyContent: 'space-between',
+  },
+  checked: {
+    marginRight: 15,
+    color: Colors.Primary,
+  },
 });

package/src/screens/Automate/ScriptDetail/__test__/index.test.js

@@ -1,4 +1,5 @@
 import React from 'react';
+import { BackHandler, Platform } from 'react-native';
 import { create, act } from 'react-test-renderer';
 import Toast from 'react-native-toast-message';
 import { useNavigation } from '@react-navigation/native';
@@ -40,10 +41,12 @@ describe('Test ScriptDetail', () => {
   const mockGoBack = useNavigation().goBack;
   const mockAddListener = useNavigation().addListener;
   const mockedNavigate = useNavigation().navigate;
+  const mockedDispatch = useNavigation().dispatch;
   beforeEach(() => {
     mockGoBack.mockClear();
     mockAddListener.mockClear();
     mockedNavigate.mockClear();
+    mockedDispatch.mockClear();
     global.mockedNavigate.mockClear();
     mock.reset();
     route = {
@@ -108,6 +111,34 @@ describe('Test ScriptDetail', () => {
     };
   });
 
+  const expectMockedDispatch = (screen, screenParams) => {
+    const mockState = {
+      routes: [{ name: 'Home' }],
+    };
+    const updatedState = mockedDispatch.mock.calls[0][0](mockState);
+    expect(updatedState).toEqual({
+      type: 'RESET',
+      payload: {
+        index: 2,
+        routes: [
+          {
+            name: 'Home',
+          },
+          {
+            name: Routes.ScriptDetail,
+            params: route.params,
+          },
+          {
+            name: screen,
+            params: screenParams,
+          },
+        ],
+      },
+    });
+    expect(mockedDispatch).toHaveBeenCalledTimes(1);
+    mockedDispatch.mockClear();
+  };
+
   it('test rename script', async () => {
     await act(async () => {
       tree = await create(wrapComponent(route));
@@ -367,56 +398,41 @@ describe('Test ScriptDetail', () => {
           AccessibilityLabel.AUTOMATE_LIST_SCRIPT_ACTION &&
         el.type === TouchableOpacity
     );
-    expect(listScriptActions).toHaveLength(4);
+    expect(listScriptActions).toHaveLength(5);
     await act(async () => {
       await listScriptActions[0].props.onPress();
     });
 
-    expect(global.mockedNavigate).toHaveBeenCalledWith(
-      Routes.SelectControlDevices,
-      {
-        unitId: route.params.preAutomate.unit,
-        automateId: route.params.preAutomate.id,
-        numberActionCanAdd: 2,
-        closeScreen: undefined,
-        routeName: null,
-      }
-    );
-    mockedNavigate.mockClear();
+    expectMockedDispatch(Routes.SelectControlDevices, {
+      unitId: route.params.preAutomate.unit,
+      automateId: route.params.preAutomate.id,
+      numberActionCanAdd: 2,
+      closeScreen: undefined,
+      routeName: null,
+    });
     await act(async () => {
       await listScriptActions[1].props.onPress();
     });
 
-    expect(global.mockedNavigate).toHaveBeenCalledWith(
-      Routes.SetupScriptNotify,
-      {
-        automate: route.params.preAutomate,
-        unitId: route.params.preAutomate.unit,
-      }
-    );
-    mockedNavigate.mockClear();
+    expectMockedDispatch(Routes.SetupScriptNotify, {
+      automate: route.params.preAutomate,
+      unitId: route.params.preAutomate.unit,
+    });
     await act(async () => {
       await listScriptActions[2].props.onPress();
     });
 
-    expect(global.mockedNavigate).toHaveBeenCalledWith(
-      Routes.SetupScriptDelay,
-      {
-        automate: route.params.preAutomate,
-      }
-    );
-    mockedNavigate.mockClear();
+    expectMockedDispatch(Routes.SetupScriptDelay, {
+      automate: route.params.preAutomate,
+    });
     await act(async () => {
       await listScriptActions[3].props.onPress();
     });
 
-    expect(global.mockedNavigate).toHaveBeenCalledWith(
-      Routes.SetupScriptEmail,
-      {
-        automate: route.params.preAutomate,
-        unitId: route.params.preAutomate.unit,
-      }
-    );
+    expectMockedDispatch(Routes.SetupScriptEmail, {
+      automate: route.params.preAutomate,
+      unitId: route.params.preAutomate.unit,
+    });
   });
 
   it('test press disable script', async () => {
@@ -596,11 +612,35 @@ describe('Test ScriptDetail', () => {
       menuActionMore.props.onItemClick({ doAction: jest.fn() });
       menuActionMore.props.listMenuItem[2].doAction();
     });
-    expect(global.mockedNavigate).toBeCalledWith(Routes.AddUnknownTypeSmart, {
-      automate,
-      closeScreen: undefined,
+
+    const mockState = {
+      routes: [{ name: 'Home' }],
+    };
+    const updatedState = mockedDispatch.mock.calls[0][0](mockState);
+    expect(updatedState).toEqual({
+      type: 'RESET',
+      payload: {
+        index: 2,
+        routes: [
+          {
+            name: 'Home',
+          },
+          {
+            name: Routes.ScriptDetail,
+            params: route.params,
+          },
+          {
+            name: Routes.AddUnknownTypeSmart,
+            params: {
+              automate: automate,
+              closeScreen: undefined,
+            },
+          },
+        ],
+      },
     });
-    global.mockedNavigate.mockClear();
+    expect(mockedDispatch).toHaveBeenCalledTimes(1);
+
     expect(menuActionMore.props.listMenuItem[0].text).toEqual('Device display');
     await act(async () => {
       menuActionMore.props.listMenuItem[0].doAction();
@@ -705,6 +745,7 @@ describe('Test ScriptDetail', () => {
   });
 
   it('test navigate to UnitDetail on event beforeRemove', async () => {
+    Platform.OS = 'ios';
     route.params.closeScreen = Routes.UnitDetail;
     route.params.preAutomate.unit = 2;
 
@@ -729,4 +770,31 @@ describe('Test ScriptDetail', () => {
       unitId: 2,
     });
   });
+
+  it('test navigate to UnitDetail on event hardwareBackPress', async () => {
+    Platform.OS = 'android';
+    jest.spyOn(BackHandler, 'addEventListener');
+    route.params.closeScreen = Routes.UnitDetail;
+    route.params.preAutomate.unit = 2;
+
+    await act(async () => {
+      await create(wrapComponent(route));
+    });
+
+    expect(BackHandler.addEventListener).toHaveBeenCalledWith(
+      'hardwareBackPress',
+      expect.any(Function)
+    );
+    const backPressHandler = BackHandler.addEventListener.mock.calls.find(
+      ([eventName]) => eventName === 'hardwareBackPress'
+    )[1];
+
+    await act(async () => {
+      backPressHandler();
+    });
+
+    expect(mockedNavigate).toHaveBeenCalledWith(Routes.UnitDetail, {
+      unitId: 2,
+    });
+  });
 });