foreman_rh_cloud 11.4.0 → 12.1.0

data/webpack/ForemanInventoryUpload/Components/InventorySettings/MinimalInventoryDropdown.js

@@ -0,0 +1,107 @@
+import React, { useState } from 'react';
+import PropTypes from 'prop-types';
+import { useDispatch, useSelector } from 'react-redux';
+import { translate as __ } from 'foremanReact/common/I18n';
+import {
+  Dropdown,
+  DropdownItem,
+  DropdownList,
+  MenuToggle,
+} from '@patternfly/react-core';
+
+import { childSettingsDict } from './AdvancedSetting/AdvancedSettingsConstants';
+import { setSetting } from './InventorySettingsActions';
+
+import { selectInsightsMinimalDataCollection } from './InventorySettingsSelectors';
+
+const MinimalInventoryDropdown = ({ setChosenValue }) => {
+  const dispatch = useDispatch();
+  const [isOpen, setIsOpen] = useState(false);
+  const dropdownValues = {
+    minimal: {
+      title: __('Minimal data collection'),
+      description: __(
+        'Only send the minimum required data to Red Hat cloud, obfuscation settings are disabled'
+      ),
+    },
+    optional: {
+      title: __('Optional data collection'),
+      description: __(
+        'Send additional data to enhance Insights services, as per the settings'
+      ),
+    },
+  };
+  const valueToBoolean = {
+    minimal: true,
+    optional: false,
+  };
+  const currentSettingBoolean = useSelector(
+    selectInsightsMinimalDataCollection
+  );
+  const currentDropdownValue = currentSettingBoolean ? 'minimal' : 'optional';
+  const onToggleClick = () => {
+    setIsOpen(!isOpen);
+  };
+  const onSelect = (_event, value) => {
+    setIsOpen(false);
+    setChosenValue(value);
+
+    dispatch(
+      setSetting({
+        setting: 'insights_minimal_data_collection',
+        value: valueToBoolean[value],
+      })
+    );
+
+    if (value === 'minimal') {
+      // If user wants to move to minimal data collection, turn on all related settings.
+      // These will be overridden by insights_minimal_data_collection anyway, but this takes care of the visuals.
+      Object.values(childSettingsDict).forEach(setting => {
+        dispatch(
+          setSetting({
+            setting: setting.name,
+            value: true,
+          })
+        );
+      });
+    }
+  };
+  return (
+    <Dropdown
+      isOpen={isOpen}
+      onSelect={onSelect}
+      onOpenChange={val => setIsOpen(val)}
+      toggle={toggleRef => (
+        <MenuToggle
+          ref={toggleRef}
+          isFullWidth
+          onClick={onToggleClick}
+          isExpanded={isOpen}
+        >
+          {dropdownValues[currentDropdownValue].title}
+        </MenuToggle>
+      )}
+      shouldFocusToggleOnSelect
+    >
+      <div style={{ maxWidth: '28em' }}>
+        <DropdownList>
+          {Object.entries(dropdownValues).map(([value, item]) => (
+            <DropdownItem
+              value={value}
+              key={value}
+              description={item.description}
+            >
+              {item.title}
+            </DropdownItem>
+          ))}
+        </DropdownList>
+      </div>
+    </Dropdown>
+  );
+};
+
+MinimalInventoryDropdown.propTypes = {
+  setChosenValue: PropTypes.func.isRequired,
+};
+
+export default MinimalInventoryDropdown;

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

@@ -1,12 +1,19 @@
 import React from 'react';
 import { noop } from 'foremanReact/common/helpers';
+import { translate as __ } from 'foremanReact/common/I18n';
 
 export const props = {
   items: [
     {
       icon: 'some-icon',
-      name: 'some-name',
-      component: () => <p>test</p>,
+      name: __('Generating'),
+      component: () => <p>test1</p>,
+      onClick: noop,
+    },
+    {
+      icon: 'other-icon',
+      name: __('Uploading'),
+      component: () => <p>test2</p>,
       onClick: noop,
     },
   ],

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

@@ -1,4 +1,5 @@
 import React from 'react';
+import { useSelector } from 'react-redux';
 import PropTypes from 'prop-types';
 import {
   TabContainer,
@@ -9,8 +10,10 @@ import {
   Icon,
   noop,
 } from 'patternfly-react';
+import { translate as __ } from 'foremanReact/common/I18n';
 import './navContainer.scss';
 import FullScreenModal from '../FullScreenModal';
+import { selectSubscriptionConnectionEnabled } from '../InventorySettings/InventorySettingsSelectors';
 
 const NavContainer = ({
   items,
@@ -18,8 +21,13 @@ const NavContainer = ({
   toggleFullScreen,
   terminalProps,
 }) => {
+  const subscriptionConnectionEnabled = useSelector(
+    selectSubscriptionConnectionEnabled
+  );
   const navItems = items.map((item, index) => {
     const { name, icon, onClick } = item;
+    if (name === __('Uploading') && !subscriptionConnectionEnabled) return null;
+
     return (
       <NavItem
         key={index}

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/PageHeader.scss

@@ -12,18 +12,18 @@
       font-size: 16px;
     }
 
-    .pf-c-dropdown__toggle {
+    .pf-v5-c-dropdown__toggle {
       margin-right: -20px;
     }
 
     .settings-alert {
       margin-bottom: 30px;
-      .pf-c-alert {
+      .pf-v5-c-alert {
         margin-bottom: 5px;
 
-        .pf-c-alert__title {
-          font-size: var(--pf-global--FontSize--md);
-          line-height: var(--pf-global--LineHeight--md);
+        .pf-v5-c-alert__title {
+          font-size: var(--pf-v5-global--FontSize--md);
+          line-height: var(--pf-v5-global--LineHeight--md);
         }
       }
     }

data/webpack/ForemanInventoryUpload/Components/PageHeader/PageTitle.js

@@ -1,12 +1,11 @@
 import React, { useState } from 'react';
+import { Grid, GridItem } from '@patternfly/react-core';
 import {
   Dropdown,
   DropdownItem,
   KebabToggle,
   DropdownPosition,
-  Grid,
-  GridItem,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import Head from 'foremanReact/components/Head';
 import {
   INVENTORY_PAGE_TITLE,
@@ -58,7 +57,9 @@ const PageTitle = () => {
           className="title-dropdown"
           onSelect={() => setIsDropdownOpen(false)}
           toggle={
-            <KebabToggle onToggle={isOpen => setIsDropdownOpen(isOpen)} />
+            <KebabToggle
+              onToggle={(_event, isOpen) => setIsDropdownOpen(isOpen)}
+            />
           }
           isOpen={isDropdownOpen}
           isPlain

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/CloudPingModal/index.js

@@ -2,7 +2,11 @@
 import React, { useCallback, useEffect, useState } from 'react';
 import PropTypes from 'prop-types';
 import { useDispatch, useSelector } from 'react-redux';
-import { Table, TableBody, TableHeader } from '@patternfly/react-table';
+import {
+  Table,
+  TableBody,
+  TableHeader,
+} from '@patternfly/react-table/deprecated';
 import {
   Card,
   CardTitle,
@@ -11,6 +15,7 @@ import {
   ModalVariant,
   Spinner,
   Text,
+  Icon,
 } from '@patternfly/react-core';
 import {
   CheckCircleIcon,
@@ -108,8 +113,18 @@ const CloudPingModal = ({ title, isOpen, toggle }) => {
 
 const StatusIcon = ({ isPending, authStatus }) => {
   if (isPending) return <Spinner size="sm" />;
-  if (authStatus.success) return <CheckCircleIcon color="green" />;
-  if (authStatus.error) return <ExclamationCircleIcon color="red" />;
+  if (authStatus.success)
+    return (
+      <Icon color="green">
+        <CheckCircleIcon />
+      </Icon>
+    );
+  if (authStatus.error)
+    return (
+      <Icon color="red">
+        <ExclamationCircleIcon />
+      </Icon>
+    );
   return <Spinner size="sm" />;
 };
 

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/SyncButton/SyncButton.js

@@ -22,7 +22,6 @@ class SyncButton extends React.Component {
         <Button
           className="sync_button"
           onClick={handleClick}
-          size="lg"
           isDisabled={status === STATUS.PENDING}
           variant="secondary"
         >

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

@@ -6,7 +6,6 @@ exports[`SyncButton rendering render with Props 1`] = `
     className="sync_button"
     isDisabled={false}
     onClick={[Function]}
-    size="lg"
     variant="secondary"
   >
      Sync all inventory status

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);
+  });
 });