foreman_rh_cloud 1.0.10 → 1.0.14

data/webpack/ForemanInventoryUpload/Components/IpsObfuscationSwitcher/index.js

@@ -0,0 +1,20 @@
+import { bindActionCreators } from 'redux';
+import { connect } from 'react-redux';
+
+import * as actions from './IpsObfuscationSwitcherActions';
+import IpsObfuscationSwitcher from './IpsObfuscationSwitcher';
+import { selectIpsObfuscationEnabled } from '../AccountList/AccountListSelectors';
+
+// map state to props
+const mapStateToProps = state => ({
+  ipsObfuscationEnabled: selectIpsObfuscationEnabled(state),
+});
+
+// map action dispatchers to props
+const mapDispatchToProps = dispatch => bindActionCreators(actions, dispatch);
+
+// export connected component
+export default connect(
+  mapStateToProps,
+  mapDispatchToProps
+)(IpsObfuscationSwitcher);

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/SyncButton.js

@@ -21,7 +21,12 @@ class SyncButton extends React.Component {
     return (
       <React.Fragment>
         <SyncModal show={this.state.showModal} toggleModal={this.toggleModal} />
-        <Button className="sync_button" onClick={handleClick} bsSize="lg">
+        <Button
+          className="sync_button"
+          onClick={handleClick}
+          bsSize="lg"
+          disabled={status === STATUS.PENDING}
+        >
           {!cloudToken && (
             <span>
               <Icon name="warning" />{' '}
@@ -40,12 +45,13 @@ class SyncButton extends React.Component {
 }
 
 SyncButton.propTypes = {
-  cloudToken: PropTypes.string.isRequired,
+  cloudToken: PropTypes.string,
   handleSync: PropTypes.func.isRequired,
   status: PropTypes.string,
 };
 
 SyncButton.defaultProps = {
+  cloudToken: null,
   status: null,
 };
 

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/SyncButtonActions.js

@@ -6,7 +6,7 @@ import Toast from './components/Toast';
 import {
   INVENTORY_SYNC_REQUEST,
   INVENTORY_SYNC_SUCCESS,
-  INVENTORY_SYNC_FAILURE,
+  INVENTORY_SYNC_ERROR,
 } from './SyncButtonConstants';
 
 export const handleSync = () => async dispatch => {
@@ -40,20 +40,18 @@ export const handleSync = () => async dispatch => {
     response: { data: { message: toastMessage } = {} } = {},
   }) {
     dispatch({
-      type: INVENTORY_SYNC_FAILURE,
+      type: INVENTORY_SYNC_ERROR,
       payload: {
         error: message,
       },
     });
 
-    if (toastMessage) {
-      dispatch(
-        addToast({
-          sticky: true,
-          type: 'error',
-          message: toastMessage,
-        })
-      );
-    }
+    dispatch(
+      addToast({
+        sticky: true,
+        type: 'error',
+        message: toastMessage || message,
+      })
+    );
   }
 };

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/SyncButtonConstants.js

@@ -1,3 +1,3 @@
 export const INVENTORY_SYNC_REQUEST = 'INVENTORY_SYNC_REQUEST';
 export const INVENTORY_SYNC_SUCCESS = 'INVENTORY_SYNC_SUCCESS';
-export const INVENTORY_SYNC_FAILURE = 'INVENTORY_SYNC_FAILURE';
+export const INVENTORY_SYNC_ERROR = 'INVENTORY_SYNC_ERROR';

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/SyncButtonReducer.js

@@ -3,7 +3,7 @@ import { STATUS } from 'foremanReact/constants';
 import {
   INVENTORY_SYNC_REQUEST,
   INVENTORY_SYNC_SUCCESS,
-  INVENTORY_SYNC_FAILURE,
+  INVENTORY_SYNC_ERROR,
 } from './SyncButtonConstants';
 
 export default (
@@ -24,7 +24,7 @@ export default (
         syncHosts,
         disconnectHosts,
       });
-    case INVENTORY_SYNC_FAILURE:
+    case INVENTORY_SYNC_ERROR:
       return state.merge({
         ...state,
         status: STATUS.ERROR,

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/__tests__/SyncButtonActions.test.js

@@ -8,6 +8,24 @@ API.post.mockImplementation(async () => successResponse);
 
 const fixtures = {
   'should handleSync': () => handleSync(),
+  'should handleSync with error message': () => {
+    API.post.mockImplementationOnce(() =>
+      Promise.reject(new Error('Network error!'))
+    );
+
+    return handleSync();
+  },
+  'should handleSync with custom error message': () => {
+    API.post.mockImplementationOnce(() => {
+      const customError = new Error('Server error!');
+      customError.response = {
+        data: { message: 'Custom error to display in a toast' },
+      };
+      return Promise.reject(customError);
+    });
+
+    return handleSync();
+  },
 };
 
 describe('SyncButton actions', () => testActionSnapshotWithFixtures(fixtures));

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/__tests__/SyncButtonReducer.test.js

@@ -1,10 +1,9 @@
 import { testReducerSnapshotWithFixtures } from 'react-redux-test-utils';
 import reducer from '../SyncButtonReducer';
-import { syncHosts, disconnectHosts, error } from './SyncButtonFixtures';
+import { syncHosts, disconnectHosts } from './SyncButtonFixtures';
 import {
   INVENTORY_SYNC_REQUEST,
   INVENTORY_SYNC_SUCCESS,
-  INVENTORY_SYNC_FAILURE,
 } from '../SyncButtonConstants';
 
 const fixtures = {
@@ -21,12 +20,6 @@ const fixtures = {
       payload: { syncHosts, disconnectHosts },
     },
   },
-  'should handle INVENTORY_SYNC_FAILURE': {
-    action: {
-      type: INVENTORY_SYNC_FAILURE,
-      payload: { error },
-    },
-  },
 };
 
 describe('AccountList reducer', () =>

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/__tests__/__snapshots__/SyncButtonActions.test.js.snap

@@ -34,3 +34,65 @@ Array [
   ],
 ]
 `;
+
+exports[`SyncButton actions should handleSync with custom error message 1`] = `
+Array [
+  Array [
+    Object {
+      "payload": Object {},
+      "type": "INVENTORY_SYNC_REQUEST",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "error": "Server error!",
+      },
+      "type": "INVENTORY_SYNC_ERROR",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "Custom error to display in a toast",
+          "sticky": true,
+          "type": "error",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;
+
+exports[`SyncButton actions should handleSync with error message 1`] = `
+Array [
+  Array [
+    Object {
+      "payload": Object {},
+      "type": "INVENTORY_SYNC_REQUEST",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "error": "Network error!",
+      },
+      "type": "INVENTORY_SYNC_ERROR",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "Network error!",
+          "sticky": true,
+          "type": "error",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/__tests__/__snapshots__/SyncButtonReducer.test.js.snap

@@ -1,12 +1,5 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`AccountList reducer should handle INVENTORY_SYNC_FAILURE 1`] = `
-Object {
-  "error": "some-error",
-  "status": "ERROR",
-}
-`;
-
 exports[`AccountList reducer should handle INVENTORY_SYNC_REQUEST 1`] = `
 Object {
   "error": null,

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/components/Modal.js

@@ -21,10 +21,10 @@ const SyncModal = ({ show, toggleModal }) => (
     <Modal.Body>
       <Grid>
         <p>
-          {__(`Please go over the following steps to add a RHSM API token:`)}
+          {__(`Please go over the following steps to add a Red Hat API token:`)}
         </p>
         <p>
-          {__(`1. Obtain an RHSM API token: `)}
+          {__(`1. Obtain an Red Hat API token: `)}
           <a
             href="https://access.redhat.com/management/api"
             target="_blank"

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/components/Toast.js

@@ -8,7 +8,7 @@ const Toast = ({ syncHosts, disconnectHosts }) => {
   return (
     <span>
       <p>
-        {__("Total org's hosts with subscriprion: ")}
+        {__('Hosts with subscription in organization: ')}
         <strong>{totalHosts}</strong>
       </p>
       <p>

data/webpack/ForemanInventoryUpload/Components/StatusChart/StatusChart.js

@@ -27,7 +27,7 @@ const StatusChart = ({ completed }) => {
     <Grid.Col sm={4}>
       <div className="status-chart">
         <DonutChart
-          id="donunt-chart-1"
+          id="donut-chart-1"
           size={{
             width: 210,
             height: 210,

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

@@ -43,7 +43,7 @@ exports[`StatusChart rendering render without Props 1`] = `
           "width": 11,
         }
       }
-      id="donunt-chart-1"
+      id="donut-chart-1"
       legend={
         Object {
           "show": false,

data/webpack/ForemanInventoryUpload/Components/TabContainer/tabContainer.scss

@@ -1,4 +1,4 @@
-.tab-content {
+.dashboard .tab-content {
   padding: 20px;
   min-height: 0;
 

data/webpack/ForemanInventoryUpload/Components/TabHeader/TabHeader.js

@@ -2,6 +2,7 @@ import React from 'react';
 import PropTypes from 'prop-types';
 import { noop, Grid, Button, Icon } from 'patternfly-react';
 import { sprintf, translate as __ } from 'foremanReact/common/I18n';
+import { isExitCodeLoading } from '../../ForemanInventoryHelpers';
 import './tabHeader.scss';
 
 const TabHeader = ({ exitCode, onRestart, onDownload, toggleFullScreen }) => (
@@ -12,7 +13,11 @@ const TabHeader = ({ exitCode, onRestart, onDownload, toggleFullScreen }) => (
     <Grid.Col sm={6}>
       <div className="tab-action-buttons">
         {onRestart ? (
-          <Button bsStyle="primary" onClick={onRestart}>
+          <Button
+            bsStyle="primary"
+            onClick={onRestart}
+            disabled={isExitCodeLoading(exitCode)}
+          >
             {__('Restart')}
           </Button>
         ) : null}

data/webpack/ForemanInventoryUpload/Components/Terminal/Terminal.js

@@ -4,6 +4,7 @@ import { Grid, Spinner } from 'patternfly-react';
 import isEqual from 'lodash/isEqual';
 import './terminal.scss';
 import { isTerminalScrolledDown } from './TerminalHelper';
+import { isExitCodeLoading } from '../../ForemanInventoryHelpers';
 
 class Terminal extends React.Component {
   constructor(props) {
@@ -67,10 +68,7 @@ class Terminal extends React.Component {
     } else {
       modifiedLogs = <p>{logs}</p>;
     }
-    const exitCodeLowerCase = exitCode.toLowerCase();
-    const loading =
-      exitCodeLowerCase.indexOf('running') !== -1 ||
-      exitCodeLowerCase.indexOf('restarting') !== -1;
+    const loading = isExitCodeLoading(exitCode);
     return (
       <Grid.Col sm={12}>
         <div

data/webpack/ForemanInventoryUpload/ForemanInventoryHelpers.js

@@ -10,3 +10,11 @@ export const getActionsHistoryUrl = () =>
   foremanUrl(
     '/foreman_tasks/tasks?search=action++%3D++ForemanInventoryUpload%3A%3AAsync%3A%3AGenerateReportJob+or+action++%3D++ForemanInventoryUpload%3A%3AAsync%3A%3AGenerateAllReportsJob&page=1'
   );
+
+export const isExitCodeLoading = exitCode => {
+  const exitCodeLC = exitCode.toLowerCase();
+  return (
+    exitCodeLC.indexOf('running') !== -1 ||
+    exitCodeLC.indexOf('restarting') !== -1
+  );
+};

data/webpack/ForemanRhCloudPages.js

@@ -1,7 +1,9 @@
 import ForemanInventoryUpload from './ForemanInventoryUpload';
 import InsightsCloudSync from './InsightsCloudSync';
+import InsightsHostDetailsTab from './InsightsHostDetailsTab';
 
 export default [
   { name: 'ForemanInventoryUpload', type: ForemanInventoryUpload },
   { name: 'InsightsCloudSync', type: InsightsCloudSync },
+  { name: 'InsightsHostDetailsTab', type: InsightsHostDetailsTab },
 ];

data/webpack/ForemanRhCloudReducers.js

@@ -1,10 +1,12 @@
 import { combineReducers } from 'redux';
 import inventoryUploadReducers from './ForemanInventoryUpload/ForemanInventoryUploadReducers';
 import insightsReducers from './InsightsCloudSync/InsightsCloudSyncReducers';
+import { hostInsightsReducers } from './InsightsHostDetailsTab';
 
 export default {
   ForemanRhCloud: combineReducers({
     ...inventoryUploadReducers,
     ...insightsReducers,
+    ...hostInsightsReducers,
   }),
 };

data/webpack/ForemanRhCloudSelectors.js

@@ -1,5 +1,10 @@
 export const selectForemanRhCloud = state => state.ForemanRhCloud;
+
 export const selectForemanInventoryUpload = state =>
   selectForemanRhCloud(state).inventoryUpload;
+
 export const selectInsightsCloudSync = state =>
   selectForemanRhCloud(state).InsightsCloudSync;
+
+export const selectHostInsights = state =>
+  selectForemanRhCloud(state).hostInsights;

data/webpack/ForemanRhCloudTestHelpers.js

@@ -1,10 +1,12 @@
 export const rhCloudStateWrapper = (
   inventoryState = {},
-  insightsState = {}
+  insightsState = {},
+  hostInsightsState = {}
 ) => ({
   ForemanRhCloud: {
     inventoryUpload: { ...inventoryState },
     InsightsCloudSync: { ...insightsState },
+    hostInsights: { ...hostInsightsState },
   },
 });
 
@@ -13,3 +15,6 @@ export const inventoryStateWrapper = innerState =>
 
 export const insightsStateWrapper = innerState =>
   rhCloudStateWrapper({}, innerState);
+
+export const hostInsightsStateWrapper = innerState =>
+  rhCloudStateWrapper({}, {}, innerState);

data/webpack/InsightsCloudSync/Components/InsightsSettings/InsightsSettingsActions.js

@@ -1,10 +1,9 @@
 import API from 'foremanReact/API';
+import { addToast } from 'foremanReact/redux/actions/toasts';
 import { insightsCloudUrl } from '../../InsightsCloudSyncHelpers';
 import {
   INSIGHTS_SYNC_SETTING_SET,
-  INSIGHTS_SYNC_SETTING_SET_FAILURE,
   INSIGHTS_SYNC_SETTINGS_GET_SUCCESS,
-  INSIGHTS_SYNC_SETTINGS_GET_FAILURE,
 } from './InsightsSettingsConstants';
 
 export const getInsightsSyncSettings = () => async dispatch => {
@@ -20,15 +19,14 @@ export const getInsightsSyncSettings = () => async dispatch => {
         },
       },
     });
-  } catch (error) {
-    dispatch({
-      type: INSIGHTS_SYNC_SETTINGS_GET_FAILURE,
-      payload: {
-        settings: {
-          error: error.message,
-        },
-      },
-    });
+  } catch ({ message }) {
+    dispatch(
+      addToast({
+        sticky: true,
+        type: 'error',
+        message,
+      })
+    );
   }
 };
 
@@ -47,14 +45,13 @@ export const setInsightsSyncEnabled = currentInsightsSyncEnabled => async dispat
         },
       },
     });
-  } catch (error) {
-    dispatch({
-      type: INSIGHTS_SYNC_SETTING_SET_FAILURE,
-      payload: {
-        settings: {
-          error: error.message,
-        },
-      },
-    });
+  } catch ({ message }) {
+    dispatch(
+      addToast({
+        sticky: true,
+        type: 'error',
+        message,
+      })
+    );
   }
 };