foreman_rh_cloud 1.0.10 → 1.0.14

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

@@ -1,10 +1,4 @@
 export const INSIGHTS_SYNC_SETTING_SET = 'INSIGHTS_SYNC_SETTING_SET';
 
-export const INSIGHTS_SYNC_SETTING_SET_FAILURE =
-  'INSIGHTS_SYNC_SETTING_SET_FAILURE';
-
 export const INSIGHTS_SYNC_SETTINGS_GET_SUCCESS =
   'INSIGHTS_SYNC_SETTINGS_GET_SUCCESS';
-
-export const INSIGHTS_SYNC_SETTINGS_GET_FAILURE =
-  'INSIGHTS_SYNC_SETTINGS_GET_FAILURE';

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

@@ -1,9 +1,7 @@
 import Immutable from 'seamless-immutable';
 import {
   INSIGHTS_SYNC_SETTING_SET,
-  INSIGHTS_SYNC_SETTING_SET_FAILURE,
   INSIGHTS_SYNC_SETTINGS_GET_SUCCESS,
-  INSIGHTS_SYNC_SETTINGS_GET_FAILURE,
 } from './InsightsSettingsConstants';
 
 const initialState = Immutable({
@@ -15,9 +13,7 @@ export default (state = initialState, action) => {
 
   switch (action.type) {
     case INSIGHTS_SYNC_SETTINGS_GET_SUCCESS:
-    case INSIGHTS_SYNC_SETTINGS_GET_FAILURE:
     case INSIGHTS_SYNC_SETTING_SET:
-    case INSIGHTS_SYNC_SETTING_SET_FAILURE:
       return state.merge({
         ...state,
         ...settings,

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

@@ -22,12 +22,28 @@ const runWithGetState = (state, action, params) => dispatch => {
 const fixtures = {
   'should generate INSIGHTS_SYNC_SETTINGS_GET_SUCCESS action': () =>
     runWithGetState({ settings: {} }, getInsightsSyncSettings, {}),
+  'should handle getInsightsSyncSettings with error ': () => {
+    API.get.mockImplementationOnce(() =>
+      Promise.reject(new Error('Network error!'))
+    );
+    return runWithGetState({ settings: {} }, getInsightsSyncSettings, {});
+  },
   'should generate INSIGHTS_SYNC_SETTING_SET action': () =>
     runWithGetState(
       { settings: { insightsSyncEnabled: false } },
       setInsightsSyncEnabled,
       true
     ),
+  'should handle setInsightsSyncEnabled with error ': () => {
+    API.patch.mockImplementationOnce(() =>
+      Promise.reject(new Error('Network error!'))
+    );
+    return runWithGetState(
+      { settings: { insightsSyncEnabled: false } },
+      setInsightsSyncEnabled,
+      true
+    );
+  },
 };
 
 describe('InsightsSettings actions', () => {

data/webpack/InsightsCloudSync/Components/InsightsSettings/__tests__/InsightsSettingsReducer.test.js

@@ -1,12 +1,9 @@
 import { testReducerSnapshotWithFixtures } from 'react-redux-test-utils';
-
+import reducer from '../InsightsSettingsReducer';
 import {
   INSIGHTS_SYNC_SETTING_SET,
-  INSIGHTS_SYNC_SETTING_SET_FAILURE,
   INSIGHTS_SYNC_SETTINGS_GET_SUCCESS,
-  INSIGHTS_SYNC_SETTINGS_GET_FAILURE,
 } from '../InsightsSettingsConstants';
-import reducer from '../InsightsSettingsReducer';
 
 const fixtures = {
   'should return the initial state': {},
@@ -20,16 +17,6 @@ const fixtures = {
       },
     },
   },
-  'should handle INSIGHTS_SYNC_SETTINGS_GET_FAILURE': {
-    action: {
-      type: INSIGHTS_SYNC_SETTINGS_GET_FAILURE,
-      payload: {
-        settings: {
-          error: 'test error',
-        },
-      },
-    },
-  },
   'should handle INSIGHTS_SYNC_SETTING_SET': {
     action: {
       type: INSIGHTS_SYNC_SETTING_SET,
@@ -40,16 +27,6 @@ const fixtures = {
       },
     },
   },
-  'should handle INSIGHTS_SYNC_SETTING_SET_FAILURE': {
-    action: {
-      type: INSIGHTS_SYNC_SETTING_SET_FAILURE,
-      payload: {
-        settings: {
-          error: 'test set error',
-        },
-      },
-    },
-  },
 };
 
 describe('InsightsSettings reducer', () =>

data/webpack/InsightsCloudSync/Components/InsightsSettings/__tests__/__snapshots__/InsightsSettingsActions.test.js.snap

@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`InsightsSettings actions should generate INSIGHTS_SYNC_SETTINGS_GET_SUCCESS action 1`] = `
+exports[`InsightsSettings actions should generate INSIGHTS_SYNC_SETTING_SET action 1`] = `
 Array [
   Array [
     Object {
@@ -9,13 +9,13 @@ Array [
           "insightsSyncEnabled": true,
         },
       },
-      "type": "INSIGHTS_SYNC_SETTINGS_GET_SUCCESS",
+      "type": "INSIGHTS_SYNC_SETTING_SET",
     },
   ],
 ]
 `;
 
-exports[`InsightsSettings actions should generate INSIGHTS_SYNC_SETTING_SET action 1`] = `
+exports[`InsightsSettings actions should generate INSIGHTS_SYNC_SETTINGS_GET_SUCCESS action 1`] = `
 Array [
   Array [
     Object {
@@ -24,7 +24,41 @@ Array [
           "insightsSyncEnabled": true,
         },
       },
-      "type": "INSIGHTS_SYNC_SETTING_SET",
+      "type": "INSIGHTS_SYNC_SETTINGS_GET_SUCCESS",
+    },
+  ],
+]
+`;
+
+exports[`InsightsSettings actions should handle getInsightsSyncSettings with error  1`] = `
+Array [
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "Network error!",
+          "sticky": true,
+          "type": "error",
+        },
+      },
+      "type": "TOASTS_ADD",
+    },
+  ],
+]
+`;
+
+exports[`InsightsSettings actions should handle setInsightsSyncEnabled with error  1`] = `
+Array [
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "message": "Network error!",
+          "sticky": true,
+          "type": "error",
+        },
+      },
+      "type": "TOASTS_ADD",
     },
   ],
 ]

data/webpack/InsightsCloudSync/Components/InsightsSettings/__tests__/__snapshots__/InsightsSettingsReducer.test.js.snap

@@ -6,20 +6,6 @@ Object {
 }
 `;
 
-exports[`InsightsSettings reducer should handle INSIGHTS_SYNC_SETTING_SET_FAILURE 1`] = `
-Object {
-  "error": "test set error",
-  "insightsSyncEnabled": false,
-}
-`;
-
-exports[`InsightsSettings reducer should handle INSIGHTS_SYNC_SETTINGS_GET_FAILURE 1`] = `
-Object {
-  "error": "test error",
-  "insightsSyncEnabled": false,
-}
-`;
-
 exports[`InsightsSettings reducer should handle INSIGHTS_SYNC_SETTINGS_GET_SUCCESS 1`] = `
 Object {
   "insightsSyncEnabled": true,

data/webpack/InsightsCloudSync/InsightsCloudSync.js

@@ -19,7 +19,7 @@ const InsightsCloudSync = ({ data: { settingsUrl }, syncInsights }) => {
              recommendations output for hosts managed here`)}
           </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/InsightsCloudSync/InsightsCloudSyncActions.js

@@ -1,9 +1,7 @@
 import API from 'foremanReact/API';
+import { addToast } from 'foremanReact/redux/actions/toasts';
 import { insightsCloudUrl } from './InsightsCloudSyncHelpers';
-import {
-  INSIGHTS_CLOUD_SYNC_SUCCESS,
-  INSIGHTS_CLOUD_SYNC_FAILURE,
-} from './InsightsCloudSyncConstants';
+import { INSIGHTS_CLOUD_SYNC_SUCCESS } from './InsightsCloudSyncConstants';
 
 export const syncInsights = () => async dispatch => {
   try {
@@ -12,12 +10,13 @@ export const syncInsights = () => async dispatch => {
       type: INSIGHTS_CLOUD_SYNC_SUCCESS,
       payload: {},
     });
-  } catch (error) {
-    dispatch({
-      type: INSIGHTS_CLOUD_SYNC_FAILURE,
-      payload: {
-        error: error.message,
-      },
-    });
+  } catch ({ message }) {
+    dispatch(
+      addToast({
+        sticky: true,
+        type: 'error',
+        message,
+      })
+    );
   }
 };

data/webpack/InsightsCloudSync/InsightsCloudSyncConstants.js

@@ -1,6 +1,4 @@
 import { translate as __ } from 'foremanReact/common/I18n';
 
 export const INSIGHTS_CLOUD_SYNC_SUCCESS = 'INSIGHTS_CLOUD_SYNC_SUCCESS';
-export const INSIGHTS_CLOUD_SYNC_FAILURE = 'INSIGHTS_CLOUD_SYNC_FAILURE';
-
 export const INSIGHTS_SYNC_PAGE_TITLE = __('Red Hat Insights cloud sync');

data/webpack/InsightsCloudSync/__snapshots__/InsightsCloudSync.test.js.snap

@@ -19,7 +19,7 @@ exports[`InsightsCloudSync render 1`] = `
              recommendations output for hosts managed here
       </p>
       <p>
-        1. Obtain an RHSM API token: 
+        1. Obtain an Red Hat API token: 
         <a
           href="https://access.redhat.com/management/api"
           rel="noopener noreferrer"

data/webpack/InsightsHostDetailsTab/InsightsTab.js

@@ -0,0 +1,64 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { orderBy } from 'lodash';
+import { Grid, ListView, noop } from 'patternfly-react';
+import ListItem from './components/ListItem';
+import './InsightsTab.scss';
+
+class InsightsHostDetailsTab extends React.Component {
+  componentDidMount() {
+    const { fetchHits, hostID } = this.props;
+    fetchHits(hostID);
+  }
+
+  render() {
+    const { hits } = this.props;
+
+    if (!hits.length) {
+      return <h2>No recommendations were found for this host!</h2>;
+    }
+    const hitsSorted = orderBy(hits, ['insights_hit.total_risk'], ['desc']);
+    const items = hitsSorted.map(
+      (
+        {
+          insights_hit: {
+            title,
+            total_risk: totalRisk,
+            results_url: resultsUrl,
+            solution_url: solutionUrl,
+          },
+        },
+        index
+      ) => (
+        <ListItem
+          key={index}
+          title={title}
+          totalRisk={totalRisk}
+          resultsUrl={resultsUrl}
+          solutionUrl={solutionUrl}
+        />
+      )
+    );
+    return (
+      <Grid.Row>
+        <Grid.Col xs={12}>
+          <h2>Recommendations</h2>
+          <ListView id="hits_list">{items}</ListView>
+        </Grid.Col>
+      </Grid.Row>
+    );
+  }
+}
+
+InsightsHostDetailsTab.propTypes = {
+  hostID: PropTypes.number.isRequired,
+  fetchHits: PropTypes.func,
+  hits: PropTypes.array,
+};
+
+InsightsHostDetailsTab.defaultProps = {
+  fetchHits: noop,
+  hits: [],
+};
+
+export default InsightsHostDetailsTab;

data/webpack/InsightsHostDetailsTab/InsightsTab.scss

@@ -0,0 +1,86 @@
+#host_details_insights_react_container {
+  #btn_toolbar {
+    float: right;
+  }
+
+  #btn_fullscreen {
+    margin-left: 5px;
+
+    .fa-arrows-alt {
+      margin-left: 5px;
+    }
+  }
+
+  h2 {
+    margin-top: 5px;
+  }
+
+  #hits_list {
+    max-height: 650px;
+    overflow-y: scroll;
+    overflow-x: hidden;
+    margin-top: 15px;
+
+    .list-view-pf-expand {
+      padding: 0;
+    }
+
+    .list-group-item-header {
+      .list-view-pf-expand .fa-angle-right {
+        margin-top: 6px;
+      }
+
+      .list-view-pf-main-info .list-view-pf-description {
+        width: 83%;
+
+        .list-group-item-heading {
+          width: 375px;
+        }
+
+        p {
+          margin: 0;
+          font-size: 14px;
+        }
+      }
+
+      .list-view-pf-additional-info {
+        .risk-label {
+          padding: 5px 8px;
+          border-radius: 12px;
+
+          &.Low {
+            background-color: #e7f1fa;
+          }
+
+          &.Moderate {
+            background-color: #fdf7e7;
+          }
+
+          &.Important {
+            background-color: #f9dddd;
+          }
+
+          &.Critical {
+            background-color: #ffecec;
+          }
+
+          p {
+            margin: 0;
+          }
+        }
+      }
+    }
+
+    &::-webkit-scrollbar {
+      width: 12px;
+      height: 12px;
+    }
+
+    &::-webkit-scrollbar-thumb {
+      background: #222;
+      border-radius: 6px;
+      border: 3px solid transparent;
+      background-clip: content-box;
+    }
+  }
+}

data/webpack/InsightsHostDetailsTab/InsightsTabActions.js

@@ -0,0 +1,31 @@
+import API from 'foremanReact/API';
+import { addToast } from 'foremanReact/redux/actions/toasts';
+import { insightsCloudUrl } from '../InsightsCloudSync/InsightsCloudSyncHelpers';
+import {
+  INSIGHTS_HITS_REQUEST,
+  INSIGHTS_HITS_SUCCESS,
+} from './InsightsTabConstants';
+
+export const fetchHits = hostID => async dispatch => {
+  try {
+    dispatch({
+      type: INSIGHTS_HITS_REQUEST,
+      payload: {},
+    });
+    const {
+      data: { hits },
+    } = await API.get(insightsCloudUrl(`hits/${hostID}`));
+    dispatch({
+      type: INSIGHTS_HITS_SUCCESS,
+      payload: { hits },
+    });
+  } catch ({ message }) {
+    dispatch(
+      addToast({
+        sticky: true,
+        type: 'error',
+        message,
+      })
+    );
+  }
+};

data/webpack/InsightsHostDetailsTab/InsightsTabConstants.js

@@ -0,0 +1,2 @@
+export const INSIGHTS_HITS_REQUEST = 'INSIGHTS_HITS_REQUEST';
+export const INSIGHTS_HITS_SUCCESS = 'INSIGHTS_HITS_SUCCESS';

data/webpack/InsightsHostDetailsTab/InsightsTabReducer.js

@@ -0,0 +1,19 @@
+import Immutable from 'seamless-immutable';
+import { INSIGHTS_HITS_SUCCESS } from './InsightsTabConstants';
+
+const initialState = Immutable({
+  hits: [],
+});
+
+export default (state = initialState, action) => {
+  const { payload: { hits } = {} } = action;
+
+  switch (action.type) {
+    case INSIGHTS_HITS_SUCCESS:
+      return state.merge({
+        hits,
+      });
+    default:
+      return state;
+  }
+};