Add automated tests

assets/js/components/HostDetails/HostDetails.test.jsx

@@ -4,6 +4,7 @@ import userEvent from '@testing-library/user-event';
 import 'intersection-observer';
 import '@testing-library/jest-dom';
 import MockAdapter from 'axios-mock-adapter';
+import { faker } from '@faker-js/faker';
 
 import {
   withState,
@@ -49,6 +50,48 @@ describe('HostDetails component', () => {
     ).toBeVisible();
   });
 
+  it('should disable start execution button when checks are not selected', () => {
+    const host = hostFactory.build({ selected_checks: [] });
+    const { id: hostID } = host;
+    const state = {
+      ...defaultInitialState,
+      hostsList: {
+        hosts: [host],
+      },
+    };
+    const [StatefulHostDetails] = withState(<HostDetails />, state);
+
+    renderWithRouterMatch(StatefulHostDetails, {
+      path: '/hosts/:hostID',
+      route: `/hosts/${hostID}`,
+    });
+
+    const startExecutionButton = screen.getByText('Start Execution');
+    expect(startExecutionButton).toBeDisabled();
+  });
+
+  it('should enable start execution button when checks are selected', () => {
+    const host = hostFactory.build({
+      selected_checks: [faker.animal.bear(), faker.animal.bear()],
+    });
+    const { id: hostID } = host;
+    const state = {
+      ...defaultInitialState,
+      hostsList: {
+        hosts: [host],
+      },
+    };
+    const [StatefulHostDetails] = withState(<HostDetails />, state);
+
+    renderWithRouterMatch(StatefulHostDetails, {
+      path: '/hosts/:hostID',
+      route: `/hosts/${hostID}`,
+    });
+
+    const startExecutionButton = screen.getByText('Start Execution');
+    expect(startExecutionButton).toBeEnabled();
+  });
+
   describe('agent version', () => {
     it('should not show any warning message if the agent version is correct', () => {
       const hosts = hostFactory.buildList(1, { agent_version: '2.0.0' });

assets/js/components/HostDetails/HostSettingsPage.test.jsx

@@ -70,4 +70,44 @@ describe('HostSettingsPage component', () => {
 
     expect(screen.getByText('Save Checks Selection')).toBeVisible();
   });
+
+  it('should render HostSettingsPage with a disabled start execution button, as no checks are selected', () => {
+    const hosts = hostFactory.buildList(2, {
+      provider: 'azure',
+      selected_checks: [],
+    });
+    const state = {
+      ...defaultInitialState,
+      hostsList: { hosts },
+    };
+    const { id: hostID } = hosts[1];
+
+    const [StatefulHostSettingsPage] = withState(<HostSettingsPage />, state);
+
+    renderWithRouterMatch(StatefulHostSettingsPage, {
+      path: 'hosts/:hostID/settings',
+      route: `/hosts/${hostID}/settings`,
+    });
+    const startExecutionButton = screen.getByText('Start Execution');
+    expect(startExecutionButton).toBeDisabled();
+  });
+  it('should render HostSettingsPage with a disabled start execution button, as checks are selected', () => {
+    const hosts = hostFactory.buildList(2, {
+      provider: 'azure',
+      selected_checks: [faker.animal.bear(), faker.animal.bear()],
+    });
+    const state = {
+      ...defaultInitialState,
+      hostsList: { hosts },
+    };
+    const { id: hostID } = hosts[1];
+    const [StatefulHostSettingsPage] = withState(<HostSettingsPage />, state);
+
+    renderWithRouterMatch(StatefulHostSettingsPage, {
+      path: 'hosts/:hostID/settings',
+      route: `/hosts/${hostID}/settings`,
+    });
+    const startExecutionButton = screen.getByText('Start Execution');
+    expect(startExecutionButton).toBeEnabled();
+  });
 });

assets/js/state/lastExecutions.test.js

@@ -2,11 +2,13 @@ import { faker } from '@faker-js/faker';
 
 import lastExecutionsReducer, {
   setExecutionRequested,
+  setHostChecksExecutionRequested,
   setLastExecutionLoading,
   setLastExecutionEmpty,
   setLastExecutionError,
   setLastExecution,
   setExecutionStarted,
+  REQUESTED_EXECUTION_STATE,
 } from './lastExecutions';
 
 describe('lastExecutions reducer', () => {
@@ -183,4 +185,31 @@ describe('lastExecutions reducer', () => {
 
     expect(lastExecutionsReducer(initialState, action)).toEqual(expectedState);
   });
+
+  it('should set requested state on host execution', () => {
+    const initialState = {};
+    const hostID = faker.datatype.uuid();
+    const checks = [faker.datatype.uuid(), faker.datatype.uuid()];
+
+    const agentID = faker.datatype.uuid();
+
+    const host = {
+      agent_id: agentID,
+      id: hostID,
+    };
+    const data = { host, checks };
+    const expectedState = {
+      [hostID]: {
+        data: {
+          status: REQUESTED_EXECUTION_STATE,
+          targets: [{ agent_id: host, checks }],
+        },
+        loading: false,
+        error: null,
+      },
+    };
+
+    const action = setHostChecksExecutionRequested(data);
+    expect(lastExecutionsReducer(initialState, action)).toEqual(expectedState);
+  });
 });

assets/js/state/sagas/lastExecutions.test.js

@@ -10,9 +10,16 @@ import {
   setLastExecutionEmpty,
   setLastExecutionError,
   setExecutionRequested,
+  setHostChecksExecutionRequested,
 } from '@state/lastExecutions';
 import { notify } from '@state/actions/notifications';
-import { updateLastExecution, requestExecution } from './lastExecutions';
+import { hostFactory } from '@lib/test-utils/factories';
+
+import {
+  updateLastExecution,
+  requestExecution,
+  requestHostExecution,
+} from './lastExecutions';
 
 const axiosMock = new MockAdapter(networkClient);
 const lastExecutionURL = (groupID) =>
@@ -21,6 +28,9 @@ const lastExecutionURL = (groupID) =>
 const triggerChecksExecutionURL = (clusterId) =>
   `/clusters/${clusterId}/checks/request_execution`;
 
+const hostTriggerChecksExecutionURL = (hostID) =>
+  `/hosts/${hostID}/checks/request_execution`;
+
 describe('lastExecutions saga', () => {
   beforeEach(() => {
     axiosMock.reset();
@@ -140,4 +150,46 @@ describe('lastExecutions saga', () => {
       })
     );
   });
+
+  it('should set the last host execution to requested state', async () => {
+    const host = hostFactory.build();
+    const { id: hostID, hostname: hostName } = host;
+    const checks = [faker.datatype.uuid(), faker.datatype.uuid()];
+
+    axiosMock.onPost(hostTriggerChecksExecutionURL(hostID)).reply(202, {});
+    const payload = { checks, host };
+
+    const dispatched = await recordSaga(requestHostExecution, {
+      payload,
+    });
+    expect(dispatched).toContainEqual(setHostChecksExecutionRequested(payload));
+    expect(dispatched).toContainEqual(
+      notify({
+        text: `Checks execution requested, host: ${hostName}`,
+        icon: '🐰',
+      })
+    );
+  });
+
+  it('should not set the host last execution to requested state on failure', async () => {
+    const host = hostFactory.build();
+    const { id: hostID, hostname: hostName } = host;
+    const checks = [faker.datatype.uuid(), faker.datatype.uuid()];
+
+    axiosMock.onPost(hostTriggerChecksExecutionURL(hostID)).reply(400, {});
+
+    const payload = { checks, host };
+    const dispatched = await recordSaga(requestHostExecution, {
+      payload,
+    });
+    expect(dispatched).not.toContainEqual(
+      setHostChecksExecutionRequested(payload)
+    );
+    expect(dispatched).toContainEqual(
+      notify({
+        text: `Unable to start execution for host: ${hostName}`,
+        icon: '❌',
+      })
+    );
+  });
 });