foreman_rh_cloud 1.0.6 → 1.0.11

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

@@ -0,0 +1,10 @@
+export const filterAccounts = (accounts, accountIds, filterTerm) => {
+  if (!filterTerm || !accountIds.length) {
+    return accountIds;
+  }
+
+  const filterTermLowerCased = filterTerm.toLowerCase();
+  return accountIds.filter(id =>
+    accounts[id].label.toLowerCase().includes(filterTermLowerCased)
+  );
+};

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

@@ -1,11 +1,12 @@
 import Immutable from 'seamless-immutable';
+import { AUTO_UPLOAD_TOGGLE } from '../AutoUploadSwitcher/AutoUploadSwitcherConstants';
+import { HOST_OBFUSCATION_TOGGLE } from '../HostObfuscationSwitcher/HostObfuscationSwitcherConstants';
 import {
   INVENTORY_ACCOUNT_STATUS_POLLING,
   INVENTORY_ACCOUNT_STATUS_POLLING_ERROR,
   INVENTORY_ACCOUNT_STATUS_POLLING_START,
   INVENTORY_PROCESS_RESTART,
 } from './AccountListConstants';
-import { AUTO_UPLOAD_TOGGLE } from '../AutoUploadSwitcher/AutoUploadSwitcherConstants';
 
 const initialState = Immutable({
   accounts: {},
@@ -22,6 +23,8 @@ export default (state = initialState, action) => {
       accountID,
       processStatusName,
       autoUploadEnabled,
+      hostObfuscationEnabled,
+      cloudToken,
     } = {},
   } = action;
 
@@ -31,6 +34,8 @@ export default (state = initialState, action) => {
         ...state,
         accounts,
         autoUploadEnabled,
+        hostObfuscationEnabled,
+        cloudToken,
         error: null,
       });
     case INVENTORY_ACCOUNT_STATUS_POLLING_ERROR:
@@ -57,6 +62,11 @@ export default (state = initialState, action) => {
         ...state,
         autoUploadEnabled,
       });
+    case HOST_OBFUSCATION_TOGGLE:
+      return state.merge({
+        ...state,
+        hostObfuscationEnabled,
+      });
     default:
       return state;
   }

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

@@ -1,10 +1,19 @@
-import { selectForemanInventoryUpload } from '../../ForemanInventoryUploadSelectors';
+import { selectForemanInventoryUpload } from '../../../ForemanRhCloudSelectors';
 
 export const selectAccountsList = state =>
   selectForemanInventoryUpload(state).accountsList;
+
 export const selectAccounts = state => selectAccountsList(state).accounts;
+
 export const selectPollingProcessID = state =>
   selectAccountsList(state).pollingProcessID;
+
 export const selectError = state => selectAccountsList(state).error;
+
 export const selectAutoUploadEnabled = state =>
   selectAccountsList(state).autoUploadEnabled;
+
+export const selectHostObfuscationEnabled = state =>
+  selectAccountsList(state).hostObfuscationEnabled;
+
+export const selectCloudToken = state => selectAccountsList(state).cloudToken;

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

@@ -0,0 +1,15 @@
+import React from 'react';
+import { EmptyState, Icon } from 'patternfly-react';
+import { translate as __ } from 'foremanReact/common/I18n';
+import './emptyResults.scss';
+
+const inventoryEmptyResults = () => (
+  <EmptyState>
+    <EmptyState.Title>
+      <Icon className="no_results_icon" name="meh-o" />
+      {__("Oops! Couldn't find organization that matches your query")}
+    </EmptyState.Title>
+  </EmptyState>
+);
+
+export default inventoryEmptyResults;

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

@@ -0,0 +1,13 @@
+import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+
+import EmptyResults from '../EmptyResults';
+
+const fixtures = {
+  'render without Props': {},
+  /** fixtures, props for the component */
+};
+
+describe('EmptyResults', () => {
+  describe('rendering', () =>
+    testComponentSnapshotsWithFixtures(EmptyResults, fixtures));
+});

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

@@ -0,0 +1,18 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`EmptyResults rendering render without Props 1`] = `
+<EmptyState
+  className=""
+>
+  <EmptyStateTitle
+    className=""
+  >
+    <Icon
+      className="no_results_icon"
+      name="meh-o"
+      type="fa"
+    />
+    Oops! Couldn't find organization that matches your query
+  </EmptyStateTitle>
+</EmptyState>
+`;

data/webpack/ForemanInventoryUpload/Components/AccountList/Components/EmptyResults/emptyResults.scss

@@ -0,0 +1,7 @@
+.blank-slate-pf {
+  margin-top: 100px;
+
+  .no_results_icon {
+    margin-right: 5px;
+  }
+}

data/webpack/ForemanInventoryUpload/Components/AccountList/Components/EmptyResults/index.js

@@ -0,0 +1 @@
+export { default } from './EmptyResults';

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

@@ -4,7 +4,7 @@ import PropTypes from 'prop-types';
 import ListItemStatus from '../ListItemStatus';
 import Dashboard from '../../../Dashboard';
 
-const ListItem = ({ accountID, account, initExpanded }) => (
+const ListItem = ({ accountID, account }) => (
   <ListView.Item
     leftContent={<ListView.Icon name="user" />}
     heading={account.label}
@@ -13,7 +13,6 @@ const ListItem = ({ accountID, account, initExpanded }) => (
     ]}
     stacked
     hideCloseIcon
-    initExpanded={initExpanded}
   >
     <Dashboard accountID={accountID} account={account} />
   </ListView.Item>
@@ -26,7 +25,6 @@ ListItem.propTypes = {
     upload_report_status: PropTypes.string,
     label: PropTypes.string,
   }),
-  initExpanded: PropTypes.bool,
 };
 
 ListItem.defaultProps = {
@@ -35,7 +33,6 @@ ListItem.defaultProps = {
     upload_report_status: 'unknown',
     label: 'default_org_name',
   },
-  initExpanded: false,
 };
 
 export default ListItem;

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

@@ -5,6 +5,7 @@ import { props } from '../AccountList.fixtures';
 
 const fixtures = {
   'render with props': props,
+  'show empty results': { ...props, filterTerm: 'not_matching_term' },
 };
 
 describe('AccountList', () => {

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

@@ -1,13 +1,20 @@
 import { testActionSnapshotWithFixtures } from 'react-redux-test-utils';
+import API from 'foremanReact/API';
 import {
   fetchAccountsStatus,
   startAccountStatusPolling,
   stopAccountStatusPolling,
   restartProcess,
 } from '../AccountListActions';
-import { pollingProcessID } from '../AccountList.fixtures';
+import {
+  pollingProcessID,
+  fetchAccountsStatusResponse,
+} from '../AccountList.fixtures';
 import { accountID, activeTab } from '../../Dashboard/Dashboard.fixtures';
 
+jest.mock('foremanReact/API');
+API.get.mockImplementation(async () => fetchAccountsStatusResponse);
+
 const fixtures = {
   'should fetchAccountsStatus': () => fetchAccountsStatus(),
   'should startAccountStatusPolling': () =>

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

@@ -0,0 +1,12 @@
+import { filterAccounts } from '../AccountListHelper';
+import { accounts, accountIDs } from '../AccountList.fixtures';
+
+describe('AccountList helpers', () => {
+  it('account ids filter should return all accounts where label contains "test"', () => {
+    expect(filterAccounts(accounts, accountIDs, 'test')).toEqual(accountIDs);
+  });
+
+  it('account ids filter should not match', () => {
+    expect(filterAccounts(accounts, accountIDs, 'no_match')).toEqual([]);
+  });
+});

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

@@ -2,7 +2,7 @@ import React from 'react';
 import { IntegrationTestHelper } from 'react-redux-test-utils';
 
 import AccountList from '../index';
-import reducers from '../../../ForemanInventoryUploadReducers';
+import reducers from '../../../../ForemanRhCloudReducers';
 
 describe('AccountList integration test', () => {
   it('should flow', async () => {

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

@@ -15,8 +15,11 @@ import {
   accountID,
   processStatusName,
   autoUploadEnabled,
+  hostObfuscationEnabled,
+  cloudToken,
 } from '../AccountList.fixtures';
 import { AUTO_UPLOAD_TOGGLE } from '../../AutoUploadSwitcher/AutoUploadSwitcherConstants';
+import { HOST_OBFUSCATION_TOGGLE } from '../../HostObfuscationSwitcher/HostObfuscationSwitcherConstants';
 
 const fixtures = {
   'should return the initial state': {},
@@ -26,6 +29,8 @@ const fixtures = {
       payload: {
         accounts,
         autoUploadEnabled,
+        hostObfuscationEnabled,
+        cloudToken,
       },
     },
   },
@@ -65,6 +70,14 @@ const fixtures = {
       },
     },
   },
+  'should handle HOST_OBFUSCATION_TOGGLE': {
+    action: {
+      type: HOST_OBFUSCATION_TOGGLE,
+      payload: {
+        hostObfuscationEnabled,
+      },
+    },
+  },
 };
 
 describe('AccountList reducer', () =>

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

@@ -4,22 +4,24 @@ import {
   selectAccounts,
   selectPollingProcessID,
   selectAutoUploadEnabled,
+  selectHostObfuscationEnabled,
 } from '../AccountListSelectors';
 import {
   pollingProcessID,
   accounts,
   autoUploadEnabled,
+  hostObfuscationEnabled,
 } from '../AccountList.fixtures';
+import { rhCloudStateWrapper } from '../../../../ForemanRhCloudTestHelpers';
 
-const state = {
-  ForemanInventoryUpload: {
-    accountsList: {
-      accounts,
-      pollingProcessID,
-      autoUploadEnabled,
-    },
+const state = rhCloudStateWrapper({
+  accountsList: {
+    accounts,
+    pollingProcessID,
+    autoUploadEnabled,
+    hostObfuscationEnabled,
   },
-};
+});
 
 const fixtures = {
   'should return AccountsList': () => selectAccountsList(state),
@@ -28,6 +30,8 @@ const fixtures = {
     selectPollingProcessID(state),
   'should return AccountList autoUploadEnabled': () =>
     selectAutoUploadEnabled(state),
+  'should return AccountList hostObfuscationEnabled': () =>
+    selectHostObfuscationEnabled(state),
 };
 
 describe('AccountList selectors', () =>

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

@@ -13,7 +13,6 @@ exports[`AccountList rendering render with props 1`] = `
       }
     }
     accountID="Account1"
-    initExpanded={true}
     key="0"
   />
   <ListItem
@@ -25,7 +24,6 @@ exports[`AccountList rendering render with props 1`] = `
       }
     }
     accountID="Account2"
-    initExpanded={false}
     key="1"
   />
   <ListItem
@@ -37,8 +35,9 @@ exports[`AccountList rendering render with props 1`] = `
       }
     }
     accountID="Account3"
-    initExpanded={false}
     key="2"
   />
 </ListView>
 `;
+
+exports[`AccountList rendering show empty results 1`] = `<inventoryEmptyResults />`;

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

@@ -5,9 +5,28 @@ Array [
   Array [
     Object {
       "payload": Object {
-        "error": "Cannot read property 'data' of undefined",
+        "accounts": Object {
+          "Account1": Object {
+            "generate_report_status": "running",
+            "label": "test_org1",
+            "upload_report_status": "running",
+          },
+          "Account2": Object {
+            "generate_report_status": "failure",
+            "label": "test_org2",
+            "upload_report_status": "unknown",
+          },
+          "Account3": Object {
+            "generate_report_status": "running",
+            "label": "test_org3",
+            "upload_report_status": "success",
+          },
+        },
+        "autoUploadEnabled": true,
+        "cloudToken": "some-cloud-token",
+        "hostObfuscationEnabled": true,
       },
-      "type": "INVENTORY_ACCOUNT_STATUS_POLLING_ERROR",
+      "type": "INVENTORY_ACCOUNT_STATUS_POLLING",
     },
   ],
 ]

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

@@ -9,6 +9,15 @@ Object {
 }
 `;
 
+exports[`AccountList reducer should handle HOST_OBFUSCATION_TOGGLE 1`] = `
+Object {
+  "accounts": Object {},
+  "error": null,
+  "hostObfuscationEnabled": true,
+  "pollingProcessID": 0,
+}
+`;
+
 exports[`AccountList reducer should handle INVENTORY_ACCOUNT_STATUS_POLLING 1`] = `
 Object {
   "accounts": Object {
@@ -29,7 +38,9 @@ Object {
     },
   },
   "autoUploadEnabled": true,
+  "cloudToken": "some-cloud-token",
   "error": null,
+  "hostObfuscationEnabled": true,
   "pollingProcessID": 0,
 }
 `;

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

@@ -22,6 +22,8 @@ Object {
 
 exports[`AccountList selectors should return AccountList autoUploadEnabled 1`] = `true`;
 
+exports[`AccountList selectors should return AccountList hostObfuscationEnabled 1`] = `true`;
+
 exports[`AccountList selectors should return AccountList pollingProcessID 1`] = `0`;
 
 exports[`AccountList selectors should return AccountsList 1`] = `
@@ -44,6 +46,7 @@ Object {
     },
   },
   "autoUploadEnabled": true,
+  "hostObfuscationEnabled": true,
   "pollingProcessID": 0,
 }
 `;

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

@@ -9,12 +9,14 @@ import {
   selectPollingProcessID,
   selectError,
 } from './AccountListSelectors';
+import { selectFilterTerm } from '../InventoryFilter/InventoryFilterSelectors';
 
 // map state to props
 const mapStateToProps = state => ({
   accounts: selectAccounts(state),
   pollingProcessID: selectPollingProcessID(state),
   error: selectError(state),
+  filterTerm: selectFilterTerm(state),
 });
 
 // map action dispatchers to props

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

@@ -6,17 +6,17 @@ import './autoUploadSwitcher.scss';
 
 const AutoUploadSwitcher = ({ autoUploadEnabled, handleToggle }) => (
   <div className="auto_upload_switcher">
-    <Switch
-      size="mini"
-      value={autoUploadEnabled}
-      onChange={() => handleToggle(autoUploadEnabled)}
-    />
+    <span>Auto upload</span>
     <FieldLevelHelp
       content={__(
         'Enable automatic upload of your host inventory to the Red Hat cloud'
       )}
     />
-    <span>Allow Auto Upload</span>
+    <Switch
+      size="mini"
+      value={autoUploadEnabled}
+      onChange={() => handleToggle(autoUploadEnabled)}
+    />
   </div>
 );