@eohjsc/react-native-smart-city 0.2.98 → 0.3.1

package/src/screens/AddNewGateway/PlugAndPlay/ConnectWifiWarning.js

@@ -56,7 +56,7 @@ const ConnectWifiWarning = memo(({ route }) => {
   const { setAction } = useContext(SCContext);
 
   const getPermissionWifiAndroid = useCallback(async () => {
-    const granted = await PermissionsAndroid.request(
+    return await PermissionsAndroid.request(
       PermissionsAndroid.PERMISSIONS.ACCESS_FINE_LOCATION,
       {
         title: t('location_permission_required_wifi_title'),
@@ -65,15 +65,21 @@ const ConnectWifiWarning = memo(({ route }) => {
         buttonPositive: t('allow'),
       }
     );
-    if (granted === PermissionsAndroid.RESULTS.GRANTED) {
-      ToastBottomHelper.success('GRANTED');
-    } else {
-      ToastBottomHelper.error('DENIED');
-    }
-  }, [t]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, []);
 
   useEffect(() => {
-    isAndroid && getPermissionWifiAndroid();
+    if (isAndroid) {
+      (async () => {
+        const granted = await getPermissionWifiAndroid();
+        if (!granted) {
+          ToastBottomHelper.error(
+            'Fail permission',
+            'You need to grant permission to continue'
+          );
+        }
+      })();
+    }
   }, [getPermissionWifiAndroid]);
 
   const handleSend = async () => {
@@ -83,7 +89,7 @@ const ConnectWifiWarning = memo(({ route }) => {
       await socket.on('message', (msg, rinfo) => {
         clearInterval(intervalSend);
         const data = JSON.parse(msg.toString());
-        if (data.hasOwnProperty('wifi')) {
+        if (Object.prototype.hasOwnProperty.call(data, 'wifi')) {
           navigate(Routes.GatewayWifiList, {
             list_wifi: data.wifi,
             unit_id: unit_id,
@@ -113,12 +119,12 @@ const ConnectWifiWarning = memo(({ route }) => {
         goBack();
       });
     } else {
-      const socket = dgram.createSocket({ type: 'udp4' });
+      const dgSocket = dgram.createSocket({ type: 'udp4' });
       await setIsPercentConnect(1);
-      await socket.bind(54321);
-      await socket.on('message', (msg, rinfo) => {
+      await dgSocket.bind(54321);
+      await dgSocket.on('message', (msg, rinfo) => {
         const data = JSON.parse(msg.toString());
-        if (data.hasOwnProperty('wifi')) {
+        if (Object.prototype.hasOwnProperty.call(data, 'wifi')) {
           navigate(Routes.GatewayWifiList, {
             list_wifi: data.wifi,
             unit_id: unit_id,
@@ -128,12 +134,12 @@ const ConnectWifiWarning = memo(({ route }) => {
             wifi_pass: wifi_pass,
             unit_name: unit_name,
             devicePrefixName: devicePrefixName,
-            socket: socket,
+            socket: dgSocket,
           });
         }
       });
-      await socket.once('listening', async () => {
-        await socket.send(
+      await dgSocket.once('listening', async () => {
+        await dgSocket.send(
           JSON.stringify({ type: 'scan', data: { wifi: '' } }),
           undefined,
           undefined,
@@ -142,7 +148,7 @@ const ConnectWifiWarning = memo(({ route }) => {
           undefined
         );
       });
-      socket.on('error', () => {
+      dgSocket.on('error', () => {
         ToastBottomHelper.error(t('server_error'));
         setAction(Action.IS_CONNECT_WIFI_GATEWAY, false);
         goBack();

package/src/screens/AddNewGateway/PlugAndPlay/GatewayWifiList.js

@@ -60,8 +60,8 @@ const GatewayWifiList = memo(({ route }) => {
   };
 
   const onPasswordChange = useCallback(
-    (password) => {
-      setPassword(password?.toString());
+    (newPassword) => {
+      setPassword(newPassword?.toString());
     },
     [setPassword]
   );
@@ -69,7 +69,7 @@ const GatewayWifiList = memo(({ route }) => {
   const handleSocketOnMsg = useCallback(
     (msg) => {
       const data = JSON.parse(msg.toString());
-      if (data.hasOwnProperty('gateway')) {
+      if (Object.prototype.hasOwnProperty.call(data, 'gateway')) {
         const checkWifiInterval = setInterval(() => {
           // eslint-disable-next-line promise/prefer-await-to-then
           WifiManager.getCurrentWifiSSID().then((ssid) => {

package/src/screens/AddNewGateway/SetupGatewayWifi.js

@@ -52,6 +52,7 @@ const SetupGatewayWifi = memo(({ route }) => {
             setWifiName(ssid);
           });
         }
+        // eslint-disable-next-line no-empty
       } catch (err) {}
     };
     wifiPermission();

package/src/screens/AddNewGateway/__test__/AddNewGateway.test.js

@@ -1,6 +1,6 @@
 import React from 'react';
 import { act, create } from 'react-test-renderer';
-import axios from 'axios';
+import MockAdapter from 'axios-mock-adapter';
 
 import AddNewGateway from '../index';
 import GroupCheckBox from '../../../commons/GroupCheckBox';
@@ -10,6 +10,7 @@ import { TextInput } from 'react-native';
 import { getTranslate } from '../../../utils/I18n';
 import { SCProvider } from '../../../context';
 import { mockSCStore } from '../../../context/mockStore';
+import api from '../../../utils/Apis/axios';
 
 const wrapComponent = (route) => (
   <SCProvider initState={mockSCStore({})}>
@@ -17,7 +18,7 @@ const wrapComponent = (route) => (
   </SCProvider>
 );
 
-jest.mock('axios');
+const mock = new MockAdapter(api.axiosInstance);
 
 jest.mock('react', () => {
   return { ...jest.requireActual('react'), memo: (x) => x };
@@ -41,7 +42,6 @@ describe('Test AddNewGateway', () => {
   let route;
 
   afterEach(() => {
-    axios.get.mockClear();
     mockedNavigate.mockClear();
   });
 
@@ -149,9 +149,7 @@ describe('Test AddNewGateway', () => {
         stations: [{ id: 2, name: 'Station name' }],
       },
     };
-    axios.get.mockImplementation(async () => {
-      return response;
-    });
+    mock.onGet().reply(200, response.data);
 
     await act(async () => {
       tree = await create(wrapComponent(route));

package/src/screens/AddNewGateway/__test__/ConnectedGateway.test.js

@@ -1,6 +1,5 @@
 import React from 'react';
 import { act, create } from 'react-test-renderer';
-import axios from 'axios';
 
 import { TESTID } from '../../../configs/Constants';
 import ConnectedGateway from '../ConnectedGateway';
@@ -8,8 +7,6 @@ import { SCProvider } from '../../../context';
 import { mockSCStore } from '../../../context/mockStore';
 import { getTranslate } from '../../../utils/I18n';
 
-jest.mock('axios');
-
 const mockedNavigate = jest.fn();
 jest.mock('@react-navigation/native', () => {
   return {
@@ -31,7 +28,6 @@ describe('Test ConnectedGateway', () => {
   let route;
 
   afterEach(() => {
-    axios.get.mockClear();
     mockedNavigate.mockClear();
   });
 

package/src/screens/AddNewGateway/__test__/ConnectingGateway.test.js

@@ -1,6 +1,6 @@
 import React from 'react';
 import { act, create } from 'react-test-renderer';
-import axios from 'axios';
+import MockAdapter from 'axios-mock-adapter';
 
 import ConnectingGateway from '../ConnectingGateway';
 import Text from '../../../commons/Text';
@@ -9,8 +9,9 @@ import Routes from '../../../utils/Route';
 import { getTranslate } from '../../../utils/I18n';
 import { SCProvider } from '../../../context';
 import { mockSCStore } from '../../../context/mockStore';
+import api from '../../../utils/Apis/axios';
 
-jest.mock('axios');
+const mock = new MockAdapter(api.axiosInstance);
 
 const mockedNavigate = jest.fn();
 jest.mock('@react-navigation/native', () => {
@@ -33,7 +34,6 @@ describe('Test ConnectingGateway', () => {
   let route;
 
   afterEach(() => {
-    axios.get.mockClear();
     mockedNavigate.mockClear();
   });
 
@@ -64,14 +64,7 @@ describe('Test ConnectingGateway', () => {
 
   test('call setInterval api success', async () => {
     jest.useFakeTimers();
-    const response = {
-      status: 200,
-      data: {},
-    };
-    axios.get.mockImplementation(async () => {
-      return response;
-    });
-
+    mock.onGet(API.CHIP.CHECK_FINALIZED()).reply(200, {});
     await act(async () => {
       tree = await create(wrapComponent(route));
     });
@@ -79,11 +72,6 @@ describe('Test ConnectingGateway', () => {
       await jest.runOnlyPendingTimers();
     });
     expect(setInterval).toHaveBeenCalled();
-    expect(axios.get).toHaveBeenCalledWith(API.CHIP.CHECK_FINALIZED(), {
-      params: {
-        chip_id: 1,
-      },
-    });
     expect(mockedNavigate).not.toHaveBeenCalledWith(
       Routes.ConnectedGateway,
       route.params
@@ -92,14 +80,7 @@ describe('Test ConnectingGateway', () => {
 
   test('call setInterval api fail', async () => {
     jest.useFakeTimers();
-    const response = {
-      status: 400,
-      data: {},
-    };
-    axios.get.mockImplementation(async () => {
-      return response;
-    });
-
+    mock.onGet(API.CHIP.CHECK_FINALIZED()).reply(400);
     await act(async () => {
       tree = await create(wrapComponent(route));
     });
@@ -107,11 +88,6 @@ describe('Test ConnectingGateway', () => {
       await jest.runOnlyPendingTimers();
     });
     expect(setInterval).toHaveBeenCalled();
-    expect(axios.get).toHaveBeenCalledWith(API.CHIP.CHECK_FINALIZED(), {
-      params: {
-        chip_id: 1,
-      },
-    });
     expect(mockedNavigate).not.toHaveBeenCalledWith(
       Routes.ConnectedGateway,
       route.params

package/src/screens/AddNewGateway/__test__/SelectGateway.test.js

@@ -1,6 +1,5 @@
 import React from 'react';
 import { act, create } from 'react-test-renderer';
-import axios from 'axios';
 
 import AddNewGatewaySelectGateway from '../SelectGateway';
 import { SCProvider } from '../../../context';
@@ -13,8 +12,6 @@ const wrapComponent = (route) => (
   </SCProvider>
 );
 
-jest.mock('axios');
-
 jest.mock('react', () => {
   return { ...jest.requireActual('react'), memo: (x) => x };
 });
@@ -38,7 +35,6 @@ describe('Test AddNewGatewaySelectGateway', () => {
   let route;
 
   afterEach(() => {
-    axios.get.mockClear();
     mockedNavigate.mockClear();
   });
 

package/src/screens/AddNewGateway/__test__/SetupGateway.test.js

@@ -1,6 +1,5 @@
 import React from 'react';
 import { act, create } from 'react-test-renderer';
-import axios from 'axios';
 
 import { TESTID } from '../../../configs/Constants';
 import { SCProvider } from '../../../context';
@@ -16,8 +15,6 @@ const wrapComponent = (route) => (
   </SCProvider>
 );
 
-jest.mock('axios');
-
 jest.mock('react', () => {
   return { ...jest.requireActual('react'), memo: (x) => x };
 });
@@ -59,7 +56,6 @@ describe('Test SetupGatewayWifi', () => {
   let route;
 
   afterEach(() => {
-    axios.get.mockClear();
     mockedNavigate.mockClear();
   });
 

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

@@ -1,5 +1,5 @@
-import axios from 'axios';
 import React from 'react';
+import MockAdapter from 'axios-mock-adapter';
 import { Platform, TextInput, TouchableOpacity } from 'react-native';
 import { act, create } from 'react-test-renderer';
 
@@ -9,13 +9,15 @@ import { SCProvider } from '../../../context';
 import { mockSCStore } from '../../../context/mockStore';
 import Routes from '../../../utils/Route';
 import { HeaderCustom } from '../../../commons/Header';
+import api from '../../../utils/Apis/axios';
+import { API } from '../../../configs';
 
 const wrapComponent = (route) => (
   <SCProvider initState={mockSCStore({})}>
     <AddNewOneTap route={route} />
   </SCProvider>
 );
-jest.mock('axios');
+const mock = new MockAdapter(api.axiosInstance);
 
 jest.mock('react-redux', () => {
   return {
@@ -41,14 +43,13 @@ jest.mock('react', () => {
   };
 });
 
-jest.mock('axios');
 let tree;
 
 describe('test AddNewOneTap', () => {
   beforeEach(() => {
-    axios.post.mockClear();
     mockedNavigate.mockClear();
   });
+
   test('create AddNewOneTap success', async () => {
     Platform.OS = 'ios';
     let route = {
@@ -71,9 +72,7 @@ describe('test AddNewOneTap', () => {
       },
     };
 
-    axios.post.mockImplementation(async () => {
-      return response;
-    });
+    mock.onPost(API.AUTOMATE.CREATE_AUTOMATE()).reply(200, response.data);
 
     await act(async () => {
       tree = await create(wrapComponent(route));
@@ -100,19 +99,14 @@ describe('test AddNewOneTap', () => {
       await item[0].props.onPress();
     });
   });
+
   test('create AddNewOneTap fail', async () => {
     Platform.OS = 'android';
     let route = {
       params: { type: 'one_tap' },
     };
 
-    const response = {
-      status: 400,
-    };
-
-    axios.post.mockImplementation(async () => {
-      return response;
-    });
+    mock.onPost(API.AUTOMATE.CREATE_AUTOMATE()).reply(400);
 
     await act(async () => {
       tree = await create(wrapComponent(route));
@@ -144,15 +138,7 @@ describe('test AddNewOneTap', () => {
         isAutomateTab: false,
       },
     };
-
-    const response = {
-      status: 400,
-    };
-
-    axios.post.mockImplementation(async () => {
-      return response;
-    });
-
+    mock.onPost(API.AUTOMATE.CREATE_AUTOMATE()).reply(400);
     await act(async () => {
       tree = await create(wrapComponent(route));
     });
@@ -165,7 +151,7 @@ describe('test AddNewOneTap', () => {
     expect(mockedNavigate).toHaveBeenCalledWith(Routes.ScriptDetail, {
       id: 1,
       name: 'script test',
-      type: 'one_tap',
+      type: undefined,
       havePermission: true,
       unit: { id: 1 },
       isMultiUnits: false,

package/src/screens/AddNewOneTap/index.js

@@ -25,6 +25,7 @@ const AddNewOneTap = memo(({ route }) => {
     isMultiUnits,
     automateId,
     scriptName,
+    oldType,
   } = route.params;
   const t = useTranslations();
   const { navigate, dispatch, goBack } = useNavigation();
@@ -91,7 +92,7 @@ const AddNewOneTap = memo(({ route }) => {
       navigate(Routes.ScriptDetail, {
         id: automateId,
         name: scriptName,
-        type: type,
+        type: oldType,
         havePermission: true,
         unit,
         isMultiUnits,

package/src/screens/AllCamera/index.js

@@ -35,9 +35,9 @@ const AllCamera = () => {
   const [dataFullScreen, setDataFullScreen] = useState();
 
   const handleFullScreen = useCallback(
-    (data) => {
+    (dataFullscreen) => {
       setIsFullScreen(!isFullScreen);
-      setDataFullScreen(data);
+      setDataFullScreen(dataFullscreen);
     },
     [isFullScreen]
   );
@@ -74,8 +74,8 @@ const AllCamera = () => {
   };
 
   const goToPlayBack = useCallback(
-    (item, thumbnail) => () => {
-      navigate(Routes.PlaybackCamera, { item, thumbnail });
+    (item, itemThumbnail) => () => {
+      navigate(Routes.PlaybackCamera, { item, thumbnail: itemThumbnail });
     },
     [navigate]
   );

package/src/screens/Automate/MultiUnits.js

@@ -39,17 +39,17 @@ const MultiUnits = () => {
   );
 
   const getData = useCallback(
-    async (params) => {
+    async (fetchParams) => {
       if (isMultiUnits) {
-        const { success, data } = await axiosGet(
+        const { success, data: automateData } = await axiosGet(
           API.AUTOMATE.GET_MULTI_UNITS(),
-          params
+          fetchParams
         );
-        success && setData(data);
+        success && setData(automateData);
       } else {
         await fetchWithCache(API.UNIT.AUTOMATE(unit?.id), {}, (response) => {
-          const { success, data } = response;
-          success && setData(data);
+          const { success, data: automateData } = response;
+          success && setData(automateData);
         });
       }
     },
@@ -60,14 +60,14 @@ const MultiUnits = () => {
     setTabActive(tab);
   };
 
-  const onPressItem = (item, isOwner) => () => {
+  const onPressItem = (item, isItemOwner) => () => {
     navigate(Routes.UnitStack, {
       screen: Routes.ScriptDetail,
       params: {
         id: item?.id,
         name: item?.script?.name,
         type: item?.type,
-        havePermission: isOwner || idUser === item?.user,
+        havePermission: isItemOwner || idUser === item?.user,
         unit,
         isMultiUnits,
         automate: item,

package/src/screens/Automate/__test__/MultiUnits.test.js

@@ -1,7 +1,7 @@
 import React, { useState } from 'react';
 import { TouchableOpacity } from 'react-native';
 import { act, create } from 'react-test-renderer';
-import axios from 'axios';
+import MockAdapter from 'axios-mock-adapter';
 import { useRoute } from '@react-navigation/native';
 
 import MultiUnits from '../MultiUnits';
@@ -13,8 +13,10 @@ import Routes from '../../../utils/Route';
 import WrapHeaderScrollable from '../../../commons/Sharing/WrapHeaderScrollable';
 import { getTranslate } from '../../../utils/I18n';
 import { AUTOMATE_TYPE } from '../../../configs/Constants';
+import api from '../../../utils/Apis/axios';
+import { API } from '../../../configs';
 
-jest.mock('axios');
+const mock = new MockAdapter(api.axiosInstance);
 const mockedNavigate = jest.fn();
 const mockSetState = jest.fn();
 
@@ -47,7 +49,6 @@ describe('Test MultiUnits', () => {
   let tree;
 
   beforeEach(() => {
-    axios.get.mockClear();
     mockedNavigate.mockClear();
     mockSetState.mockClear();
     useRoute.mockClear();
@@ -93,9 +94,7 @@ describe('Test MultiUnits', () => {
         },
       ],
     };
-    await axios.get.mockImplementation(async () => {
-      return response;
-    });
+    mock.onGet(API.AUTOMATE.GET_MULTI_UNITS()).reply(200, response.data);
     useRoute.mockReturnValue({
       params: {
         isMultiUnits: true,
@@ -189,9 +188,7 @@ describe('Test MultiUnits', () => {
         },
       ],
     };
-    await axios.get.mockImplementation(async () => {
-      return response;
-    });
+    mock.onGet(API.AUTOMATE.GET_MULTI_UNITS()).reply(200, response.data);
     useRoute.mockReturnValue({
       params: {
         isMultiUnits: true,

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

@@ -1,7 +1,7 @@
 import React, { useState } from 'react';
 import { FlatList, TouchableOpacity } from 'react-native';
 import { act, create } from 'react-test-renderer';
-import axios from 'axios';
+import MockAdapter from 'axios-mock-adapter';
 
 import Automate from '../';
 import { SCProvider } from '../../../context';
@@ -10,8 +10,10 @@ import ItemOneTap from '../../../commons/SubUnit/OneTap/ItemOneTap';
 import ItemAddNew from '../../../commons/Device/ItemAddNew';
 import Routes from '../../../utils/Route';
 import { TESTID } from '../../../configs/Constants';
+import api from '../../../utils/Apis/axios';
+import { API } from '../../../configs';
 
-jest.mock('axios');
+const mock = new MockAdapter(api.axiosInstance);
 const mockedNavigate = jest.fn();
 const mockSetState = jest.fn();
 
@@ -46,7 +48,6 @@ describe('Test Automate', () => {
   let tree;
 
   beforeEach(() => {
-    axios.get.mockClear();
     mockedNavigate.mockClear();
     mockSetState.mockClear();
   });
@@ -86,9 +87,7 @@ describe('Test Automate', () => {
         },
       ],
     };
-    await axios.get.mockImplementation(async () => {
-      return response;
-    });
+    mock.onGet(API.AUTOMATE.GET_SMART()).reply(200, response.data);
 
     useState.mockImplementation(() => [false, mockSetState]);
     useState.mockImplementation(() => [response.data, mockSetState]);
@@ -141,9 +140,7 @@ describe('Test Automate', () => {
         { automates: [], unit_id: 3, unit_name: 'La Vida' },
       ],
     };
-    await axios.get.mockImplementation(async () => {
-      return response;
-    });
+    mock.onGet(API.AUTOMATE.GET_SMART()).reply(200, response.data);
 
     useState.mockImplementation(() => [false, mockSetState]);
     useState.mockImplementation(() => [response.data, mockSetState]);
@@ -199,9 +196,7 @@ describe('Test Automate', () => {
         { automates: [], unit_id: 3, unit_name: 'La Vida' },
       ],
     };
-    await axios.get.mockImplementation(async () => {
-      return response;
-    });
+    mock.onGet(API.AUTOMATE.GET_SMART()).reply(200, response.data);
 
     useState.mockImplementation(() => [false, mockSetState]);
     useState.mockImplementation(() => [response.data, mockSetState]);

package/src/screens/Automate/index.js

@@ -109,13 +109,13 @@ const Automate = () => {
       const isOwner = owner_unit_id
         ? owner_unit_id === idUser
         : type === UNIT_TYPES.MULTI;
-      const renderItemAutomate = ({ item }) => {
+      const renderItemAutomate = ({ item: automate }) => {
         return (
           <ItemOneTap
             isOwner={isOwner}
-            automate={item}
+            automate={automate}
             wrapSyles={styles.wrapAutomateItem}
-            onPressItem={onPressItem(item, unit_id, type, isOwner)}
+            onPressItem={onPressItem(automate, unit_id, type, isOwner)}
           />
         );
       };