@eohjsc/react-native-smart-city 0.2.97 → 0.3.0

package/src/screens/Sharing/SelectPermission.js

@@ -1,4 +1,4 @@
-import React, { useCallback, useEffect, useMemo, useState } from 'react';
+import React, { memo, useCallback, useEffect, useMemo, useState } from 'react';
 import {
   SafeAreaView,
   View,
@@ -32,8 +32,12 @@ const SelectPermission = ({ route }) => {
   const [isTickAllDevices, setIsTickAllDevices] = useState(false);
   const [activeItemId, setActiveItemId] = useState(-1);
   const [loading, setLoading] = useState(true);
-  const [dataDeviceShared, setDataDeviceShared] = useState([]);
   const [hasDataChecked, setHasDataChecked] = useState(false);
+  const [dataDeviceShared, setDataDeviceShared] = useState([]);
+
+  const isUpdateSharedDevice = useMemo(() => {
+    return type === 'update_shared';
+  }, [type]);
 
   const onTickTitle = (idGroup, isChecked, id) => {
     if (!idGroup) {
@@ -98,6 +102,10 @@ const SelectPermission = ({ route }) => {
       sensor.actions.some((i) => !i.isChecked)
     );
 
+    stationCheck(data);
+  };
+
+  const stationCheck = (data) => {
     for (let station of data) {
       if (station.sensors.length) {
         station.isChecked = !station.sensors.some((i) => !i.isChecked);
@@ -115,51 +123,80 @@ const SelectPermission = ({ route }) => {
 
     sensor.isChecked = isChecked;
 
-    for (let station of data) {
-      if (station.sensors.length) {
-        station.isChecked = !station.sensors.some((i) => !i.isChecked);
-      }
-    }
-    setIsTickAllDevices(!dataStationTemp.some((i) => !i.isChecked));
-    dataStationTemp = data;
-    setDataStations(data);
+    stationCheck(data);
   };
 
   const autoCheckedGroup = useCallback(async () => {
-    if (type === 'share_device') {
+    if (isUpdateSharedDevice) {
       if (hasDataChecked) {
         const stationIds = object_Ids(dataDeviceShared)?.stationIds;
         const sensorIds = object_Ids(dataDeviceShared)?.sensorIds;
         const actionIds = object_Ids(dataDeviceShared)?.actionIds;
         const configIds = object_Ids(dataDeviceShared)?.configIds;
-        const data = dataStations?.map((i) => ({
-          ...i,
-          isChecked: stationIds.includes(i.id),
-        }));
-        for (let station in data) {
-          for (let item in data[station].sensors) {
-            const itemTemp = data[station].sensors[item];
-            data[station].sensors[item] = {
+        const sensor_checked = [];
+        const action_checked = [];
+        const config_checked = [];
+        const action_temp = [];
+        const config_temp = [];
+        for (let station in dataStations) {
+          for (let item in dataStations[station].sensors) {
+            const itemTemp = dataStations[station].sensors[item];
+            dataStations[station].sensors[item] = {
               ...itemTemp,
               isChecked: sensorIds.includes(itemTemp?.id),
-              actions: itemTemp.actions.map((i) => ({
-                ...i,
-                isChecked: actionIds?.includes(i.id),
-              })),
-              read_configs: itemTemp.read_configs.map((i) => ({
-                ...i,
-                isChecked: configIds?.includes(i.id),
-              })),
+              actions: itemTemp.actions.map((i) => {
+                if (actionIds?.includes(i.id)) {
+                  action_checked.push(i);
+                }
+                action_temp.push(i);
+                return {
+                  ...i,
+                  isChecked: actionIds?.includes(i.id),
+                };
+              }),
+              read_configs: itemTemp.read_configs.map((i) => {
+                if (configIds?.includes(i.id)) {
+                  config_checked.push(i);
+                }
+                config_temp.push(i);
+                return {
+                  ...i,
+                  isChecked: configIds?.includes(i.id),
+                };
+              }),
             };
+            if (dataStations[station].sensors[item]?.isChecked) {
+              sensor_checked.push(dataStations[station].sensors[item]);
+            }
           }
         }
+        const check_action_config = () => {
+          return (
+            action_checked?.length === action_temp?.length &&
+            config_checked?.length === config_temp?.length
+          );
+        };
+        const data = dataStations?.map((i) => {
+          return {
+            ...i,
+            isChecked:
+              check_action_config() &&
+              i?.sensors?.length === sensor_checked?.length &&
+              stationIds.includes(i.id),
+          };
+        });
         const sensorIdsDefault = object_Ids(data)?.sensorIds;
+        const stationIdsDefault = object_Ids(data)?.stationIds;
         setDataStations(data);
-        setIsTickAllDevices(sensorIdsDefault?.length === sensorIds?.length);
+        setIsTickAllDevices(
+          check_action_config() &&
+            sensorIdsDefault?.length === sensorIds?.length &&
+            stationIdsDefault?.length === stationIds?.length
+        );
         setHasDataChecked(false);
       }
     }
-  }, [dataDeviceShared, dataStations, hasDataChecked, type]);
+  }, [dataDeviceShared, dataStations, hasDataChecked, isUpdateSharedDevice]);
 
   const GroupSenSorItem = ({ item = {}, index }) => {
     const { name = '', sensors = [], isChecked, id = '' } = item;
@@ -174,11 +211,11 @@ const SelectPermission = ({ route }) => {
           idGroup={id}
         />
         <View style={styles.wrapSensor}>
-          {sensors.map((i, index) => (
+          {sensors?.map((i, index) => (
             <SensorItem
               item={i}
               key={i.id}
-              isRenderSeparated={index !== sensors.length - 1}
+              isRenderSeparated={index !== sensors?.length - 1}
               onTickedChild={onTickedChild}
               onTickedSensor={onTickedSensor}
               titleGroup={name}
@@ -199,6 +236,7 @@ const SelectPermission = ({ route }) => {
       for (let item of station.sensors) {
         let arrIdControlTemp = [],
           arrIdReadTemp = [];
+
         for (let i of item.actions) {
           i.isChecked && arrIdControlTemp.push(i.id);
         }
@@ -219,18 +257,18 @@ const SelectPermission = ({ route }) => {
       Alert.alert('', t('choose_at_least_one'));
       return;
     }
-    if (type === 'share_device') {
+    if (isUpdateSharedDevice) {
       const phone =
         member?.phone_number && member?.email
           ? member?.phone_number
           : member?.phone_number || '';
       const email = member?.phone_number ? '' : member?.email || '';
-
       const { success } = await axiosPost(API.SHARE.SHARE(), {
         phone,
         email,
         unit: unit?.id,
         permissions: { read_permissions, control_permissions },
+        is_remove_old_permission: true,
       });
       if (success) {
         navigation.goBack();
@@ -249,48 +287,47 @@ const SelectPermission = ({ route }) => {
 
   const renderFlatList = useMemo(() => {
     return (
-      Boolean(dataStations.length) && (
-        <FlatList
-          keyExtractor={(item) => item.id}
-          extraData={dataStations}
-          data={dataStations}
-          renderItem={renderGroupItem}
-          ListHeaderComponent={
-            Boolean(dataStations?.length) && (
-              <TitleCheckBox
-                title={t('text_all_devices')}
-                wrapStyle={styles.wrapAllDevices}
-                onPress={onTickTitle}
-                titleStyle={styles.textAllDevice}
-                isChecked={isTickAllDevices}
-              />
-            )
-          }
-        />
-      )
+      <FlatList
+        keyExtractor={(item) => item.id}
+        extraData={dataStations}
+        data={dataStations}
+        renderItem={renderGroupItem}
+        ListHeaderComponent={
+          dataStations?.length > 0 ? (
+            <TitleCheckBox
+              title={t('text_all_devices')}
+              wrapStyle={styles.wrapAllDevices}
+              onPress={onTickTitle}
+              titleStyle={styles.textAllDevice}
+              isChecked={isTickAllDevices}
+            />
+          ) : (
+            <></>
+          )
+        }
+        scrollIndicatorInsets={{ right: 1 }}
+      />
     );
+
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [dataStations, isTickAllDevices, activeItemId]);
 
-  const getShareUnitPermission = useCallback(async () => {
-    if (type === 'share_device') {
-      if (dataStations?.length > 0) {
-        const response = await axiosGet(
-          API.SHARE.UNIT_MEMBER_SHARE_DEVICE(unit?.id, member?.id)
-        );
-        if (response.success) {
-          setDataDeviceShared(response.data);
-          setHasDataChecked(true);
-        }
-      }
-      setLoading(false);
-    }
-  }, [dataStations?.length, member?.id, type, unit?.id]);
-
   useEffect(() => {
     autoCheckedGroup();
   }, [autoCheckedGroup]);
 
+  const getShareUnitPermission = useCallback(async () => {
+    if (isUpdateSharedDevice && dataStations?.length > 0) {
+      const response = await axiosGet(
+        API.SHARE.UNIT_MEMBER_SHARE_DEVICE(unit?.id, member?.id)
+      );
+      if (response?.success) {
+        setDataDeviceShared(response?.data);
+        setHasDataChecked(true);
+      }
+    }
+  }, [dataStations?.length, isUpdateSharedDevice, member?.id, unit?.id]);
+
   useEffect(() => {
     getShareUnitPermission();
   }, [getShareUnitPermission]);
@@ -309,7 +346,7 @@ const SelectPermission = ({ route }) => {
       }
       setLoading(false);
     })();
-  }, [unit]);
+  }, [dataStations?.length, isUpdateSharedDevice, member?.id, unit]);
 
   useEffect(() => {
     dataStationTemp = dataStations;
@@ -341,7 +378,7 @@ const SelectPermission = ({ route }) => {
           testIDPrefix={TESTID.PREFIX.SHARING_SELECT_PERMISSION}
           leftTitle={t('cancel')}
           onLeftClick={() => navigation.goBack()}
-          rightTitle={type === 'share_device' ? t('done') : t('next')}
+          rightTitle={isUpdateSharedDevice ? t('done') : t('next')}
           rightDisabled={false}
           onRightClick={onPressBottom}
         />
@@ -350,4 +387,4 @@ const SelectPermission = ({ route }) => {
   );
 };
 
-export default SelectPermission;
+export default memo(SelectPermission);

package/src/screens/Sharing/__test__/InfoMemberUnit.test.js

@@ -1,14 +1,18 @@
 import React from 'react';
 import { create, act } from 'react-test-renderer';
 import { TouchableOpacity } from 'react-native';
+import MockAdapter from 'axios-mock-adapter';
+
 import InfoMemberUnit from '../InfoMemberUnit';
 import { HeaderCustom } from '../../../commons/Header';
-import axios from 'axios';
 import { SCProvider } from '../../../context';
 import { mockSCStore } from '../../../context/mockStore';
 import { AlertAction } from '../../../commons';
 import API from '../../../configs/API';
 import { TESTID } from '../../../configs/Constants';
+import api from '../../../utils/Apis/axios';
+
+const mock = new MockAdapter(api.axiosInstance);
 
 jest.mock('../../../hooks/Common', () => {
   return {
@@ -24,20 +28,20 @@ jest.mock('react', () => {
   };
 });
 
+const mockGoBack = jest.fn();
+
 jest.mock('@react-navigation/native', () => {
   return {
     ...jest.requireActual('@react-navigation/native'),
     useRoute: jest.fn(),
     useNavigation: () => ({
-      goBack: jest.fn(),
+      goBack: mockGoBack,
       navigate: mockedNavigate,
     }),
     useIsFocused: () => mockUseIsFocused,
   };
 });
 
-jest.mock('axios');
-
 const wrapComponent = (route) => (
   <SCProvider initState={mockSCStore({})}>
     <InfoMemberUnit route={route} />
@@ -49,7 +53,6 @@ describe('Test InfoMemberUnit', () => {
   let route;
 
   beforeEach(() => {
-    axios.get.mockClear();
     mockedNavigate.mockClear();
     route = {
       params: {
@@ -65,23 +68,17 @@ describe('Test InfoMemberUnit', () => {
       },
     };
   });
-  it('render InfoMemberUnit', async () => {
-    mockUseIsFocused.mockImplementation(() => true);
-    const response = {
-      status: 200,
-      data: {
-        id: 1,
-        identity: 'owner',
-        phone_number: '88888',
-        name: 'user',
-        email: 'abc@gmail.com',
-      },
-    };
-    axios.get.mockImplementation(async () => {
-      return response;
+
+  it('render InfoMemberUnit without params', async () => {
+    mock.onGet(API.SHARE.UNIT_MEMBER_INFO(1, 1)).reply(200, {
+      id: 1,
+      identity: 'owner',
+      phone_number: '88888',
+      name: 'user',
+      email: 'abc@gmail.com',
     });
     await act(async () => {
-      tree = await create(wrapComponent(route));
+      tree = await create(wrapComponent({}));
     });
     const instance = tree.root;
     const header = instance.findAllByType(HeaderCustom);
@@ -92,13 +89,35 @@ describe('Test InfoMemberUnit', () => {
       alertAction[0].props.rightButtonClick();
     });
   });
+
+  it('render InfoMemberUnit', async () => {
+    mock.onGet(API.SHARE.UNIT_MEMBER_INFO(1, 1)).reply(200, {
+      id: 1,
+      identity: 'owner',
+      phone_number: '88888',
+      name: 'user',
+      email: 'abc@gmail.com',
+    });
+    await act(async () => {
+      tree = await create(wrapComponent(route));
+    });
+    const instance = tree.root;
+    const header = instance.findAllByType(HeaderCustom);
+    const alertAction = instance.findAllByType(AlertAction);
+    await act(async () => {
+      alertAction[0].props.rightButtonClick();
+    });
+    expect(header).toHaveLength(1);
+    expect(alertAction).toHaveLength(1);
+  });
+
   it('render InfoMemberUnit delete member', async () => {
-    mockUseIsFocused.mockImplementation(() => true);
-    const response = {
-      status: 200,
-    };
-    axios.delete.mockImplementation(async () => {
-      return response;
+    mock.onGet(API.SHARE.UNIT_MEMBER_INFO(1, 1)).reply(200, {
+      id: 1,
+      identity: 'owner',
+      phone_number: '88888',
+      name: 'user',
+      email: 'abc@gmail.com',
     });
     await act(async () => {
       tree = await create(wrapComponent(route));
@@ -111,11 +130,10 @@ describe('Test InfoMemberUnit', () => {
     );
     expect(header).toHaveLength(1);
     expect(buttonRemove).toHaveLength(1);
+    mock.onDelete(API.SHARE.UNITS_MEMBER_DETAIL(1)).reply(200);
     await act(async () => {
       buttonRemove[0].props.onPress();
     });
-    expect(axios.delete).toHaveBeenCalledWith(
-      API.SHARE.UNITS_MEMBER_DETAIL(1, undefined)
-    );
+    expect(mockGoBack).toBeCalled();
   });
 });

package/src/screens/Sharing/__test__/MemberList.test.js

@@ -1,26 +1,26 @@
-/* eslint-disable promise/prefer-await-to-callbacks */
 import React from 'react';
 import { create, act } from 'react-test-renderer';
-import axios from 'axios';
+
 import MemberList from '../MemberList';
 import { AlertAction } from '../../../commons';
-import API from '../../../configs/API';
-import { TESTID } from '../../../configs/Constants';
-import { TouchableOpacity } from 'react-native';
 import { SCProvider } from '../../../context';
 import { mockSCStore } from '../../../context/mockStore';
+import { TouchableOpacity } from 'react-native';
 import Routes from '../../../utils/Route';
 
+jest.mock('../../../hooks/Common', () => {
+  return {
+    useIsOwnerOfUnit: () => ({ isOwner: true }),
+  };
+});
+
 const mockedNavigate = jest.fn();
-const mockedDispatch = jest.fn();
-const mockedAddListener = jest.fn();
 const mockUseIsFocused = jest.fn();
 
 jest.mock('@react-navigation/native', () => {
   return {
     ...jest.requireActual('@react-navigation/native'),
     useNavigation: () => ({
-      addListener: mockedAddListener,
       navigate: mockedNavigate,
     }),
     useIsFocused: () => ({
@@ -29,22 +29,8 @@ jest.mock('@react-navigation/native', () => {
   };
 });
 
-jest.mock('axios');
-jest.mock('react-native-toast-message');
-
-jest.mock('react-redux', () => {
-  return {
-    ...jest.requireActual('react-redux'),
-    useDispatch: () => mockedDispatch,
-    useSelector: jest.fn(),
-    connect: () => {
-      return (component) => component;
-    },
-  };
-});
-
 const wrapComponent = (route, state) => (
-  <SCProvider initState={mockSCStore(state)}>
+  <SCProvider initState={mockSCStore({})}>
     <MemberList route={route} />
   </SCProvider>
 );
@@ -74,17 +60,6 @@ describe('test MemberList', () => {
     };
   });
 
-  afterEach(() => {
-    axios.get.mockClear();
-  });
-
-  test('render MemberList', async () => {
-    await act(async () => {
-      await create(wrapComponent(route, localState));
-    });
-    expect(axios.get).toHaveBeenCalledWith(API.SHARE.UNITS_MEMBERS(1), {});
-  });
-
   test('AlertAction rightButtonClick', async () => {
     let tree;
     await act(async () => {
@@ -95,100 +70,11 @@ describe('test MemberList', () => {
     await act(async () => {
       alertAction.props.rightButtonClick();
     });
-  });
-
-  test('get dataMembers', async () => {
-    const response = {
-      status: 200,
-      data: [
-        {
-          id: 1,
-          name: 'a',
-          share_id: 1,
-        },
-      ],
-    };
-    axios.get.mockImplementation(async () => {
-      return response;
-    });
-
-    let tree;
-    await act(async () => {
-      tree = await create(wrapComponent(route, localState));
-    });
-    const instance = tree.root;
-    const sharingMember = instance.findAll(
-      (el) => el.props.testID === TESTID.SHARING_MEMBER
-    );
-    expect(sharingMember).toHaveLength(1);
-    expect(axios.get).toHaveBeenCalledWith(API.SHARE.UNITS_MEMBERS(1), {});
-  });
-
-  test('WrapHeaderScrollable rightHeader', async () => {
-    const response = {
-      status: 200,
-      data: [
-        {
-          id: 1,
-          name: 'a',
-        },
-      ],
-    };
-    axios.get.mockImplementation(async () => {
-      return response;
-    });
-
-    let tree;
-    await act(async () => {
-      tree = await create(wrapComponent(route, localState));
-    });
-    const instance = tree.root;
-    const button = instance.find(
-      (el) =>
-        el.props.testID === TESTID.MEMBER_LIST_RIGHT_HEADER_TOUCH &&
-        el.type === TouchableOpacity
-    );
-    await act(async () => {
-      await button.props.onPress();
-    });
-    expect(axios.get).toHaveBeenCalledWith(API.SHARE.UNITS_MEMBERS(1), {});
-    expect(mockedNavigate).toHaveBeenCalledWith(Routes.AddMemberStack, {
+    const MemberListButtons = instance.findAllByType(TouchableOpacity);
+    await MemberListButtons[1].props.onPress();
+    expect(mockedNavigate).toBeCalledWith(Routes.AddMemberStack, {
       screen: Routes.SharingSelectPermission,
-      params: { unit: { id: route.params.unitId } },
-    });
-  });
-
-  test('WrapHeaderScrollable rightHeader but not owner', async () => {
-    route.params.unit.user_id = 1;
-    const response = {
-      status: 200,
-      data: [
-        {
-          id: 1,
-          name: 'a',
-        },
-      ],
-    };
-    axios.get.mockImplementation(async () => {
-      return response;
-    });
-
-    let tree;
-    await act(async () => {
-      tree = await create(wrapComponent(route, localState));
-    });
-    const instance = tree.root;
-    const button = instance.find(
-      (el) =>
-        el.props.testID === TESTID.MEMBER_LIST_RIGHT_HEADER_TOUCH &&
-        el.type === TouchableOpacity
-    );
-    const alertAction = instance.findByType(AlertAction);
-    expect(alertAction.props.visible).toEqual(false);
-    await act(async () => {
-      await button.props.onPress();
+      params: { unit: { id: 1 } },
     });
-    expect(alertAction.props.visible).toEqual(true);
-    expect(axios.get).toHaveBeenCalledWith(API.SHARE.UNITS_MEMBERS(1), {});
   });
 });

package/src/screens/Sharing/__test__/MemberList2.test.js

@@ -0,0 +1,80 @@
+import React from 'react';
+import { create, act } from 'react-test-renderer';
+
+import MemberList from '../MemberList';
+import { AlertAction } from '../../../commons';
+import { SCProvider } from '../../../context';
+import { mockSCStore } from '../../../context/mockStore';
+import { TouchableOpacity } from 'react-native';
+import Routes from '../../../utils/Route';
+
+jest.mock('../../../hooks/Common', () => {
+  return {
+    useIsOwnerOfUnit: () => ({ isOwner: false }),
+  };
+});
+
+const mockedNavigate = jest.fn();
+const mockUseIsFocused = jest.fn();
+
+jest.mock('@react-navigation/native', () => {
+  return {
+    ...jest.requireActual('@react-navigation/native'),
+    useNavigation: () => ({
+      navigate: mockedNavigate,
+    }),
+    useIsFocused: () => ({
+      isFocused: mockUseIsFocused,
+    }),
+  };
+});
+
+const wrapComponent = (route, state) => (
+  <SCProvider initState={mockSCStore({})}>
+    <MemberList route={route} />
+  </SCProvider>
+);
+
+describe('test MemberList', () => {
+  let route;
+  let localState = {
+    auth: {
+      account: {
+        user: {
+          id: 2,
+        },
+      },
+    },
+  };
+
+  beforeEach(() => {
+    route = {
+      params: {
+        unitId: 1,
+        unit: {
+          id: 1,
+          user_id: 2,
+          name: '',
+        },
+      },
+    };
+  });
+
+  test('AlertAction rightButtonClick', async () => {
+    let tree;
+    await act(async () => {
+      tree = await create(wrapComponent(route, localState));
+    });
+    const instance = tree.root;
+    const alertAction = instance.findByType(AlertAction);
+    await act(async () => {
+      alertAction.props.rightButtonClick();
+    });
+    const MemberListButtons = instance.findAllByType(TouchableOpacity);
+    await MemberListButtons[1].props.onPress();
+    expect(mockedNavigate).not.toBeCalledWith(Routes.AddMemberStack, {
+      screen: Routes.SharingSelectPermission,
+      params: { unit: { id: 1 } },
+    });
+  });
+});