foreman-tasks 1.1.3 → 1.2.0

data/webpack/ForemanTasks/Components/TaskActions/TaskAction.test.js

@@ -0,0 +1,60 @@
+import { testActionSnapshotWithFixtures } from '@theforeman/test';
+import API from 'foremanReact/API';
+import {
+  cancelTaskRequest,
+  resumeTaskRequest,
+  forceCancelTaskRequest,
+  unlockTaskRequest,
+} from './';
+
+jest.mock('foremanReact/components/common/table', () => ({
+  getTableItemsAction: jest.fn(controller => controller),
+}));
+
+jest.mock('foremanReact/API');
+
+const task = ['some-id', 'some-name'];
+
+const fixtures = {
+  'should cancelTaskRequest and succeed': () => cancelTaskRequest(...task),
+  'should cancelTaskRequest and fail': () => {
+    API.post.mockImplementation(() =>
+      Promise.reject(new Error('Network Error'))
+    );
+    return cancelTaskRequest(...task);
+  },
+
+  'should resumeTaskRequest and succeed': () => {
+    API.post.mockImplementation(() => ({ data: 'some-data' }));
+    return resumeTaskRequest(...task);
+  },
+  'should resumeTaskRequest and fail': () => {
+    API.post.mockImplementation(() =>
+      Promise.reject(new Error('Network Error'))
+    );
+    return resumeTaskRequest(...task);
+  },
+  'should forceCancelTaskRequest and succeed': () => {
+    API.post.mockImplementation(() => ({ data: 'some-data' }));
+    return forceCancelTaskRequest(...task);
+  },
+  'should forceCancelTaskRequest and fail': () => {
+    API.post.mockImplementation(() =>
+      Promise.reject(new Error('Network Error'))
+    );
+    return forceCancelTaskRequest(...task);
+  },
+  'should unlockTaskRequest and succeed': () => {
+    API.post.mockImplementation(() => ({ data: 'some-data' }));
+    return unlockTaskRequest(...task);
+  },
+  'should unlockTaskRequest and fail': () => {
+    API.post.mockImplementation(() =>
+      Promise.reject(new Error('Network Error'))
+    );
+    return forceCancelTaskRequest(...task);
+  },
+};
+describe('Tasks actions', () => {
+  testActionSnapshotWithFixtures(fixtures);
+});

data/webpack/ForemanTasks/Components/{TasksTable/TasksTableActionHelpers.js → TaskActions/TaskActionHelpers.js}

@@ -2,6 +2,11 @@ import { translate as __, sprintf } from 'foremanReact/common/I18n';
 import { addToast } from 'foremanReact/redux/actions/toasts';
 import { TASKS_DASHBOARD_JS_QUERY_MODES } from '../TasksDashboard/TasksDashboardConstants';
 import { timeToHoursNumber } from '../TasksDashboard/TasksDashboardHelper';
+import {
+  successToastData,
+  errorToastData,
+  warningToastData,
+} from '../common/ToastsHelpers';
 
 export const convertDashboardQuery = query => {
   const {
@@ -29,14 +34,24 @@ export const convertDashboardQuery = query => {
 };
 
 export const resumeToastInfo = {
-  resumed: { type: 'success', text: __('was resumed') },
-  failed: { type: 'error', text: __('could not be resumed') },
-  skipped: { type: 'warning', text: __('task has to be resumable') },
+  resumed: successToastData(__('was resumed')),
+  failed: errorToastData(__('could not be resumed')),
+  skipped: warningToastData(__('task has to be resumable')),
 };
 
 export const cancelToastInfo = {
-  cancelled: { type: 'success', text: __('was cancelled') },
-  skipped: { type: 'warning', text: __('task has to be cancellable') },
+  cancelled: successToastData(__('was cancelled')),
+  skipped: warningToastData(__('task has to be cancellable')),
+};
+
+export const forceCancelToastInfo = {
+  forceCancelled: successToastData(__('resources were unlocked with force.')),
+  failed: warningToastData(__('cannot be cancelled with force at the moment.')),
+};
+
+export const unlockToastInfo = {
+  unlocked: successToastData(__('resources were unlocked ')),
+  failed: warningToastData(__('resources cannot be unlocked at the moment.')),
 };
 
 export const toastDispatch = ({ type, name, toastInfo, dispatch }) => {
@@ -45,7 +60,7 @@ export const toastDispatch = ({ type, name, toastInfo, dispatch }) => {
       type: toastInfo[type].type,
       message: sprintf('%(name)s Task execution %(type)s', {
         name,
-        type: toastInfo[type].text,
+        type: toastInfo[type].message,
       }),
     })
   );

data/webpack/ForemanTasks/Components/{TasksTable/__tests__/TasksTableActionHelpers.test.js → TaskActions/TaskActionHelpers.test.js}

@@ -1,8 +1,8 @@
-import { convertDashboardQuery } from '../TasksTableActionHelpers';
+import { convertDashboardQuery } from './TaskActionHelpers';
 import {
   TASKS_DASHBOARD_JS_QUERY_MODES,
   TASKS_DASHBOARD_AVAILABLE_TIMES,
-} from '../../TasksDashboard/TasksDashboardConstants';
+} from '../TasksDashboard/TasksDashboardConstants';
 
 let realDate;
 

data/webpack/ForemanTasks/Components/TaskActions/TaskActionsConstants.js

@@ -0,0 +1,16 @@
+export const UNLOCK_MODAL = 'unlockModal';
+export const FORCE_UNLOCK_MODAL = 'forceUnlockModal';
+
+export const TASKS_RESUME_REQUEST = 'TASKS_RESUME_REQUEST';
+export const TASKS_RESUME_SUCCESS = 'TASKS_RESUME_SUCCESS';
+export const TASKS_RESUME_FAILURE = 'TASKS_RESUME_FAILURE';
+export const TASKS_CANCEL_REQUEST = 'TASKS_CANCEL_REQUEST';
+export const TASKS_CANCEL_SUCCESS = 'TASKS_CANCEL_SUCCESS';
+export const TASKS_CANCEL_FAILURE = 'TASKS_CANCEL_FAILURE';
+
+export const TASKS_FORCE_CANCEL_REQUEST = 'TASKS_FORCE_CANCEL_REQUEST';
+export const TASKS_FORCE_CANCEL_SUCCESS = 'TASKS_FORCE_CANCEL_SUCCESS';
+export const TASKS_FORCE_CANCEL_FAILURE = 'TASKS_FORCE_CANCEL_FAILURE';
+export const TASKS_UNLOCK_REQUEST = 'TASKS_UNLOCK_REQUEST';
+export const TASKS_UNLOCK_SUCCESS = 'TASKS_UNLOCK_SUCCESS';
+export const TASKS_UNLOCK_FAILURE = 'TASKS_UNLOCK_FAILURE';

data/webpack/ForemanTasks/Components/TaskActions/UnlockModals.js

@@ -0,0 +1,60 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { translate as __, sprintf } from 'foremanReact/common/I18n';
+import { ClickConfirmation } from '../common/ClickConfirmation';
+import { UNLOCK_MODAL, FORCE_UNLOCK_MODAL } from './TaskActionsConstants';
+
+const confirmationMessage = __(
+  'I understand that this may cause harm and have working database backups of all backend services.'
+);
+
+export const UnlockModal = ({ onClick, id }) => (
+  <ClickConfirmation
+    id={id}
+    title={__('Unlock')}
+    body={__(
+      "This will unlock the resources that the task is running against. Please note that this might lead to inconsistent state and should be used with caution, after making sure that the task can't be resumed."
+    )}
+    confirmationMessage={confirmationMessage}
+    confirmAction={__('Unlock')}
+    onClick={onClick}
+    confirmType="warning"
+  />
+);
+
+export const ForceUnlockModal = ({ onClick, id, selectedRowsLen }) => (
+  <ClickConfirmation
+    id={id}
+    title={__('Force Unlock')}
+    body={sprintf(
+      __(
+        `Resources for %s task(s) will be unlocked and will not prevent other tasks from being run. As the task(s) might be still running, it should be avoided to use this unless you are really sure the task(s) got stuck.`
+      ),
+      selectedRowsLen
+    )}
+    confirmationMessage={confirmationMessage}
+    confirmAction={__('Force Unlock')}
+    onClick={onClick}
+    confirmType="danger"
+  />
+);
+
+UnlockModal.propTypes = {
+  onClick: PropTypes.func.isRequired,
+  id: PropTypes.string,
+};
+
+ForceUnlockModal.propTypes = {
+  onClick: PropTypes.func.isRequired,
+  id: PropTypes.string,
+  selectedRowsLen: PropTypes.number,
+};
+
+UnlockModal.defaultProps = {
+  id: UNLOCK_MODAL,
+};
+
+ForceUnlockModal.defaultProps = {
+  id: FORCE_UNLOCK_MODAL,
+  selectedRowsLen: 1,
+};

data/webpack/ForemanTasks/Components/TaskActions/UnlockModals.test.js

@@ -0,0 +1,14 @@
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
+
+import { UnlockModal, ForceUnlockModal } from './UnlockModals';
+
+const fixtures = {
+  render: { taskID: 'some-id', onClick: jest.fn() },
+};
+
+describe('UnlockModal', () => {
+  testComponentSnapshotsWithFixtures(UnlockModal, fixtures);
+});
+describe('ForceUnlockModal', () => {
+  testComponentSnapshotsWithFixtures(ForceUnlockModal, fixtures);
+});

data/webpack/ForemanTasks/Components/TaskActions/__snapshots__/TaskAction.test.js.snap

@@ -0,0 +1,233 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Tasks actions should cancelTaskRequest and fail 1`] = `
+Array [
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "Trying to cancel some-name task",
+          "type": "info",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+  Array [
+    Object {
+      "type": "TASKS_CANCEL_REQUEST",
+    },
+  ],
+  Array [
+    Object {
+      "payload": [Error: Network Error],
+      "type": "TASKS_CANCEL_FAILURE",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "some-name Task execution task has to be cancellable",
+          "type": "warning",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;
+
+exports[`Tasks actions should cancelTaskRequest and succeed 1`] = `
+Array [
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "Trying to cancel some-name task",
+          "type": "info",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+  Array [
+    Object {
+      "type": "TASKS_CANCEL_REQUEST",
+    },
+  ],
+  Array [
+    Object {
+      "type": "TASKS_CANCEL_SUCCESS",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "some-name Task execution was cancelled",
+          "type": "success",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;
+
+exports[`Tasks actions should forceCancelTaskRequest and fail 1`] = `
+Array [
+  Array [
+    Object {
+      "type": "TASKS_FORCE_CANCEL_REQUEST",
+    },
+  ],
+  Array [
+    Object {
+      "type": "TASKS_FORCE_CANCEL_FAILURE",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "some-name Task execution cannot be cancelled with force at the moment.",
+          "type": "warning",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;
+
+exports[`Tasks actions should forceCancelTaskRequest and succeed 1`] = `
+Array [
+  Array [
+    Object {
+      "type": "TASKS_FORCE_CANCEL_REQUEST",
+    },
+  ],
+  Array [
+    Object {
+      "type": "TASKS_FORCE_CANCEL_SUCCESS",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "some-name Task execution resources were unlocked with force.",
+          "type": "success",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;
+
+exports[`Tasks actions should resumeTaskRequest and fail 1`] = `
+Array [
+  Array [
+    Object {
+      "type": "TASKS_RESUME_REQUEST",
+    },
+  ],
+  Array [
+    Object {
+      "payload": [Error: Network Error],
+      "type": "TASKS_RESUME_FAILURE",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "some-name Task execution could not be resumed",
+          "type": "error",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;
+
+exports[`Tasks actions should resumeTaskRequest and succeed 1`] = `
+Array [
+  Array [
+    Object {
+      "type": "TASKS_RESUME_REQUEST",
+    },
+  ],
+  Array [
+    Object {
+      "type": "TASKS_RESUME_SUCCESS",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "some-name Task execution was resumed",
+          "type": "success",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;
+
+exports[`Tasks actions should unlockTaskRequest and fail 1`] = `
+Array [
+  Array [
+    Object {
+      "type": "TASKS_FORCE_CANCEL_REQUEST",
+    },
+  ],
+  Array [
+    Object {
+      "type": "TASKS_FORCE_CANCEL_FAILURE",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "some-name Task execution cannot be cancelled with force at the moment.",
+          "type": "warning",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;
+
+exports[`Tasks actions should unlockTaskRequest and succeed 1`] = `
+Array [
+  Array [
+    Object {
+      "type": "TASKS_UNLOCK_REQUEST",
+    },
+  ],
+  Array [
+    Object {
+      "type": "TASKS_UNLOCK_SUCCESS",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "some-name Task execution resources were unlocked ",
+          "type": "success",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;