foreman_rh_cloud 1.0.10 → 1.0.14

data/webpack/ForemanInventoryUpload/Components/AccountList/AccountListReducer.js

@@ -1,6 +1,8 @@
 import Immutable from 'seamless-immutable';
 import { AUTO_UPLOAD_TOGGLE } from '../AutoUploadSwitcher/AutoUploadSwitcherConstants';
 import { HOST_OBFUSCATION_TOGGLE } from '../HostObfuscationSwitcher/HostObfuscationSwitcherConstants';
+import { IPS_OBFUSCATION_TOGGLE } from '../IpsObfuscationSwitcher/IpsObfuscationSwitcherConstants';
+import { EXCLUDE_PACKAGES_TOGGLE } from '../ExcludePackagesSwitcher/ExcludePackagesSwitcherConstants';
 import {
   INVENTORY_ACCOUNT_STATUS_POLLING,
   INVENTORY_ACCOUNT_STATUS_POLLING_ERROR,
@@ -24,7 +26,9 @@ export default (state = initialState, action) => {
       processStatusName,
       autoUploadEnabled,
       hostObfuscationEnabled,
+      ipsObfuscationEnabled,
       cloudToken,
+      excludePackages,
     } = {},
   } = action;
 
@@ -35,7 +39,9 @@ export default (state = initialState, action) => {
         accounts,
         autoUploadEnabled,
         hostObfuscationEnabled,
+        ipsObfuscationEnabled,
         cloudToken,
+        excludePackages,
         error: null,
       });
     case INVENTORY_ACCOUNT_STATUS_POLLING_ERROR:
@@ -67,6 +73,16 @@ export default (state = initialState, action) => {
         ...state,
         hostObfuscationEnabled,
       });
+    case EXCLUDE_PACKAGES_TOGGLE:
+      return state.merge({
+        ...state,
+        excludePackages,
+      });
+    case IPS_OBFUSCATION_TOGGLE:
+      return state.merge({
+        ...state,
+        ipsObfuscationEnabled,
+      });
     default:
       return state;
   }

data/webpack/ForemanInventoryUpload/Components/AccountList/AccountListSelectors.js

@@ -16,4 +16,10 @@ export const selectAutoUploadEnabled = state =>
 export const selectHostObfuscationEnabled = state =>
   selectAccountsList(state).hostObfuscationEnabled;
 
+export const selectIpsObfuscationEnabled = state =>
+  selectAccountsList(state).ipsObfuscationEnabled;
+
 export const selectCloudToken = state => selectAccountsList(state).cloudToken;
+
+export const selectExcludePackages = state =>
+  selectAccountsList(state).excludePackages;

data/webpack/ForemanInventoryUpload/Components/AccountList/__tests__/AccountListActions.test.js

@@ -22,6 +22,13 @@ const fixtures = {
   'should stopAccountStatusPolling': () =>
     stopAccountStatusPolling(pollingProcessID),
   'should restartProcess': () => restartProcess(accountID, activeTab),
+  'should invoke toast notification upon failure': () => {
+    API.post.mockImplementationOnce(() =>
+      Promise.reject(new Error('test error'))
+    );
+
+    return restartProcess(accountID, activeTab);
+  },
 };
 
 describe('AccountList actions', () => testActionSnapshotWithFixtures(fixtures));

data/webpack/ForemanInventoryUpload/Components/AccountList/__tests__/AccountListReducer.test.js

@@ -9,29 +9,25 @@ import {
 } from '../AccountListConstants';
 import reducer from '../AccountListReducer';
 import {
-  accounts,
   error,
   pollingProcessID,
   accountID,
   processStatusName,
   autoUploadEnabled,
   hostObfuscationEnabled,
-  cloudToken,
+  ipsObfuscationEnabled,
+  pollingResponse,
 } from '../AccountList.fixtures';
 import { AUTO_UPLOAD_TOGGLE } from '../../AutoUploadSwitcher/AutoUploadSwitcherConstants';
 import { HOST_OBFUSCATION_TOGGLE } from '../../HostObfuscationSwitcher/HostObfuscationSwitcherConstants';
+import { IPS_OBFUSCATION_TOGGLE } from '../../IpsObfuscationSwitcher/IpsObfuscationSwitcherConstants';
 
 const fixtures = {
   'should return the initial state': {},
   'should handle INVENTORY_ACCOUNT_STATUS_POLLING': {
     action: {
       type: INVENTORY_ACCOUNT_STATUS_POLLING,
-      payload: {
-        accounts,
-        autoUploadEnabled,
-        hostObfuscationEnabled,
-        cloudToken,
-      },
+      payload: pollingResponse,
     },
   },
   'should handle INVENTORY_ACCOUNT_STATUS_POLLING_ERROR': {
@@ -78,6 +74,14 @@ const fixtures = {
       },
     },
   },
+  'should handle IPS_OBFUSCATION_TOGGLE': {
+    action: {
+      type: IPS_OBFUSCATION_TOGGLE,
+      payload: {
+        ipsObfuscationEnabled,
+      },
+    },
+  },
 };
 
 describe('AccountList reducer', () =>

data/webpack/ForemanInventoryUpload/Components/AccountList/__tests__/AccountListSelectors.test.js

@@ -5,12 +5,16 @@ import {
   selectPollingProcessID,
   selectAutoUploadEnabled,
   selectHostObfuscationEnabled,
+  selectCloudToken,
+  selectExcludePackages,
 } from '../AccountListSelectors';
 import {
   pollingProcessID,
   accounts,
   autoUploadEnabled,
   hostObfuscationEnabled,
+  cloudToken,
+  excludePackages,
 } from '../AccountList.fixtures';
 import { rhCloudStateWrapper } from '../../../../ForemanRhCloudTestHelpers';
 
@@ -20,6 +24,8 @@ const state = rhCloudStateWrapper({
     pollingProcessID,
     autoUploadEnabled,
     hostObfuscationEnabled,
+    cloudToken,
+    excludePackages,
   },
 });
 
@@ -32,6 +38,9 @@ const fixtures = {
     selectAutoUploadEnabled(state),
   'should return AccountList hostObfuscationEnabled': () =>
     selectHostObfuscationEnabled(state),
+  'should return AccountList cloudToken': () => selectCloudToken(state),
+  'should return AccountList excludePackages': () =>
+    selectExcludePackages(state),
 };
 
 describe('AccountList selectors', () =>

data/webpack/ForemanInventoryUpload/Components/AccountList/__tests__/__snapshots__/AccountListActions.test.js.snap

@@ -24,7 +24,9 @@ Array [
         },
         "autoUploadEnabled": true,
         "cloudToken": "some-cloud-token",
+        "excludePackages": false,
         "hostObfuscationEnabled": true,
+        "ipsObfuscationEnabled": true,
       },
       "type": "INVENTORY_ACCOUNT_STATUS_POLLING",
     },
@@ -32,6 +34,23 @@ Array [
 ]
 `;
 
+exports[`AccountList actions should invoke toast notification upon failure 1`] = `
+Array [
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "test error",
+          "sticky": true,
+          "type": "error",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;
+
 exports[`AccountList actions should restartProcess 1`] = `
 Array [
   Array [

data/webpack/ForemanInventoryUpload/Components/AccountList/__tests__/__snapshots__/AccountListReducer.test.js.snap

@@ -40,7 +40,9 @@ Object {
   "autoUploadEnabled": true,
   "cloudToken": "some-cloud-token",
   "error": null,
+  "excludePackages": false,
   "hostObfuscationEnabled": true,
+  "ipsObfuscationEnabled": true,
   "pollingProcessID": 0,
 }
 `;
@@ -81,6 +83,15 @@ Object {
 }
 `;
 
+exports[`AccountList reducer should handle IPS_OBFUSCATION_TOGGLE 1`] = `
+Object {
+  "accounts": Object {},
+  "error": null,
+  "ipsObfuscationEnabled": true,
+  "pollingProcessID": 0,
+}
+`;
+
 exports[`AccountList reducer should return the initial state 1`] = `
 Object {
   "accounts": Object {},

data/webpack/ForemanInventoryUpload/Components/AccountList/__tests__/__snapshots__/AccountListSelectors.test.js.snap

@@ -22,6 +22,10 @@ Object {
 
 exports[`AccountList selectors should return AccountList autoUploadEnabled 1`] = `true`;
 
+exports[`AccountList selectors should return AccountList cloudToken 1`] = `"some-cloud-token"`;
+
+exports[`AccountList selectors should return AccountList excludePackages 1`] = `false`;
+
 exports[`AccountList selectors should return AccountList hostObfuscationEnabled 1`] = `true`;
 
 exports[`AccountList selectors should return AccountList pollingProcessID 1`] = `0`;
@@ -46,6 +50,8 @@ Object {
     },
   },
   "autoUploadEnabled": true,
+  "cloudToken": "some-cloud-token",
+  "excludePackages": false,
   "hostObfuscationEnabled": true,
   "pollingProcessID": 0,
 }

data/webpack/ForemanInventoryUpload/Components/AutoUploadSwitcher/AutoUploadSwitcher.fixtures.js

@@ -1 +1,3 @@
+export const currentAutoUploadEnabled = false;
+
 export const handleToggleResponse = { data: { autoUploadEnabled: true } };

data/webpack/ForemanInventoryUpload/Components/AutoUploadSwitcher/AutoUploadSwitcherActions.js

@@ -1,9 +1,7 @@
 import API from 'foremanReact/API';
+import { addToast } from 'foremanReact/redux/actions/toasts';
 import { inventoryUrl } from '../../ForemanInventoryHelpers';
-import {
-  AUTO_UPLOAD_TOGGLE,
-  AUTO_UPLOAD_TOGGLE_ERROR,
-} from './AutoUploadSwitcherConstants';
+import { AUTO_UPLOAD_TOGGLE } from './AutoUploadSwitcherConstants';
 
 export const handleToggle = currentAutoUploadEnabled => async dispatch => {
   const toggledAutoUploadEnabled = !currentAutoUploadEnabled;
@@ -19,12 +17,13 @@ export const handleToggle = currentAutoUploadEnabled => async dispatch => {
         autoUploadEnabled,
       },
     });
-  } catch (error) {
-    dispatch({
-      type: AUTO_UPLOAD_TOGGLE_ERROR,
-      payload: {
-        error: error.message,
-      },
-    });
+  } catch ({ message }) {
+    dispatch(
+      addToast({
+        sticky: true,
+        type: 'error',
+        message,
+      })
+    );
   }
 };

data/webpack/ForemanInventoryUpload/Components/AutoUploadSwitcher/AutoUploadSwitcherConstants.js

@@ -1,2 +1 @@
 export const AUTO_UPLOAD_TOGGLE = 'AUTO_UPLOAD_TOGGLE';
-export const AUTO_UPLOAD_TOGGLE_ERROR = 'AUTO_UPLOAD_TOGGLE_ERROR';

data/webpack/ForemanInventoryUpload/Components/AutoUploadSwitcher/__tests__/AutoUploadSwitcherActions.test.js

@@ -1,13 +1,23 @@
 import { testActionSnapshotWithFixtures } from 'react-redux-test-utils';
 import API from 'foremanReact/API';
 import { handleToggle } from '../AutoUploadSwitcherActions';
-import { handleToggleResponse } from '../AutoUploadSwitcher.fixtures';
+import {
+  handleToggleResponse,
+  currentAutoUploadEnabled,
+} from '../AutoUploadSwitcher.fixtures';
 
 jest.mock('foremanReact/API');
 API.post.mockImplementation(async () => handleToggleResponse);
 
 const fixtures = {
-  'should handleToggle': () => handleToggle(),
+  'should handleToggle': () => handleToggle(currentAutoUploadEnabled),
+  'should handleToggle with error': () => {
+    API.post.mockImplementationOnce(() =>
+      Promise.reject(new Error('Network error!'))
+    );
+
+    return handleToggle(currentAutoUploadEnabled);
+  },
 };
 
 describe('AutoUploadSwitcher actions', () =>

data/webpack/ForemanInventoryUpload/Components/AutoUploadSwitcher/__tests__/__snapshots__/AutoUploadSwitcherActions.test.js.snap

@@ -12,3 +12,20 @@ Array [
   ],
 ]
 `;
+
+exports[`AutoUploadSwitcher actions should handleToggle with error 1`] = `
+Array [
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "Network error!",
+          "sticky": true,
+          "type": "error",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;

data/webpack/ForemanInventoryUpload/Components/ExcludePackagesSwitcher/ExcludePackagesSwitcher.fixtures.js

@@ -0,0 +1 @@
+export const handleToggleResponse = { data: { excludePackages: true } };

data/webpack/ForemanInventoryUpload/Components/ExcludePackagesSwitcher/ExcludePackagesSwitcher.js

@@ -0,0 +1,30 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { Switch, FieldLevelHelp } from 'patternfly-react';
+import { translate as __ } from 'foremanReact/common/I18n';
+import './excludePackagesSwitcher.scss';
+
+const ExcludePackagesSwitcher = ({ excludePackages, handleToggle }) => (
+  <div className="exclude_packages_switcher">
+    <span>Exclude Packages</span>
+    <FieldLevelHelp
+      content={__('Exclude packages from being uploaded to the Red Hat cloud')}
+    />
+    <Switch
+      size="mini"
+      value={excludePackages}
+      onChange={() => handleToggle(excludePackages)}
+    />
+  </div>
+);
+
+ExcludePackagesSwitcher.propTypes = {
+  excludePackages: PropTypes.bool,
+  handleToggle: PropTypes.func.isRequired,
+};
+
+ExcludePackagesSwitcher.defaultProps = {
+  excludePackages: false,
+};
+
+export default ExcludePackagesSwitcher;

data/webpack/ForemanInventoryUpload/Components/ExcludePackagesSwitcher/ExcludePackagesSwitcherActions.js

@@ -0,0 +1,29 @@
+import API from 'foremanReact/API';
+import { addToast } from 'foremanReact/redux/actions/toasts';
+import { inventoryUrl } from '../../ForemanInventoryHelpers';
+import { EXCLUDE_PACKAGES_TOGGLE } from './ExcludePackagesSwitcherConstants';
+
+export const handleToggle = currentExcludePackages => async dispatch => {
+  const toggledExcludePackages = !currentExcludePackages;
+  try {
+    const {
+      data: { excludePackages },
+    } = await API.post(inventoryUrl('installed_packages_inclusion'), {
+      value: toggledExcludePackages,
+    });
+    dispatch({
+      type: EXCLUDE_PACKAGES_TOGGLE,
+      payload: {
+        excludePackages,
+      },
+    });
+  } catch (error) {
+    dispatch(
+      addToast({
+        sticky: true,
+        type: 'error',
+        message: error.message,
+      })
+    );
+  }
+};

data/webpack/ForemanInventoryUpload/Components/ExcludePackagesSwitcher/ExcludePackagesSwitcherConstants.js

@@ -0,0 +1 @@
+export const EXCLUDE_PACKAGES_TOGGLE = 'EXCLUDE_PACKAGES_TOGGLE';

data/webpack/ForemanInventoryUpload/Components/ExcludePackagesSwitcher/__tests__/ExcludePackagesSwitcher.test.js

@@ -0,0 +1,13 @@
+import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { noop } from 'patternfly-react';
+
+import ExcludePackagesSwitcher from '../ExcludePackagesSwitcher';
+
+const fixtures = {
+  'render with props': { excludePackages: true, handleToggle: noop },
+};
+
+describe('ExcludePackagesSwitcher', () => {
+  describe('rendering', () =>
+    testComponentSnapshotsWithFixtures(ExcludePackagesSwitcher, fixtures));
+});

data/webpack/ForemanInventoryUpload/Components/ExcludePackagesSwitcher/__tests__/ExcludePackagesSwitcherActions.test.js

@@ -0,0 +1,21 @@
+import { testActionSnapshotWithFixtures } from 'react-redux-test-utils';
+import API from 'foremanReact/API';
+import { handleToggle } from '../ExcludePackagesSwitcherActions';
+import { handleToggleResponse } from '../ExcludePackagesSwitcher.fixtures';
+
+jest.mock('foremanReact/API');
+API.post.mockImplementation(async () => handleToggleResponse);
+
+const fixtures = {
+  'should handleToggle': () => handleToggle(),
+  'should invoke toast notification upon failure of handleToggle': () => {
+    API.post.mockImplementationOnce(() =>
+      Promise.reject(new Error('test error'))
+    );
+
+    return handleToggle();
+  },
+};
+
+describe('ExcludePackagesSwitcher actions', () =>
+  testActionSnapshotWithFixtures(fixtures));

data/webpack/ForemanInventoryUpload/Components/ExcludePackagesSwitcher/__tests__/__snapshots__/ExcludePackagesSwitcher.test.js.snap

@@ -0,0 +1,38 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`ExcludePackagesSwitcher rendering render with props 1`] = `
+<div
+  className="exclude_packages_switcher"
+>
+  <span>
+    Exclude Packages
+  </span>
+  <FieldLevelHelp
+    buttonClass=""
+    content="Exclude packages from being uploaded to the Red Hat cloud"
+    placement="top"
+    rootClose={true}
+  />
+  <Switch
+    animate={true}
+    baseClass="bootstrap-switch"
+    bsSize={null}
+    defaultValue={true}
+    disabled={false}
+    handleWidth="auto"
+    inverse={false}
+    labelText=" "
+    labelWidth="auto"
+    offColor="default"
+    offText="OFF"
+    onChange={[Function]}
+    onColor="primary"
+    onText="ON"
+    readonly={false}
+    size="mini"
+    tristate={false}
+    value={true}
+    wrapperClass="wrapper"
+  />
+</div>
+`;