@eohjsc/react-native-smart-city 0.3.90 → 0.3.92

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

@@ -0,0 +1,58 @@
+import React, { useCallback, useState } from 'react';
+import { API, Colors } from '../../../../configs';
+import _TextInput from '../../../../commons/Form/TextInput';
+import styles from '../Styles/indexStyles';
+import AlertAction from '../../../../commons/AlertAction';
+import { axiosPatch } from '../../../../utils/Apis/axios';
+import { ToastBottomHelper } from '../../../../utils/Utils';
+import { useTranslations } from '../../../../hooks/Common/useTranslations';
+import useKeyboardAnimated from '../../../../hooks/Explore/useKeyboardAnimated';
+
+const RenameScript = ({ automate, setAutomate, isVisible, setIsVisible }) => {
+  const t = useTranslations();
+  const [inputName, setInputName] = useState(automate.name);
+  const transY = useKeyboardAnimated();
+
+  const renameScript = useCallback(async () => {
+    const { success, data: script } = await axiosPatch(
+      API.AUTOMATE.SCRIPT(automate.id),
+      {
+        name: inputName,
+      }
+    );
+    if (success) {
+      setAutomate((prev) => ({ ...prev, name: script.name }));
+      ToastBottomHelper.success(t('rename_successfully'));
+    } else {
+      ToastBottomHelper.error(t('rename_failed'));
+    }
+    setIsVisible(false);
+  }, [automate.id, inputName, setAutomate, setIsVisible, t]);
+
+  const hidePopup = () => setIsVisible(false);
+
+  return (
+    <AlertAction
+      visible={isVisible}
+      hideModal={hidePopup}
+      title={t('rename_automate')}
+      message=""
+      leftButtonTitle={t('cancel')}
+      leftButtonClick={hidePopup}
+      rightButtonTitle={t('rename')}
+      rightButtonClick={renameScript}
+      rightButtonStyle={{ color: Colors.Primary }}
+      transY={transY}
+    >
+      <_TextInput
+        onChange={setInputName}
+        value={inputName}
+        defaultValue={automate?.name}
+        textInputStyle={styles.textInput}
+        maxLength={64}
+      />
+    </AlertAction>
+  );
+};
+
+export default RenameScript;

package/src/screens/{ScriptDetail → Automate/ScriptDetail}/Styles/indexStyles.js

@@ -1,4 +1,4 @@
-import { Colors } from '../../../configs';
+import { Colors } from '../../../../configs';
 import { Platform, StyleSheet } from 'react-native';
 import { getBottomSpace } from 'react-native-iphone-x-helper';
 

package/src/screens/{ScriptDetail → Automate/ScriptDetail}/__test__/index.test.js

@@ -3,23 +3,24 @@ import { create, act } from 'react-test-renderer';
 import Toast from 'react-native-toast-message';
 
 import MockAdapter from 'axios-mock-adapter';
-import { SCProvider } from '../../../context';
-import { mockSCStore } from '../../../context/mockStore';
-import ScriptDetail from '..';
-import MenuActionMore from '../../../commons/MenuActionMore';
-import AlertAction from '../../../commons/AlertAction';
-import _TextInput from '../../../commons/Form/TextInput';
+import { SCProvider } from '../../../../context';
+import { mockSCStore } from '../../../../context/mockStore';
+import ScriptDetail from '../index';
+import MenuActionMore from '../../../../commons/MenuActionMore';
+import AlertAction from '../../../../commons/AlertAction';
+import _TextInput from '../../../../commons/Form/TextInput';
 import {
-  AUTOMATE_SELECT,
   AUTOMATE_TYPE,
   AccessibilityLabel,
-} from '../../../configs/Constants';
-import { API } from '../../../configs';
+} from '../../../../configs/Constants';
+import { API } from '../../../../configs';
 import { TouchableOpacity } from 'react-native';
-import Routes from '../../../utils/Route';
-import WrapHeaderScrollable from '../../../commons/Sharing/WrapHeaderScrollable';
-import ItemAutomate from '../../../commons/Automate/ItemAutomate';
-import api from '../../../utils/Apis/axios';
+import Routes from '../../../../utils/Route';
+import WrapHeaderScrollable from '../../../../commons/Sharing/WrapHeaderScrollable';
+import ItemAutomate from '../../../../commons/Automate/ItemAutomate';
+import api from '../../../../utils/Apis/axios';
+import Text from '../../../../commons/Text';
+import { useNavigation } from '@react-navigation/native';
 
 const wrapComponent = (route) => (
   <SCProvider initState={mockSCStore({})}>
@@ -27,18 +28,6 @@ const wrapComponent = (route) => (
   </SCProvider>
 );
 
-const mockGoBack = jest.fn();
-const mockNavigate = jest.fn();
-jest.mock('@react-navigation/native', () => {
-  return {
-    ...jest.requireActual('@react-navigation/native'),
-    useNavigation: () => ({
-      goBack: mockGoBack,
-      navigate: mockNavigate,
-    }),
-  };
-});
-
 const mock = new MockAdapter(api.axiosInstance);
 
 describe('Test ScriptDetail', () => {
@@ -46,18 +35,19 @@ describe('Test ScriptDetail', () => {
   let data;
   let tree;
 
+  const mockGoBack = useNavigation().goBack;
+  const mockNavigate = useNavigation().navigate;
   beforeEach(() => {
     mockGoBack.mockClear();
+    mockNavigate.mockClear();
+    mock.reset();
     route = {
       params: {
         id: 1,
-        name: 'script',
-        unit: { id: 2 },
-        type: AUTOMATE_TYPE.ONE_TAP,
-        havePermission: true,
-        automate: {
+        preAutomate: {
+          unit: 2,
           author: 'Le Minh Tam',
-          id: 1009,
+          id: 1,
           type: 'value_change',
           value_change: {
             condition: '<',
@@ -67,9 +57,6 @@ describe('Test ScriptDetail', () => {
             sensor_id: 73,
             value: 35,
           },
-          script: {
-            id: 1,
-          },
         },
       },
     };
@@ -93,7 +80,7 @@ describe('Test ScriptDetail', () => {
     });
     const instance = tree.root;
     const menu = instance.findByType(MenuActionMore);
-    const alertAction = instance.findByType(AlertAction);
+    const alertAction = instance.findAllByType(AlertAction)[0];
     const rename = menu.props.listMenuItem[0];
 
     await act(async () => {
@@ -119,7 +106,7 @@ describe('Test ScriptDetail', () => {
     });
     const instance = tree.root;
     const menu = instance.findByType(MenuActionMore);
-    const alertAction = instance.findByType(AlertAction);
+    const alertAction = instance.findAllByType(AlertAction)[0];
     const rename = menu.props.listMenuItem[0];
 
     await act(async () => {
@@ -128,7 +115,7 @@ describe('Test ScriptDetail', () => {
     expect(menu.props.isVisible).toBeFalsy();
     expect(alertAction.props.visible).toBeTruthy();
 
-    mock.onPatch(API.AUTOMATE.SCRIPT(1)).reply(500, { name: 'new_name' });
+    mock.onPatch(API.AUTOMATE.SCRIPT(1)).reply(500);
     await act(async () => {
       await alertAction.props.rightButtonClick();
     });
@@ -142,8 +129,8 @@ describe('Test ScriptDetail', () => {
     });
     const instance = tree.root;
     const menu = instance.findByType(MenuActionMore);
-    const alertAction = instance.findByType(AlertAction);
-    const deleteItem = menu.props.listMenuItem[3];
+    const alertAction = instance.findAllByType(AlertAction)[1];
+    const deleteItem = menu.props.listMenuItem[2];
 
     await act(async () => {
       await menu.props.onItemClick(deleteItem);
@@ -191,6 +178,8 @@ describe('Test ScriptDetail', () => {
 
   it('test activate one tap', async () => {
     mock.onGet(API.AUTOMATE.SCRIPT(1)).reply(200, data);
+    route.params.preAutomate.type = AUTOMATE_TYPE.ONE_TAP;
+
     await act(async () => {
       tree = await create(wrapComponent(route));
     });
@@ -223,15 +212,9 @@ describe('Test ScriptDetail', () => {
     await act(async () => {
       await button.props.onPress();
     });
-    expect(mockNavigate).toHaveBeenCalledWith(Routes.SelectSensorDevices, {
-      unit: route.params.unit,
-      automateId: route.params.id,
-      isCreateNewAction: true,
-      scriptName: route.params.name,
-      title: AUTOMATE_SELECT.SELECT_DEVICE,
-      type: AUTOMATE_TYPE.ONE_TAP,
-      oldType: AUTOMATE_TYPE.ONE_TAP,
-      automate: route.params.automate,
+    expect(mockNavigate).toHaveBeenCalledWith(Routes.SelectControlDevices, {
+      unitId: route.params.preAutomate.unit,
+      automateId: route.params.preAutomate.id,
     });
   });
 
@@ -246,21 +229,24 @@ describe('Test ScriptDetail', () => {
     await act(async () => {
       await WrapHeaderScrollables[0].props.onGoBack();
     });
-    expect(mockNavigate).toBeCalled();
+    expect(mockGoBack).toBeCalled();
   });
 
-  const _testGoToActivityLog = (automateType, activityLogType, isMultiUnit) => {
-    it('test go to activity log', async () => {
-      route.params.type = automateType;
-      if (isMultiUnit) {
-        route.params.unit.id = undefined;
-      }
+  const _testGoToActivityLog = (
+    count,
+    automateType,
+    activityLogType,
+    unitId
+  ) => {
+    it(`test go to activity log ${count}`, async () => {
+      route.params.preAutomate.type = automateType;
+      route.params.preAutomate.unit = unitId;
       await act(async () => {
         tree = await create(wrapComponent(route));
       });
       const instance = tree.root;
       const menu = instance.findByType(MenuActionMore);
-      const gotoActivityLog = menu.props.listMenuItem[2];
+      const gotoActivityLog = menu.props.listMenuItem[1];
 
       await act(async () => {
         await menu.props.onItemClick(gotoActivityLog);
@@ -268,9 +254,9 @@ describe('Test ScriptDetail', () => {
       expect(mockNavigate).toHaveBeenCalledWith(Routes.ActivityLog, {
         id: route.params.id,
         type: activityLogType,
-        share: route.params.unit,
+        share: unitId,
         filterEnabled: {
-          user: !isMultiUnit,
+          user: !!unitId,
           date: true,
         },
       });
@@ -278,18 +264,19 @@ describe('Test ScriptDetail', () => {
   };
 
   _testGoToActivityLog(
+    1,
     AUTOMATE_TYPE.ONE_TAP,
     `automate.${AUTOMATE_TYPE.ONE_TAP}`,
-    false
+    2
   );
-  _testGoToActivityLog(AUTOMATE_TYPE.VALUE_CHANGE, 'automate', false);
-  _testGoToActivityLog(AUTOMATE_TYPE.VALUE_CHANGE, 'automate', true);
+  _testGoToActivityLog(2, AUTOMATE_TYPE.VALUE_CHANGE, 'automate', 2);
+  _testGoToActivityLog(3, AUTOMATE_TYPE.VALUE_CHANGE, 'automate', undefined);
 
   it('Test render textCondition value change >', async () => {
     route.params = {
       ...route.params,
-      type: AUTOMATE_TYPE.VALUE_CHANGE,
-      automate: {
+      preAutomate: {
+        type: AUTOMATE_TYPE.VALUE_CHANGE,
         value_change: {
           config_name: 'Light Value',
           value: 3,
@@ -302,7 +289,7 @@ describe('Test ScriptDetail', () => {
     });
     const instance = tree.root;
     const itemAutomate = instance.findByType(ItemAutomate);
-    expect(itemAutomate.props.textCondition).toEqual(
+    expect(itemAutomate.findAllByType(Text)[1].props.children).toEqual(
       'Light Value higher than 3'
     );
 
@@ -315,8 +302,8 @@ describe('Test ScriptDetail', () => {
   it('Test render textCondition value change =', async () => {
     route.params = {
       ...route.params,
-      type: AUTOMATE_TYPE.VALUE_CHANGE,
-      automate: {
+      preAutomate: {
+        type: AUTOMATE_TYPE.VALUE_CHANGE,
         value_change: { config_name: 'Light Value', value: 3, condition: '=' },
       },
     };
@@ -325,14 +312,16 @@ describe('Test ScriptDetail', () => {
     });
     const instance = tree.root;
     const itemAutomate = instance.findByType(ItemAutomate);
-    expect(itemAutomate.props.textCondition).toEqual('Light Value equal 3');
+    expect(itemAutomate.findAllByType(Text)[1].props.children).toEqual(
+      'Light Value equal 3'
+    );
   });
 
   it('Test render textCondition value change <', async () => {
     route.params = {
       ...route.params,
-      type: AUTOMATE_TYPE.VALUE_CHANGE,
-      automate: {
+      preAutomate: {
+        type: AUTOMATE_TYPE.VALUE_CHANGE,
         value_change: {
           config_name: 'Light Value',
           value: 3,
@@ -345,7 +334,7 @@ describe('Test ScriptDetail', () => {
     });
     const instance = tree.root;
     const itemAutomate = instance.findByType(ItemAutomate);
-    expect(itemAutomate.props.textCondition).toEqual(
+    expect(itemAutomate.findAllByType(Text)[1].props.children).toEqual(
       'Light Value lower than 3'
     );
   });
@@ -353,8 +342,8 @@ describe('Test ScriptDetail', () => {
   it('Test render textCondition schedule repeat everyday', async () => {
     route.params = {
       ...route.params,
-      type: AUTOMATE_TYPE.SCHEDULE,
-      automate: {
+      preAutomate: {
+        type: AUTOMATE_TYPE.SCHEDULE,
         schedule: {
           repeat: 'every_day',
           date_repeat: '2022-01-02',
@@ -367,14 +356,16 @@ describe('Test ScriptDetail', () => {
     });
     const instance = tree.root;
     const itemAutomate = instance.findByType(ItemAutomate);
-    expect(itemAutomate.props.textCondition).toEqual('Every day at 19:00');
+    expect(itemAutomate.findAllByType(Text)[1].props.children).toEqual(
+      'Every day at 19:00'
+    );
   });
 
   it('Test render textCondition schedule repeat weekday', async () => {
     route.params = {
       ...route.params,
-      type: AUTOMATE_TYPE.SCHEDULE,
-      automate: {
+      preAutomate: {
+        type: AUTOMATE_TYPE.SCHEDULE,
         schedule: {
           repeat: 'every_week',
           date_repeat: '2022-01-02',
@@ -388,7 +379,7 @@ describe('Test ScriptDetail', () => {
     });
     const instance = tree.root;
     const itemAutomate = instance.findByType(ItemAutomate);
-    expect(itemAutomate.props.textCondition).toEqual(
+    expect(itemAutomate.findAllByType(Text)[1].props.children).toEqual(
       'Mon, Tue, Thu, Sat at 19:00'
     );
   });

package/src/screens/{ScriptDetail → Automate/ScriptDetail}/__test__/useStarredScript.test.js

@@ -1,12 +1,12 @@
 import React from 'react';
 import { act, renderHook } from '@testing-library/react-hooks';
-import { SCProvider } from '../../../context';
-import { mockSCStore } from '../../../context/mockStore';
+import { SCProvider } from '../../../../context';
+import { mockSCStore } from '../../../../context/mockStore';
 import { useStarredScript } from '../hooks/useStarredScript';
 import MockAdapter from 'axios-mock-adapter';
-import api from '../../../utils/Apis/axios';
-import { API } from '../../../configs';
-import { Action } from '../../../context/actionType';
+import api from '../../../../utils/Apis/axios';
+import { API } from '../../../../configs';
+import { Action } from '../../../../context/actionType';
 
 const mockedSetAction = jest.fn();
 const mock = new MockAdapter(api.axiosInstance);
@@ -22,8 +22,9 @@ React.useContext = mockUseContext;
 
 jest.mock('react-native-deep-linking');
 
-describe('Test useStarredScript', async () => {
+describe('Test useStarredScript', () => {
   let props;
+
   beforeEach(() => {
     mockedSetAction.mockClear();
     props = {

package/src/screens/{ScriptDetail → Automate/ScriptDetail}/hooks/useStarredScript.js

@@ -1,8 +1,8 @@
 import { useCallback, useContext } from 'react';
-import { SCContext, useSCContextSelector } from '../../../context';
-import { Action } from '../../../context/actionType';
-import { axiosPost } from '../../../utils/Apis/axios';
-import { API } from '../../../configs';
+import { SCContext, useSCContextSelector } from '../../../../context';
+import { Action } from '../../../../context/actionType';
+import { axiosPost } from '../../../../utils/Apis/axios';
+import { API } from '../../../../configs';
 
 export const useStarredScript = (automate) => {
   const { setAction } = useContext(SCContext);