foreman_rh_cloud 12.0.0 → 12.1.1

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

@@ -1,25 +1,69 @@
 import React from 'react';
-import { mount, testComponentSnapshotsWithFixtures } from '@theforeman/test';
+import { Provider } from 'react-redux';
+import configureMockStore from 'redux-mock-store';
+import thunk from 'redux-thunk';
+import { screen, render } from '@testing-library/react';
+import { mount } from '@theforeman/test';
 import NavContainer from '../NavContainer';
 import { props } from '../NavContainer.fixtures';
 
-const fixtures = {
-  'render without Props': {},
-  'render with Props': props,
-};
+const middlewares = [thunk];
+const mockStore = configureMockStore(middlewares);
 
 describe('NavContainer', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(NavContainer, fixtures));
+  const renderOptions = {
+    API: {
+      INVENTORY_SETTINGS: {
+        response: { subscriptionConnectionEnabled: false },
+      },
+    },
+  };
 
-  it('click should call onTabClick prop', () => {
+  test('click should call onTabClick prop', () => {
     const onTabClick = jest.fn();
     const modifiedProps = {
       ...props,
     };
     modifiedProps.items[0].onClick = onTabClick;
-    const wrapper = mount(<NavContainer {...modifiedProps} />);
+    const store = mockStore(renderOptions);
+
+    const wrapper = mount(
+      <Provider store={store}>
+        <NavContainer {...modifiedProps} />
+      </Provider>
+    );
     wrapper.find('NavItem a').simulate('click');
     expect(onTabClick).toBeCalled();
   });
+
+  test('when subscription connection is enabled', () => {
+    const options = {
+      API: {
+        INVENTORY_SETTINGS: {
+          response: { subscriptionConnectionEnabled: true },
+        },
+      },
+    };
+    const store = mockStore(options);
+    render(
+      <Provider store={store}>
+        <NavContainer {...props} />
+      </Provider>
+    );
+
+    expect(screen.queryAllByText('Generating')).toHaveLength(1);
+    expect(screen.queryAllByText('Uploading')).toHaveLength(1);
+  });
+
+  test('when subscription connection is not enabled', () => {
+    const store = mockStore(renderOptions);
+    render(
+      <Provider store={store}>
+        <NavContainer {...props} />
+      </Provider>
+    );
+
+    expect(screen.queryAllByText('Generating')).toHaveLength(1);
+    expect(screen.queryAllByText('Uploading')).toHaveLength(0);
+  });
 });

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

@@ -1,68 +1,82 @@
 import React from 'react';
 import { Text } from '@patternfly/react-core';
+import { useSelector } from 'react-redux';
 
 import { translate as __ } from 'foremanReact/common/I18n';
 import { FormattedMessage } from 'react-intl';
+import { selectSubscriptionConnectionEnabled } from '../../../InventorySettings/InventorySettingsSelectors';
 
-export const PageDescription = () => (
-  <div id="inventory_page_description">
-    <Text>
-      {__(
-        'The Red Hat Hybrid Cloud Console provides a set of cloud services, including Red Hat Insights and Subscriptions, that provide predictive analysis, remediation of issues, and unified subscription reporting for this Foreman instance.'
-      )}
-    </Text>
-    <Text>
-      {__(
-        'The Foreman inventory upload plugin automatically uploads Foreman host inventory data to the Inventory service of Insights, where it can also be used by the Subscriptions service for subscription reporting. If you use the Subscriptions service, enabling inventory uploads is required.'
-      )}
-    </Text>
-    <Text>
-      <FormattedMessage
-        id="enable-upload-hint"
-        defaultMessage={__(
-          'To enable this reporting for all Foreman organizations, set {uploadButtonName} to on. The data will be reported automatically once per day.'
+export const PageDescription = () => {
+  const subscriptionConnectionEnabled = useSelector(
+    selectSubscriptionConnectionEnabled
+  );
+
+  return (
+    <div id="inventory_page_description">
+      <Text>
+        {__(
+          'The Red Hat Hybrid Cloud Console provides a set of cloud services, including Red Hat Insights and Subscriptions, that provide predictive analysis, remediation of issues, and unified subscription reporting for this Foreman instance.'
         )}
-        values={{
-          uploadButtonName: <strong>{__('Automatic inventory upload')}</strong>,
-        }}
-      />
-    </Text>
-    <Text>
-      <FormattedMessage
-        id="restart-button-hint"
-        defaultMessage={__(
-          'To manually upload the data for a specific organization, select an organization and click {restartButtonName}.'
+      </Text>
+      <Text>
+        {__(
+          'The Foreman inventory upload plugin automatically uploads Foreman host inventory data to the Inventory service of Insights, where it can also be used by the Subscriptions service for subscription reporting. If you use the Subscriptions service, enabling inventory uploads is required.'
         )}
-        values={{
-          restartButtonName: (
-            <strong>{__('Generate and upload report')}</strong>
-          ),
-        }}
-      />
-    </Text>
-    <Text>
-      {__('For more information about the Subscriptions service, see:')}
-      &nbsp;
-      <a
-        href="https://docs.redhat.com/en/documentation/subscription_central/1-latest/html/getting_started_with_the_subscriptions_service/index"
-        target="_blank"
-        rel="noopener noreferrer"
-      >
-        {__('About subscription watch')}
-      </a>
-    </Text>
-    <Text>
-      {__('For more information about Insights and Cloud Connector, see:')}
-      &nbsp;
-      <a
-        href="https://console.redhat.com/security/insights/"
-        target="_blank"
-        rel="noopener noreferrer"
-      >
-        {__('Red Hat Insights Data and Application Security')}
-      </a>
-    </Text>
-  </div>
-);
+      </Text>
+      {subscriptionConnectionEnabled && (
+        <Text>
+          <FormattedMessage
+            id="enable-upload-hint"
+            defaultMessage={__(
+              'To enable this reporting for all Foreman organizations, set {uploadButtonName} to on. The data will be reported automatically once per day.'
+            )}
+            values={{
+              uploadButtonName: (
+                <strong>{__('Automatic inventory upload')}</strong>
+              ),
+            }}
+          />
+        </Text>
+      )}
+      {subscriptionConnectionEnabled && (
+        <Text>
+          <FormattedMessage
+            id="restart-button-hint"
+            defaultMessage={__(
+              'To manually upload the data for a specific organization, select an organization and click {restartButtonName}.'
+            )}
+            values={{
+              restartButtonName: (
+                <strong>{__('Generate and upload report')}</strong>
+              ),
+            }}
+          />
+        </Text>
+      )}
+      <Text>
+        {__('For more information about the Subscriptions service, see:')}
+        &nbsp;
+        <a
+          href="https://docs.redhat.com/en/documentation/subscription_central/1-latest/html/getting_started_with_the_subscriptions_service/index"
+          target="_blank"
+          rel="noopener noreferrer"
+        >
+          {__('About subscription watch')}
+        </a>
+      </Text>
+      <Text>
+        {__('For more information about Insights and Cloud Connector, see:')}
+        &nbsp;
+        <a
+          href="https://console.redhat.com/security/insights/"
+          target="_blank"
+          rel="noopener noreferrer"
+        >
+          {__('Red Hat Insights Data and Application Security')}
+        </a>
+      </Text>
+    </div>
+  );
+};
 
 export default PageDescription;

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/PageDescription/__tests__/PageDescription.test.js

@@ -1,11 +1,66 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
+import React from 'react';
+import { Provider } from 'react-redux';
+import configureMockStore from 'redux-mock-store';
+import thunk from 'redux-thunk';
+import { screen, render } from '@testing-library/react';
 import { PageDescription } from '../PageDescription';
 
-const fixtures = {
-  'render without Props': {},
-};
+jest.mock('react-intl', () => {
+  const actualReactIntl = jest.requireActual('react-intl');
+  return {
+    ...actualReactIntl,
+    FormattedMessage: ({ defaultMessage, id }) => defaultMessage || id,
+    useIntl: () => ({
+      formatMessage: ({ defaultMessage, id }) => defaultMessage || id,
+    }),
+  };
+});
+const middlewares = [thunk];
+const mockStore = configureMockStore(middlewares);
 
 describe('PageDescription', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(PageDescription, fixtures));
+  test('when subscription connection is enabled', () => {
+    const renderOptions = {
+      API: {
+        INVENTORY_SETTINGS: {
+          response: { subscriptionConnectionEnabled: true },
+        },
+      },
+    };
+    const store = mockStore(renderOptions);
+
+    render(
+      <Provider store={store}>
+        <PageDescription />
+      </Provider>
+    );
+    expect(
+      screen.queryAllByText(
+        /To enable this reporting for all Foreman organizations/
+      )
+    ).toHaveLength(1);
+  });
+
+  test('when subscription connection is not enabled', () => {
+    const renderOptions = {
+      API: {
+        INVENTORY_SETTINGS: {
+          response: { subscriptionConnectionEnabled: false },
+        },
+      },
+    };
+    const store = mockStore(renderOptions);
+
+    render(
+      <Provider store={store}>
+        <PageDescription />
+      </Provider>
+    );
+
+    expect(
+      screen.queryAllByText(
+        /To enable this reporting for all Foreman organizations/
+      )
+    ).toHaveLength(0);
+  });
 });

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

@@ -1,13 +1,25 @@
 import React from 'react';
+import { useSelector } from 'react-redux';
 import SyncButton from '../SyncButton';
 import CloudConnectorButton from '../CloudConnectorButton';
 import './toolbarButtons.scss';
+import { selectSubscriptionConnectionEnabled } from '../../../InventorySettings/InventorySettingsSelectors';
 
-const ToolbarButtons = () => (
-  <div className="inventory_toolbar_buttons">
-    <CloudConnectorButton />
-    <SyncButton />
-  </div>
-);
+const ToolbarButtons = () => {
+  const subscriptionConnectionEnabled = useSelector(
+    selectSubscriptionConnectionEnabled
+  );
+
+  if (!subscriptionConnectionEnabled) {
+    return null;
+  }
+
+  return (
+    <div className="inventory_toolbar_buttons">
+      <CloudConnectorButton />
+      <SyncButton />
+    </div>
+  );
+};
 
 export default ToolbarButtons;

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/ToolbarButtons/__tests__/ToolbarButtons.test.js

@@ -1,12 +1,60 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-
+import React from 'react';
+import { Provider } from 'react-redux';
+import configureMockStore from 'redux-mock-store';
+import thunk from 'redux-thunk';
+import { screen, render } from '@testing-library/react';
 import ToolbarButtons from '../ToolbarButtons';
 
-const fixtures = {
-  'render without Props': {},
+const middlewares = [thunk];
+const mockStore = configureMockStore(middlewares);
+
+const renderOptions = {
+  API: {
+    INVENTORY_SETTINGS: {
+      response: { subscriptionConnectionEnabled: true },
+    },
+  },
+  ForemanRhCloud: {
+    inventoryUpload: {
+      accountsList: {
+        CloudConnectorStatus: {},
+      },
+    },
+  },
 };
 
 describe('ToolbarButtons', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(ToolbarButtons, fixtures));
+  test('when subscription connection is enabled', () => {
+    const store = mockStore(renderOptions);
+
+    render(
+      <Provider store={store}>
+        <ToolbarButtons />
+      </Provider>
+    );
+    expect(screen.queryAllByText('Configure cloud connector')).toHaveLength(1);
+    expect(screen.queryAllByText('Sync all inventory status')).toHaveLength(1);
+  });
+
+  test('when subscription connection is not enabled', () => {
+    const localSetting = {
+      API: {
+        INVENTORY_SETTINGS: {
+          response: { subscriptionConnectionEnabled: false },
+        },
+      },
+    };
+    const store = mockStore({
+      ...renderOptions,
+      ...localSetting,
+    });
+
+    render(
+      <Provider store={store}>
+        <ToolbarButtons />
+      </Provider>
+    );
+    expect(screen.queryAllByText('Configure cloud connector')).toHaveLength(0);
+    expect(screen.queryAllByText('Sync all inventory status')).toHaveLength(0);
+  });
 });

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

@@ -4,6 +4,7 @@ export const exitCode = 'exit 0';
 export const logs = ['No running process'];
 export const completed = 0;
 export const restartProcess = noop;
+export const downloadReports = noop;
 export const error = null;
 export const scheduled = '2019-08-21T16:14:16.520+03:00';
 export const props = {
@@ -11,6 +12,7 @@ export const props = {
   logs,
   completed,
   restartProcess,
+  downloadReports,
   error,
   scheduled,
 };

data/webpack/ForemanInventoryUpload/Components/ReportGenerate/ReportGenerate.js

@@ -12,6 +12,8 @@ const ReportGenerate = ({
   completed,
   error,
   restartProcess,
+  downloadReports,
+  downloadButtonDisabled,
   toggleFullScreen,
   scheduled,
 }) => (
@@ -19,6 +21,8 @@ const ReportGenerate = ({
     <TabHeader
       exitCode={exitCode}
       onRestart={restartProcess}
+      onDownload={downloadReports}
+      downloadButtonDisabled={downloadButtonDisabled}
       toggleFullScreen={toggleFullScreen}
     />
     <TabBody
@@ -40,6 +44,8 @@ ReportGenerate.propTypes = {
   completed: PropTypes.number,
   error: PropTypes.string,
   restartProcess: PropTypes.func,
+  downloadReports: PropTypes.func,
+  downloadButtonDisabled: PropTypes.func,
   toggleFullScreen: PropTypes.func,
   scheduled: PropTypes.string,
 };
@@ -50,6 +56,8 @@ ReportGenerate.defaultProps = {
   completed: 0,
   error: null,
   restartProcess: noop,
+  downloadReports: noop,
+  downloadButtonDisabled: noop,
   toggleFullScreen: noop,
   scheduled: null,
 };

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

@@ -5,8 +5,9 @@ exports[`ReportGenerate rendering render with Props 1`] = `
   className="report-generate"
 >
   <TabHeader
+    downloadButtonDisabled={[Function]}
     exitCode="exit 0"
-    onDownload={null}
+    onDownload={[Function]}
     onRestart={[Function]}
     toggleFullScreen={[Function]}
   />
@@ -29,8 +30,9 @@ exports[`ReportGenerate rendering render without Props 1`] = `
   className="report-generate"
 >
   <TabHeader
+    downloadButtonDisabled={[Function]}
     exitCode=""
-    onDownload={null}
+    onDownload={[Function]}
     onRestart={[Function]}
     toggleFullScreen={[Function]}
   />

data/webpack/ForemanInventoryUpload/Components/ReportUpload/ReportUpload.js

@@ -10,16 +10,11 @@ const ReportUpload = ({
   exitCode,
   logs,
   completed,
-  downloadReports,
   error,
   toggleFullScreen,
 }) => (
   <TabContainer className="report-upload">
-    <TabHeader
-      exitCode={exitCode}
-      onDownload={downloadReports}
-      toggleFullScreen={toggleFullScreen}
-    />
+    <TabHeader exitCode={exitCode} toggleFullScreen={toggleFullScreen} />
     <TabBody
       exitCode={exitCode}
       logs={logs}
@@ -36,7 +31,6 @@ ReportUpload.propTypes = {
     PropTypes.string,
   ]),
   completed: PropTypes.number,
-  downloadReports: PropTypes.func,
   error: PropTypes.string,
   toggleFullScreen: PropTypes.func,
 };
@@ -45,7 +39,6 @@ ReportUpload.defaultProps = {
   exitCode: '',
   logs: null,
   completed: 0,
-  downloadReports: noop,
   error: null,
   toggleFullScreen: noop,
 };

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

@@ -5,8 +5,9 @@ exports[`ReportUpload rendering render with Props 1`] = `
   className="report-upload"
 >
   <TabHeader
+    downloadButtonDisabled={[Function]}
     exitCode="exit 0"
-    onDownload={[Function]}
+    onDownload={null}
     onRestart={null}
     toggleFullScreen={[Function]}
   />
@@ -29,8 +30,9 @@ exports[`ReportUpload rendering render without Props 1`] = `
   className="report-upload"
 >
   <TabHeader
+    downloadButtonDisabled={[Function]}
     exitCode=""
-    onDownload={[Function]}
+    onDownload={null}
     onRestart={null}
     toggleFullScreen={[Function]}
   />

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

@@ -1,45 +1,63 @@
 import React from 'react';
+import { useSelector } from 'react-redux';
 import PropTypes from 'prop-types';
 import { Grid, Button, Icon } from 'patternfly-react';
 import { noop } from 'foremanReact/common/helpers';
 import { sprintf, translate as __ } from 'foremanReact/common/I18n';
+import { selectSubscriptionConnectionEnabled } from '../InventorySettings/InventorySettingsSelectors';
 import { isExitCodeLoading } from '../../ForemanInventoryHelpers';
 import './tabHeader.scss';
 
-const TabHeader = ({ exitCode, onRestart, onDownload, toggleFullScreen }) => (
-  <Grid.Row className="tab-header">
-    <Grid.Col sm={6}>
-      <p>{sprintf(__('Exit Code: %s'), exitCode)}</p>
-    </Grid.Col>
-    <Grid.Col sm={6}>
-      <div className="tab-action-buttons">
-        {onRestart ? (
-          <Button
-            bsStyle="primary"
-            onClick={onRestart}
-            disabled={isExitCodeLoading(exitCode)}
-          >
-            {__('Generate and upload report')}
-          </Button>
-        ) : null}
-        {onDownload ? (
-          <Button onClick={onDownload}>
-            {__('Download Report')} <Icon name="download" />
+const TabHeader = ({
+  exitCode,
+  onRestart,
+  onDownload,
+  downloadButtonDisabled,
+  toggleFullScreen,
+}) => {
+  const subscriptionConnectionEnabled = useSelector(
+    selectSubscriptionConnectionEnabled
+  );
+  const buttonGenerateLabel = subscriptionConnectionEnabled
+    ? __('Generate and upload report')
+    : __('Generate report');
+
+  return (
+    <Grid.Row className="tab-header">
+      <Grid.Col sm={6}>
+        <p>{sprintf(__('Exit Code: %s'), exitCode)}</p>
+      </Grid.Col>
+      <Grid.Col sm={6}>
+        <div className="tab-action-buttons">
+          {onRestart ? (
+            <Button
+              bsStyle="primary"
+              onClick={onRestart}
+              disabled={isExitCodeLoading(exitCode)}
+            >
+              {buttonGenerateLabel}
+            </Button>
+          ) : null}
+          {onDownload ? (
+            <Button onClick={onDownload} disabled={downloadButtonDisabled()}>
+              {__('Download Report')} <Icon name="download" />
+            </Button>
+          ) : null}
+          <Button onClick={toggleFullScreen}>
+            {__('Full Screen')}
+            <Icon name="arrows-alt" />
           </Button>
-        ) : null}
-        <Button onClick={toggleFullScreen}>
-          {__('Full Screen')}
-          <Icon name="arrows-alt" />
-        </Button>
-      </div>
-    </Grid.Col>
-  </Grid.Row>
-);
+        </div>
+      </Grid.Col>
+    </Grid.Row>
+  );
+};
 
 TabHeader.propTypes = {
   onRestart: PropTypes.func,
   onDownload: PropTypes.func,
   exitCode: PropTypes.string,
+  downloadButtonDisabled: PropTypes.func,
   toggleFullScreen: PropTypes.func,
 };
 
@@ -47,6 +65,7 @@ TabHeader.defaultProps = {
   onRestart: null,
   exitCode: '',
   onDownload: null,
+  downloadButtonDisabled: noop,
   toggleFullScreen: noop,
 };