foreman_rh_cloud 1.0.18 → 2.0.4

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

@@ -1,11 +1,10 @@
-import { testComponentSnapshotsWithFixtures } from 'react-redux-test-utils';
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 
 import AccountList from '../AccountList';
 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,20 +1,13 @@
-import { testActionSnapshotWithFixtures } from 'react-redux-test-utils';
-import API from 'foremanReact/API';
+import { testActionSnapshotWithFixtures } from '@theforeman/test';
 import {
   fetchAccountsStatus,
   startAccountStatusPolling,
   stopAccountStatusPolling,
   restartProcess,
 } from '../AccountListActions';
-import {
-  pollingProcessID,
-  fetchAccountsStatusResponse,
-} from '../AccountList.fixtures';
+import { pollingProcessID } 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': () =>
@@ -22,13 +15,6 @@ 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__/AccountListIntegration.test.js

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

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

@@ -1,4 +1,4 @@
-import { testReducerSnapshotWithFixtures } from 'react-redux-test-utils';
+import { testReducerSnapshotWithFixtures } from '@theforeman/test';
 
 import {
   INVENTORY_ACCOUNT_STATUS_POLLING,
@@ -9,25 +9,21 @@ import {
 } from '../AccountListConstants';
 import reducer from '../AccountListReducer';
 import {
+  accounts,
   error,
   pollingProcessID,
   accountID,
   processStatusName,
-  autoUploadEnabled,
-  hostObfuscationEnabled,
-  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: pollingResponse,
+      payload: {
+        accounts,
+      },
     },
   },
   'should handle INVENTORY_ACCOUNT_STATUS_POLLING_ERROR': {
@@ -58,30 +54,6 @@ const fixtures = {
       },
     },
   },
-  'should handle AUTO_UPLOAD_TOGGLE': {
-    action: {
-      type: AUTO_UPLOAD_TOGGLE,
-      payload: {
-        autoUploadEnabled,
-      },
-    },
-  },
-  'should handle HOST_OBFUSCATION_TOGGLE': {
-    action: {
-      type: HOST_OBFUSCATION_TOGGLE,
-      payload: {
-        hostObfuscationEnabled,
-      },
-    },
-  },
-  'should handle IPS_OBFUSCATION_TOGGLE': {
-    action: {
-      type: IPS_OBFUSCATION_TOGGLE,
-      payload: {
-        ipsObfuscationEnabled,
-      },
-    },
-  },
 };
 
 describe('AccountList reducer', () =>

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

@@ -1,46 +1,25 @@
-import { testSelectorsSnapshotWithFixtures } from 'react-redux-test-utils';
+import { testSelectorsSnapshotWithFixtures } from '@theforeman/test';
 import {
   selectAccountsList,
   selectAccounts,
   selectPollingProcessID,
-  selectAutoUploadEnabled,
-  selectHostObfuscationEnabled,
-  selectCloudToken,
-  selectExcludePackages,
 } from '../AccountListSelectors';
-import {
-  pollingProcessID,
-  accounts,
-  autoUploadEnabled,
-  hostObfuscationEnabled,
-  cloudToken,
-  excludePackages,
-} from '../AccountList.fixtures';
-import { rhCloudStateWrapper } from '../../../../ForemanRhCloudTestHelpers';
+import { pollingProcessID, accounts } from '../AccountList.fixtures';
 
-const state = rhCloudStateWrapper({
-  accountsList: {
-    accounts,
-    pollingProcessID,
-    autoUploadEnabled,
-    hostObfuscationEnabled,
-    cloudToken,
-    excludePackages,
+const state = {
+  ForemanInventoryUpload: {
+    accountsList: {
+      accounts,
+      pollingProcessID,
+    },
   },
-});
+};
 
 const fixtures = {
   'should return AccountsList': () => selectAccountsList(state),
   'should return AccountList accounts': () => selectAccounts(state),
   'should return AccountList pollingProcessID': () =>
     selectPollingProcessID(state),
-  'should return AccountList autoUploadEnabled': () =>
-    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__/AccountList.test.js.snap

@@ -8,36 +8,37 @@ exports[`AccountList rendering render with props 1`] = `
     account={
       Object {
         "generate_report_status": "running",
-        "id": 1,
+        "label": "test_org1",
         "upload_report_status": "running",
       }
     }
+    accountID="Account1"
+    initExpanded={true}
     key="0"
-    label="Account1"
   />
   <ListItem
     account={
       Object {
         "generate_report_status": "failure",
-        "id": 2,
+        "label": "test_org2",
         "upload_report_status": "unknown",
       }
     }
+    accountID="Account2"
+    initExpanded={false}
     key="1"
-    label="Account2"
   />
   <ListItem
     account={
       Object {
         "generate_report_status": "running",
-        "id": 3,
+        "label": "test_org3",
         "upload_report_status": "success",
       }
     }
+    accountID="Account3"
+    initExpanded={false}
     key="2"
-    label="Account3"
   />
 </ListView>
 `;
-
-exports[`AccountList rendering show empty results 1`] = `<inventoryEmptyResults />`;

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

@@ -5,47 +5,9 @@ Array [
   Array [
     Object {
       "payload": Object {
-        "accounts": Object {
-          "Account1": Object {
-            "generate_report_status": "running",
-            "id": 1,
-            "upload_report_status": "running",
-          },
-          "Account2": Object {
-            "generate_report_status": "failure",
-            "id": 2,
-            "upload_report_status": "unknown",
-          },
-          "Account3": Object {
-            "generate_report_status": "running",
-            "id": 3,
-            "upload_report_status": "success",
-          },
-        },
-        "autoUploadEnabled": true,
-        "cloudToken": "some-cloud-token",
-        "excludePackages": false,
-        "hostObfuscationEnabled": true,
-        "ipsObfuscationEnabled": true,
+        "error": "Cannot read property 'data' of undefined",
       },
-      "type": "INVENTORY_ACCOUNT_STATUS_POLLING",
-    },
-  ],
-]
-`;
-
-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",
+      "type": "INVENTORY_ACCOUNT_STATUS_POLLING_ERROR",
     },
   ],
 ]

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

@@ -1,48 +1,25 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`AccountList reducer should handle AUTO_UPLOAD_TOGGLE 1`] = `
-Object {
-  "accounts": Object {},
-  "autoUploadEnabled": true,
-  "error": null,
-  "pollingProcessID": 0,
-}
-`;
-
-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 {
     "Account1": Object {
       "generate_report_status": "running",
-      "id": 1,
+      "label": "test_org1",
       "upload_report_status": "running",
     },
     "Account2": Object {
       "generate_report_status": "failure",
-      "id": 2,
+      "label": "test_org2",
       "upload_report_status": "unknown",
     },
     "Account3": Object {
       "generate_report_status": "running",
-      "id": 3,
+      "label": "test_org3",
       "upload_report_status": "success",
     },
   },
-  "autoUploadEnabled": true,
-  "cloudToken": "some-cloud-token",
   "error": null,
-  "excludePackages": false,
-  "hostObfuscationEnabled": true,
-  "ipsObfuscationEnabled": true,
   "pollingProcessID": 0,
 }
 `;
@@ -83,15 +60,6 @@ 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

@@ -1,58 +1,46 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`AccountList selectors should return AccountList pollingProcessID 1`] = `0`;
+
 exports[`AccountList selectors should return AccountList accounts 1`] = `
 Object {
   "Account1": Object {
     "generate_report_status": "running",
-    "id": 1,
+    "label": "test_org1",
     "upload_report_status": "running",
   },
   "Account2": Object {
     "generate_report_status": "failure",
-    "id": 2,
+    "label": "test_org2",
     "upload_report_status": "unknown",
   },
   "Account3": Object {
     "generate_report_status": "running",
-    "id": 3,
+    "label": "test_org3",
     "upload_report_status": "success",
   },
 }
 `;
 
-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`;
-
 exports[`AccountList selectors should return AccountsList 1`] = `
 Object {
   "accounts": Object {
     "Account1": Object {
       "generate_report_status": "running",
-      "id": 1,
+      "label": "test_org1",
       "upload_report_status": "running",
     },
     "Account2": Object {
       "generate_report_status": "failure",
-      "id": 2,
+      "label": "test_org2",
       "upload_report_status": "unknown",
     },
     "Account3": Object {
       "generate_report_status": "running",
-      "id": 3,
+      "label": "test_org3",
       "upload_report_status": "success",
     },
   },
-  "autoUploadEnabled": true,
-  "cloudToken": "some-cloud-token",
-  "excludePackages": false,
-  "hostObfuscationEnabled": true,
   "pollingProcessID": 0,
 }
 `;

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

@@ -9,14 +9,12 @@ 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/Dashboard/Dashboard.fixtures.js

@@ -8,16 +8,20 @@ export const completed = 25;
 
 export const error = 'some-error';
 
+export const generating = {
+  logs,
+  completed,
+};
+
+export const uploading = {
+  logs,
+  completed,
+  files,
+};
+
 export const inventory = {
-  generating: {
-    logs,
-    completed,
-  },
-  uploading: {
-    logs,
-    completed,
-    files,
-  },
+  generating,
+  uploading,
 };
 
 export const accountID = 'some-account-ID';