foreman-tasks 1.0.0 → 2.0.0

data/webpack/ForemanTasks/Components/TaskDetails/Components/TaskInfo.js

@@ -2,8 +2,8 @@ import React, { Component } from 'react';
 import PropTypes from 'prop-types';
 import { Grid, Row, Col, ProgressBar } from 'patternfly-react';
 import { translate as __ } from 'foremanReact/common/I18n';
-import EllipsisWithTooltip from 'react-ellipsis-with-tooltip';
 import RelativeDateTime from 'foremanReact/components/common/dates/RelativeDateTime';
+import ReactHtmlParser from 'react-html-parser';
 
 class TaskInfo extends Component {
   isDelayed = () => {
@@ -69,9 +69,7 @@ class TaskInfo extends Component {
       [
         {
           title: 'Name',
-          value: (
-            <EllipsisWithTooltip>{action || __('N/A')}</EllipsisWithTooltip>
-          ),
+          value: action || __('N/A'),
         },
         {
           title: 'Start at',
@@ -173,14 +171,7 @@ class TaskInfo extends Component {
                   <b>{__('Troubleshooting')}</b>
                 </span>
               </p>
-              <p>
-                {help.split('\n').map((item, i) => (
-                  <React.Fragment key={i}>
-                    {item}
-                    <br />
-                  </React.Fragment>
-                ))}
-              </p>
+              <p>{ReactHtmlParser(help)}</p>
             </Col>
           </Row>
         )}

data/webpack/ForemanTasks/Components/TaskDetails/Components/__tests__/Errors.test.js

@@ -1,4 +1,4 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 
 import Errors from '../Errors';
 

data/webpack/ForemanTasks/Components/TaskDetails/Components/__tests__/Locks.test.js

@@ -1,4 +1,4 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 
 import Locks from '../Locks';
 

data/webpack/ForemanTasks/Components/TaskDetails/Components/__tests__/Raw.test.js

@@ -1,4 +1,4 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 
 import Raw from '../Raw';
 

data/webpack/ForemanTasks/Components/TaskDetails/Components/__tests__/RunningSteps.test.js

@@ -1,4 +1,4 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 
 import RunningSteps from '../RunningSteps';
 

data/webpack/ForemanTasks/Components/TaskDetails/Components/__tests__/Task.test.js

@@ -1,6 +1,15 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
-
+import React from 'react';
+import {
+  testComponentSnapshotsWithFixtures,
+  mount,
+  shallow,
+} from '@theforeman/test';
+import { useForemanModal } from 'foremanReact/components/ForemanModal/ForemanModalHooks';
 import Task from '../Task';
+import {
+  UNLOCK_MODAL,
+  FORCE_UNLOCK_MODAL,
+} from '../../../TaskActions/TaskActionsConstants';
 
 const fixtures = {
   'render without Props': { id: 'test' },
@@ -8,11 +17,67 @@ const fixtures = {
     id: 'test',
     state: 'paused',
     hasSubTasks: true,
-    allowDangerousActions: true,
+    dynflowEnableConsole: true,
+    parentTask: 'parent-id',
+    taskReload: true,
   },
 };
 
 describe('Task', () => {
   describe('rendering', () =>
     testComponentSnapshotsWithFixtures(Task, fixtures));
+  describe('click test', () => {
+    const setModalOpen = jest.fn();
+    useForemanModal.mockImplementation(id => ({
+      setModalOpen: () => setModalOpen(id),
+    }));
+    const cancelTaskRequest = jest.fn();
+    const resumeTaskRequest = jest.fn();
+    const taskReloadStart = jest.fn();
+    const id = 'some-id';
+    const action = 'some-action';
+    const props = {
+      taskReload: false,
+      taskReloadStart,
+      id,
+      action,
+      cancelTaskRequest,
+      resumeTaskRequest,
+    };
+    afterEach(() => {
+      jest.clearAllMocks();
+    });
+    it('reload', () => {
+      const component = mount(<Task {...props} />);
+      const reloadButton = component.find('.reload-button').at(0);
+      reloadButton.simulate('click');
+      expect(taskReloadStart).toBeCalled();
+    });
+    it('resume', () => {
+      const component = shallow(<Task {...props} />);
+      const resumeButton = component.find('.resume-button').at(0);
+      resumeButton.props().onClick();
+      expect(taskReloadStart).toBeCalled();
+      expect(resumeTaskRequest).toBeCalledWith(id, action);
+    });
+    it('cancel', () => {
+      const component = shallow(<Task {...props} />);
+      const cancelButton = component.find('.cancel-button').at(0);
+      cancelButton.props().onClick();
+      expect(taskReloadStart).toBeCalled();
+      expect(cancelTaskRequest).toBeCalledWith(id, action);
+    });
+    it('unlock', () => {
+      const component = shallow(<Task {...props} />);
+      const unlockButton = component.find('.unlock-button').at(0);
+      unlockButton.props().onClick();
+      expect(setModalOpen).toBeCalledWith({ id: UNLOCK_MODAL });
+    });
+    it('focrce unlock', () => {
+      const component = shallow(<Task {...props} />);
+      const forceUnlockButton = component.find('.force-unlock-button').at(0);
+      forceUnlockButton.props().onClick();
+      expect(setModalOpen).toBeCalledWith({ id: FORCE_UNLOCK_MODAL });
+    });
+  });
 });

data/webpack/ForemanTasks/Components/TaskDetails/Components/__tests__/TaskInfo.test.js

@@ -1,4 +1,4 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 
 import TaskInfo from '../TaskInfo';
 
@@ -41,7 +41,6 @@ const fixtures = {
     help:
       "A paused task represents a process that has not finished properly. Any task in paused state can lead to potential inconsistency and needs to be resolved.\nThe recommended approach is to investigate the error messages below and in 'errors' tab, address the primary cause of the issue and resume the task.",
     hasSubTasks: false,
-    allowDangerousActions: false,
     locks: [
       {
         name: 'task_owner',

data/webpack/ForemanTasks/Components/TaskDetails/Components/__tests__/__snapshots__/Task.test.js.snap

@@ -2,25 +2,14 @@
 
 exports[`Task rendering render with some Props 1`] = `
 <Fragment>
-  <ClickConfirmation
-    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."
-    closeModal={[Function]}
-    confirmAction="Unlock"
-    confirmType="warning"
-    confirmationMessage="I understand that this may cause harm and have working database backups of all backend services."
-    path="/foreman_tasks/tasks/test/unlock"
-    showModal={false}
-    title="Unlock"
+  <UnlockModal
+    id="unlockModal"
+    onClick={[Function]}
   />
-  <ClickConfirmation
-    body="Resources will be unlocked and will not prevent other tasks from being run. As the task might be still running, it should be avoided to use this unless you are really sure the task got stuck"
-    closeModal={[Function]}
-    confirmAction="Force Unlock"
-    confirmType="danger"
-    confirmationMessage="I understand that this may cause harm and have working database backups of all backend services."
-    path="/foreman_tasks/tasks/test/force_unlock"
-    showModal={false}
-    title="Force Unlock"
+  <ForceUnlockModal
+    id="forceUnlockModal"
+    onClick={[Function]}
+    selectedRowsLen={1}
   />
   <Grid
     bsClass="container"
@@ -44,13 +33,12 @@ exports[`Task rendering render with some Props 1`] = `
           bsStyle="default"
           className="reload-button"
           disabled={false}
-          hidden={false}
           onClick={[Function]}
         >
           <span
-            className="glyphicon glyphicon-refresh "
+            className="glyphicon glyphicon-refresh spin"
           />
-          Start  auto-reloading
+          Stop  auto-reloading
         </Button>
         <Button
           active={false}
@@ -58,29 +46,55 @@ exports[`Task rendering render with some Props 1`] = `
           bsClass="btn"
           bsSize="small"
           bsStyle="default"
+          className="dynflow-button"
           disabled={false}
           href="/foreman_tasks/dynflow/"
         >
           Dynflow console
         </Button>
-        <ResumeButton
+        <Button
+          active={false}
+          block={false}
+          bsClass="btn"
+          bsSize="small"
+          bsStyle="default"
+          className="resume-button"
           disabled={true}
-          id="test"
-          name=""
           onClick={[Function]}
-        />
-        <CancelButton
+        >
+          Resume
+        </Button>
+        <Button
+          active={false}
+          block={false}
+          bsClass="btn"
+          bsSize="small"
+          bsStyle="default"
+          className="cancel-button"
           disabled={true}
-          id="test"
-          name=""
           onClick={[Function]}
-        />
+        >
+          Cancel
+        </Button>
         <Button
           active={false}
           block={false}
           bsClass="btn"
           bsSize="small"
           bsStyle="default"
+          className="parent-button"
+          disabled={false}
+          href="/foreman_tasks/tasks/parent-id"
+        >
+          Parent task
+        </Button>
+        <Button
+          active={false}
+          block={false}
+          bsClass="btn"
+          bsSize="small"
+          bsStyle="default"
+          className="subtask-button"
           disabled={false}
           href="/foreman_tasks/tasks/test/sub_tasks"
         >
@@ -92,8 +106,9 @@ exports[`Task rendering render with some Props 1`] = `
           bsClass="btn"
           bsSize="small"
           bsStyle="default"
+          className="unlock-button"
           disabled={false}
-          onClick={[Function]}
+          onClick={[MockFunction]}
         >
           Unlock
         </Button>
@@ -103,8 +118,9 @@ exports[`Task rendering render with some Props 1`] = `
           bsClass="btn"
           bsSize="small"
           bsStyle="default"
+          className="force-unlock-button"
           disabled={false}
-          onClick={[Function]}
+          onClick={[MockFunction]}
         >
           Force Unlock
         </Button>
@@ -112,9 +128,9 @@ exports[`Task rendering render with some Props 1`] = `
     </Row>
     <TaskInfo
       action=""
-      allowDangerousActions={true}
       cancelTaskRequest={[Function]}
       cancellable={false}
+      dynflowEnableConsole={true}
       endedAt=""
       error={Array []}
       externalId=""
@@ -122,24 +138,20 @@ exports[`Task rendering render with some Props 1`] = `
       help=""
       id="test"
       output=""
-      parentTask=""
+      parentTask="parent-id"
       progress={0}
       refetchTaskDetails={[Function]}
       result="error"
       resumable={false}
       resumeTaskRequest={[Function]}
-      showForceUnlockModal={false}
-      showUnlockModal={false}
       startAt=""
       startBefore=""
       startedAt=""
       state="paused"
-      taskReload={false}
+      taskReload={true}
       taskReloadStart={[Function]}
       taskReloadStop={[Function]}
       timeoutId={null}
-      toggleForceUnlockModal={[Function]}
-      toggleUnlockModal={[Function]}
       username=""
       usernamePath=""
     />
@@ -149,25 +161,14 @@ exports[`Task rendering render with some Props 1`] = `
 
 exports[`Task rendering render without Props 1`] = `
 <Fragment>
-  <ClickConfirmation
-    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."
-    closeModal={[Function]}
-    confirmAction="Unlock"
-    confirmType="warning"
-    confirmationMessage="I understand that this may cause harm and have working database backups of all backend services."
-    path="/foreman_tasks/tasks/test/unlock"
-    showModal={false}
-    title="Unlock"
+  <UnlockModal
+    id="unlockModal"
+    onClick={[Function]}
   />
-  <ClickConfirmation
-    body="Resources will be unlocked and will not prevent other tasks from being run. As the task might be still running, it should be avoided to use this unless you are really sure the task got stuck"
-    closeModal={[Function]}
-    confirmAction="Force Unlock"
-    confirmType="danger"
-    confirmationMessage="I understand that this may cause harm and have working database backups of all backend services."
-    path="/foreman_tasks/tasks/test/force_unlock"
-    showModal={false}
-    title="Force Unlock"
+  <ForceUnlockModal
+    id="forceUnlockModal"
+    onClick={[Function]}
+    selectedRowsLen={1}
   />
   <Grid
     bsClass="container"
@@ -191,7 +192,6 @@ exports[`Task rendering render without Props 1`] = `
           bsStyle="default"
           className="reload-button"
           disabled={false}
-          hidden={true}
           onClick={[Function]}
         >
           <span
@@ -205,30 +205,67 @@ exports[`Task rendering render without Props 1`] = `
           bsClass="btn"
           bsSize="small"
           bsStyle="default"
-          disabled={false}
+          className="dynflow-button"
+          disabled={true}
           href="/foreman_tasks/dynflow/"
         >
           Dynflow console
         </Button>
-        <ResumeButton
+        <Button
+          active={false}
+          block={false}
+          bsClass="btn"
+          bsSize="small"
+          bsStyle="default"
+          className="resume-button"
           disabled={true}
-          id="test"
-          name=""
           onClick={[Function]}
-        />
-        <CancelButton
+        >
+          Resume
+        </Button>
+        <Button
+          active={false}
+          block={false}
+          bsClass="btn"
+          bsSize="small"
+          bsStyle="default"
+          className="cancel-button"
           disabled={true}
-          id="test"
-          name=""
           onClick={[Function]}
-        />
+        >
+          Cancel
+        </Button>
+        <Button
+          active={false}
+          block={false}
+          bsClass="btn"
+          bsSize="small"
+          bsStyle="default"
+          className="unlock-button"
+          disabled={true}
+          onClick={[MockFunction]}
+        >
+          Unlock
+        </Button>
+        <Button
+          active={false}
+          block={false}
+          bsClass="btn"
+          bsSize="small"
+          bsStyle="default"
+          className="force-unlock-button"
+          disabled={false}
+          onClick={[MockFunction]}
+        >
+          Force Unlock
+        </Button>
       </Col>
     </Row>
     <TaskInfo
       action=""
-      allowDangerousActions={false}
       cancelTaskRequest={[Function]}
       cancellable={false}
+      dynflowEnableConsole={false}
       endedAt=""
       error={Array []}
       externalId=""
@@ -242,8 +279,6 @@ exports[`Task rendering render without Props 1`] = `
       result="error"
       resumable={false}
       resumeTaskRequest={[Function]}
-      showForceUnlockModal={false}
-      showUnlockModal={false}
       startAt=""
       startBefore=""
       startedAt=""
@@ -252,8 +287,6 @@ exports[`Task rendering render without Props 1`] = `
       taskReloadStart={[Function]}
       taskReloadStop={[Function]}
       timeoutId={null}
-      toggleForceUnlockModal={[Function]}
-      toggleUnlockModal={[Function]}
       username=""
       usernamePath=""
     />