Fix unit test

wazuh · Dec 15, 2023 · a112e78 · a112e78
1 parent 075cc26
commit a112e78
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 36 deletions.
diff --git a/...blic/components/endpoints-summary/register-agent/services/register-agent-services.test.ts b/...blic/components/endpoints-summary/register-agent/services/register-agent-services.test.ts
@@ -9,16 +9,6 @@ jest.mock('../../../../react-services', () => ({
   }),
 }));
 
-const mockedResponseClusterStatus = {
-  data: {
-    data: {
-      enabled: 'yes',
-      running: 'yes',
-    },
-    error: 0,
-  },
-};
-
 describe('Register agent service', () => {
   beforeEach(() => jest.clearAllMocks());
   describe('getRemoteConfiguration', () => {
@@ -51,14 +41,13 @@ describe('Register agent service', () => {
         },
       };
 
-      WzRequest.apiReq = jest
-        .fn()
-        .mockResolvedValueOnce(mockedResponseClusterStatus)
-        .mockResolvedValueOnce(mockedResponse);
+      WzRequest.apiReq = jest.fn().mockResolvedValueOnce(mockedResponse);
       const nodeName = 'example-node';
       const res = await RegisterAgentService.getRemoteConfiguration(
-        'example-node',
+        nodeName,
+        false,
       );
+      console.log(res);
       expect(res.name).toBe(nodeName);
       expect(res.haveSecureConnection).toBe(true);
     });
@@ -84,13 +73,11 @@ describe('Register agent service', () => {
           },
         },
       };
-      WzRequest.apiReq = jest
-        .fn()
-        .mockResolvedValueOnce(mockedResponseClusterStatus)
-        .mockResolvedValueOnce(mockedResponse);
+      WzRequest.apiReq = jest.fn().mockResolvedValueOnce(mockedResponse);
       const nodeName = 'example-node';
       const res = await RegisterAgentService.getRemoteConfiguration(
-        'example-node',
+        nodeName,
+        false,
       );
       expect(res.name).toBe(nodeName);
       expect(res.haveSecureConnection).toBe(false);
@@ -124,13 +111,11 @@ describe('Register agent service', () => {
           },
         },
       };
-      WzRequest.apiReq = jest
-        .fn()
-        .mockResolvedValueOnce(mockedResponseClusterStatus)
-        .mockResolvedValueOnce(mockedResponse);
+      WzRequest.apiReq = jest.fn().mockResolvedValueOnce(mockedResponse);
       const nodeName = 'example-node';
       const res = await RegisterAgentService.getRemoteConfiguration(
-        'example-node',
+        nodeName,
+        false,
       );
       expect(res.name).toBe(nodeName);
       expect(res.isUdp).toEqual(true);
@@ -164,13 +149,11 @@ describe('Register agent service', () => {
           },
         },
       };
-      WzRequest.apiReq = jest
-        .fn()
-        .mockResolvedValueOnce(mockedResponseClusterStatus)
-        .mockResolvedValueOnce(mockedResponse);
+      WzRequest.apiReq = jest.fn().mockResolvedValueOnce(mockedResponse);
       const nodeName = 'example-node';
       const res = await RegisterAgentService.getRemoteConfiguration(
-        'example-node',
+        nodeName,
+        false,
       );
       expect(res.name).toBe(nodeName);
       expect(res.isUdp).toEqual(false);
@@ -204,13 +187,11 @@ describe('Register agent service', () => {
           },
         },
       };
-      WzRequest.apiReq = jest
-        .fn()
-        .mockResolvedValueOnce(mockedResponseClusterStatus)
-        .mockResolvedValueOnce(mockedResponse);
+      WzRequest.apiReq = jest.fn().mockResolvedValueOnce(mockedResponse);
       const nodeName = 'example-node';
       const res = await RegisterAgentService.getRemoteConfiguration(
-        'example-node',
+        nodeName,
+        false,
       );
       expect(res.name).toBe(nodeName);
       expect(res.isUdp).toEqual(false);

diff --git a/...n/public/components/endpoints-summary/register-agent/services/register-agent-services.tsx b/...n/public/components/endpoints-summary/register-agent/services/register-agent-services.tsx
@@ -73,7 +73,7 @@ async function getRemoteConfiguration(
         {},
       );
     }
-    const items = ((result.data || {}).data || {}).affected_items || [];
+    const items = result?.data?.data?.affected_items || [];
     const remote = items[0]?.remote;
     if (remote) {
       const remoteFiltered = remote.filter((item: RemoteItem) => {