foreman_rh_cloud 12.1.4 → 12.1.5

data/test/unit/services/foreman_rh_cloud/insights_api_forwarder_test.rb

@@ -0,0 +1,176 @@
+require 'test_plugin_helper'
+require 'puma/null_io'
+
+class UIRequestForwarderTest < ActiveSupport::TestCase
+  include MockCerts
+
+  setup do
+    @forwarder = ::ForemanRhCloud::InsightsApiForwarder.new
+    @user = FactoryBot.build(:user)
+    @organization = FactoryBot.build(:organization)
+    @location = FactoryBot.build(:location)
+
+    setup_certs_expectation do
+      @forwarder.stubs(:foreman_certificates)
+    end
+
+    ForemanRhCloud.stubs(:cert_base_url).returns('https://cert.cloud.example.com')
+  end
+
+  test 'should scope GET requests with proper tags' do
+    user_agent = { :foo => :bar }
+    params = {}
+
+    req = ActionDispatch::Request.new(
+      'REQUEST_URI' => '/api/vulnerability/v1/cves/abc-123/affected_systems',
+      'REQUEST_METHOD' => 'GET',
+      'HTTP_USER_AGENT' => user_agent,
+      'rack.input' => ::Puma::NullIO.new,
+      'action_dispatch.request.query_parameters' => params
+    )
+
+    ::ForemanRhCloud::TagsAuth.any_instance.expects(:update_tag)
+    @forwarder.expects(:execute_cloud_request).with do |actual_params|
+      actual = actual_params[:headers][:params]
+      assert_equal "U:\"#{@user.login}\"O:\"#{@organization.name}\"L:\"#{@location.name}\"", tag_value(actual.find { |param| param[0] == :tag && tag_name(param[1]) =~ /#{ForemanRhCloud::TagsAuth::TAG_NAME}/ }[1])
+      true
+    end
+
+    @forwarder.forward_request(req, '/api/vulnerability/v1/cves/abc-123/affected_systems', 'test_controller', @user, @organization, @location)
+
+    # This test asserts the parameters that are sent to the execute_cloud_request method.
+    # This is done by setting the expectation before the actual call.
+  end
+
+  test 'should not scope GET requests for unknown uris' do
+    user_agent = { :foo => :bar }
+    params = {}
+
+    req = ActionDispatch::Request.new(
+      'REQUEST_URI' => '/api/vulnerability/foo/bar',
+      'REQUEST_METHOD' => 'GET',
+      'HTTP_USER_AGENT' => user_agent,
+      'rack.input' => ::Puma::NullIO.new,
+      'action_dispatch.request.query_parameters' => params
+    )
+
+    ::ForemanRhCloud::TagsAuth.any_instance.expects(:update_tag).never
+    @forwarder.expects(:execute_cloud_request).with do |actual_params|
+      actual = actual_params[:headers][:params]
+      assert_equal 0, actual.count
+      true
+    end
+
+    @forwarder.forward_request(req, '/api/vulnerability/foo/bar', 'test_controller', @user, @organization, @location)
+
+    # This test asserts the parameters that are sent to the execute_cloud_request method.
+    # This is done by setting the expectation before the actual call.
+  end
+
+  test 'should merge URI params in GET requests' do
+    user_agent = { :foo => :bar }
+    params = { :page => 5, :per_page => 42 }
+
+    req = ActionDispatch::Request.new(
+      'REQUEST_URI' => '/api/vulnerability/v1/cves/abc-123/affected_systems',
+      'REQUEST_METHOD' => 'GET',
+      'HTTP_USER_AGENT' => user_agent,
+      'rack.input' => ::Puma::NullIO.new,
+      'action_dispatch.request.query_parameters' => params
+    )
+
+    ::ForemanRhCloud::TagsAuth.any_instance.expects(:update_tag)
+    @forwarder.expects(:execute_cloud_request).with do |actual_params|
+      actual = actual_params[:headers][:params]
+      assert_equal "U:\"#{@user.login}\"O:\"#{@organization.name}\"L:\"#{@location.name}\"", tag_value(actual.find { |param| param[0] == :tag && tag_name(param[1]) =~ /#{ForemanRhCloud::TagsAuth::TAG_NAME}/ }[1])
+      assert_equal 5, actual.find { |param| param[0] == :page }[1]
+      assert_equal 42, actual.find { |param| param[0] == :per_page }[1]
+      true
+    end
+
+    @forwarder.forward_request(req, '/api/vulnerability/v1/cves/abc-123/affected_systems', 'test_controller', @user, @organization, @location)
+    # This test asserts the parameters that are sent to the execute_cloud_request method.
+    # This is done by setting the expectation before the actual call.
+  end
+
+  test 'should not scope POST requests' do
+    post_data = 'Random POST data'
+    req = ActionDispatch::Request.new(
+      'REQUEST_URI' => '/foo/bar',
+      'REQUEST_METHOD' => 'POST',
+      'rack.input' => ::Puma::NullIO.new,
+      'RAW_POST_DATA' => post_data
+    )
+
+    ::ForemanRhCloud::TagsAuth.any_instance.expects(:update_tag).never
+    @forwarder.expects(:execute_cloud_request).with do |actual_params|
+      actual = actual_params[:headers][:params]
+      assert_equal 0, actual.count
+      true
+    end
+
+    @forwarder.forward_request(req, '/api/vulnerability/v1/cves', 'test_controller', @user, @organization, @location)
+
+    # This test asserts the parameters that are sent to the execute_cloud_request method.
+    # This is done by setting the expectation before the actual call.
+  end
+
+  test 'should not scope PUT requests' do
+    put_data = 'Random PUT data'
+    req = ActionDispatch::Request.new(
+      'REQUEST_URI' => '/foo/bar',
+      'REQUEST_METHOD' => 'PUT',
+      'rack.input' => ::Puma::NullIO.new,
+      'RAW_POST_DATA' => put_data
+    )
+
+    ::ForemanRhCloud::TagsAuth.any_instance.expects(:update_tag).never
+    @forwarder.expects(:execute_cloud_request).with do |actual_params|
+      actual = actual_params[:headers][:params]
+      assert_equal 0, actual.count
+      true
+    end
+
+    @forwarder.forward_request(req, '/api/vulnerability/v1/cves', 'test_controller', @user, @organization, @location)
+
+    # This test asserts the parameters that are sent to the execute_cloud_request method.
+    # This is done by setting the expectation before the actual call.
+  end
+
+  test 'should not scope PATCH requests' do
+    post_data = 'Random PATCH data'
+    req = ActionDispatch::Request.new(
+      'REQUEST_URI' => '/foo/bar',
+      'REQUEST_METHOD' => 'PATCH',
+      'rack.input' => ::Puma::NullIO.new,
+      'RAW_POST_DATA' => post_data,
+      "action_dispatch.request.path_parameters" => { :format => "json" }
+    )
+
+    ::ForemanRhCloud::TagsAuth.any_instance.expects(:update_tag).never
+    @forwarder.expects(:execute_cloud_request).with do |actual_params|
+      actual = actual_params[:headers][:params]
+      assert_equal 0, actual.count
+      true
+    end
+
+    @forwarder.forward_request(req, '/api/vulnerability/v1/cves', 'test_controller', @user, @organization, @location)
+
+    # This test asserts the parameters that are sent to the execute_cloud_request method.
+    # This is done by setting the expectation before the actual call.
+  end
+
+  def tag_value(param_value)
+    return param_value unless param_value.is_a?(String)
+
+    tag_string = CGI.unescape(param_value)
+    tag_string.split('=')[1]
+  end
+
+  def tag_name(param_value)
+    return param_value unless param_value.is_a?(String)
+
+    tag_string = CGI.unescape(param_value)
+    tag_string.split('=')[0]
+  end
+end

data/test/unit/services/foreman_rh_cloud/tags_auth_test.rb

@@ -0,0 +1,29 @@
+require 'test_plugin_helper'
+require 'json'
+
+class TagsAuthTest < ActiveSupport::TestCase
+  setup do
+    @user = FactoryBot.build(:user)
+    @logger = Logger.new(IO::NULL)
+    @org = FactoryBot.build(:organization)
+    @loc = FactoryBot.build(:location)
+    @auth = ::ForemanRhCloud::TagsAuth.new(@user, @org, @loc, @logger)
+  end
+
+  test 'Generates tags update request' do
+    uuid1 = 'test_uuid1'
+    uuid2 = 'test_uuid2'
+
+    @auth.expects(:allowed_hosts).returns([uuid1, uuid2])
+    @auth.expects(:execute_cloud_request).with do |actual_params|
+      actual = JSON.parse(actual_params[:payload])
+      assert_includes actual['host_id_list'], uuid1
+      assert_includes actual['host_id_list'], uuid2
+      assert_equal ForemanRhCloud::TagsAuth::TAG_SHORT_NAME, actual['tags'].first['key']
+      assert_equal ForemanRhCloud::TagsAuth::TAG_NAMESPACE, actual['tags'].first['namespace']
+      assert_equal "U:\"#{@user.login}\"O:\"#{@org.name}\"L:\"#{@loc.name}\"", actual['tags'].first['value']
+    end
+
+    @auth.update_tag
+  end
+end

data/webpack/CVEsHostDetailsTab/CVEsHostDetailsTab.js

@@ -1,17 +1,37 @@
 import React from 'react';
 import PropTypes from 'prop-types';
-import { translate as __ } from 'foremanReact/common/I18n';
+import { ScalprumComponent, ScalprumProvider } from '@scalprum/react-core';
+import { providerOptions } from '../common/ScalprumModule/ScalprumContext';
 
-const CVEsHostDetailsTab = ({ hostName }) => (
-  <div>
-    <h1>
-      {__('CVEs tab for host:')} {hostName}
-    </h1>
-  </div>
-);
+const CVEsHostDetailsTab = ({ systemId }) => {
+  const scope = 'vulnerability';
+  const module = './SystemDetailTable';
+  return (
+    <div className="rh-cloud-insights-vulnerability-host-details-component">
+      <ScalprumComponent scope={scope} module={module} systemId={systemId} />
+    </div>
+  );
+};
 
 CVEsHostDetailsTab.propTypes = {
-  hostName: PropTypes.string.isRequired,
+  systemId: PropTypes.string.isRequired,
+};
+
+const CVEsHostDetailsTabWrapper = ({ response }) => (
+  <ScalprumProvider {...providerOptions}>
+    <CVEsHostDetailsTab
+      // eslint-disable-next-line camelcase
+      systemId={response?.subscription_facet_attributes?.uuid}
+    />
+  </ScalprumProvider>
+);
+
+CVEsHostDetailsTabWrapper.propTypes = {
+  response: PropTypes.shape({
+    subscription_facet_attributes: PropTypes.shape({
+      uuid: PropTypes.string.isRequired,
+    }),
+  }).isRequired,
 };
 
-export default CVEsHostDetailsTab;
+export default CVEsHostDetailsTabWrapper;

data/webpack/CVEsHostDetailsTab/__tests__/CVEsHostDetailsTab.test.js

@@ -1,18 +1,25 @@
 import React from 'react';
-import { render, screen } from '@testing-library/react';
-import CVEsHostDetailsTab from '../CVEsHostDetailsTab';
+import { render } from '@testing-library/react';
+import CVEsHostDetailsTabWrapper from '../CVEsHostDetailsTab';
 
-describe('CVEsHostDetailsTab', () => {
+jest.mock('@scalprum/react-core', () => ({
+  ScalprumComponent: jest.fn(props => (
+    <div data-testid="mock-scalprum-component">{JSON.stringify(props)}</div>
+  )),
+  ScalprumProvider: jest.fn(({ children }) => <div>{children}</div>),
+}));
+
+describe('CVEsHostDetailsTabWrapper', () => {
   it('renders without crashing', () => {
-    render(<CVEsHostDetailsTab hostName="test-host.example.com" />);
+    const { container } = render(
+      <CVEsHostDetailsTabWrapper
+        response={{ subscription_facet_attributes: { uuid: '1-2-3' } }}
+      />
+    );
     expect(
-      screen.getByText('CVEs tab for host: test-host.example.com')
+      container.querySelector(
+        '.rh-cloud-insights-vulnerability-host-details-component'
+      )
     ).toBeTruthy();
   });
-
-  it('renders the host name', () => {
-    const hostName = 'test-host.example.com';
-    render(<CVEsHostDetailsTab hostName={hostName} />);
-    expect(screen.getByText(`CVEs tab for host: ${hostName}`)).toBeTruthy();
-  });
 });

data/webpack/CVEsHostDetailsTab/index.js

@@ -1,3 +1,3 @@
-import CVEsHostDetailsTab from './CVEsHostDetailsTab';
+import CVEsHostDetailsTabWrapper from './CVEsHostDetailsTab';
 
-export default CVEsHostDetailsTab;
+export default CVEsHostDetailsTabWrapper;

data/webpack/ForemanColumnExtensions/index.js

@@ -1,6 +1,7 @@
 import React from 'react';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { propsToCamelCase } from 'foremanReact/common/helpers';
+import { CVECountCell } from '../InsightsVulnerabilityHostIndexExtensions/CVECountCell';
 
 const RecommendationsCell = hostDetails => {
   const insightsAttributes = propsToCamelCase(
@@ -22,6 +23,8 @@ const RecommendationsCell = hostDetails => {
   return <a href={hitsUrl}>{hitsCount}</a>;
 };
 
+const insightsCategoryName = __('Insights');
+
 const hostsIndexColumnExtensions = [
   {
     columnName: 'insights_recommendations_count',
@@ -29,13 +32,20 @@ const hostsIndexColumnExtensions = [
     wrapper: RecommendationsCell,
     weight: 1500,
     isSorted: true,
+    tableName: 'hosts',
+    categoryName: insightsCategoryName,
+    categoryKey: 'insights',
+  },
+  {
+    columnName: 'cves_count',
+    title: __('Total CVEs'),
+    wrapper: hostDetails => <CVECountCell hostDetails={hostDetails} />,
+    weight: 2600,
+    tableName: 'hosts',
+    categoryName: insightsCategoryName,
+    categoryKey: 'insights',
+    isSorted: false,
   },
 ];
 
-hostsIndexColumnExtensions.forEach(column => {
-  column.tableName = 'hosts';
-  column.categoryName = 'Insights';
-  column.categoryKey = 'insights';
-});
-
 export default hostsIndexColumnExtensions;

data/webpack/ForemanInventoryUpload/Components/InventoryFilter/InventoryFilter.js

@@ -26,6 +26,7 @@ const InventoryFilter = ({
       <FormGroup>
         <TextInput
           id="inventory_filter_input"
+          ouiaId="inventory_filter_input"
           value={filterTerm}
           type="text"
           placeholder={__('Filter..')}

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

@@ -8,6 +8,7 @@ exports[`InventoryFilter rendering render with props 1`] = `
     <TextInput
       id="inventory_filter_input"
       onChange={[Function]}
+      ouiaId="inventory_filter_input"
       placeholder="Filter.."
       type="text"
       value="test_filter_term"

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

@@ -68,6 +68,7 @@ const MinimalInventoryDropdown = ({ setChosenValue }) => {
   };
   return (
     <Dropdown
+      ouiaId="inventory-dropdown"
       isOpen={isOpen}
       onSelect={onSelect}
       onOpenChange={val => setIsOpen(val)}
@@ -90,6 +91,7 @@ const MinimalInventoryDropdown = ({ setChosenValue }) => {
               value={value}
               key={value}
               description={item.description}
+              ouiaId={`inventory-dropdownItem-${value}`}
             >
               {item.title}
             </DropdownItem>

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

@@ -26,6 +26,7 @@ const PageTitle = () => {
   const dropdownItems = [
     <DropdownItem
       key="tasks-history-button"
+      ouiaId="tasks-history-button"
       href={getActionsHistoryUrl()}
       target="_blank"
       rel="noopener noreferrer"
@@ -34,13 +35,18 @@ const PageTitle = () => {
     </DropdownItem>,
     <DropdownItem
       key="inventory-documentation-button"
+      ouiaId="inventory-documentation-button"
       href={getInventoryDocsUrl()}
       target="_blank"
       rel="noopener noreferrer"
     >
       {DOCS_BUTTON_TEXT}
     </DropdownItem>,
-    <DropdownItem key="cloud-ping" onClick={togglePingModal}>
+    <DropdownItem
+      key="cloud-ping"
+      ouiaId="dropdownItem-cloud-ping"
+      onClick={togglePingModal}
+    >
       {CLOUD_PING_TITLE}
     </DropdownItem>,
   ];
@@ -55,6 +61,7 @@ const PageTitle = () => {
       <GridItem span={6}>
         <Dropdown
           className="title-dropdown"
+          ouiaId="title-dropdown"
           onSelect={() => setIsDropdownOpen(false)}
           toggle={
             <KebabToggle

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

@@ -25,6 +25,7 @@ exports[`PageTitle rendering render without Props 1`] = `
         Array [
           <DropdownItem
             href="/foreman_tasks/tasks?search=label+%3D+ForemanInventoryUpload%3A%3AAsync%3A%3AGenerateReportJob+or+label+%3D+ForemanInventoryUpload%3A%3AAsync%3A%3AGenerateAllReportsJob&page=1"
+            ouiaId="tasks-history-button"
             rel="noopener noreferrer"
             target="_blank"
           >
@@ -32,6 +33,7 @@ exports[`PageTitle rendering render without Props 1`] = `
           </DropdownItem>,
           <DropdownItem
             href="/links/manual/?root_url=https%3A%2F%2Faccess.redhat.com%2Fdocumentation%2Fen-us%2Fred_hat_insights%2F2023%2Fhtml%2Fred_hat_insights_remediations_guide%2Fhost-communication-with-insights_red-hat-insights-remediation-guide%23uploading-satellite-host-inventory-to-insights_configuring-satellite-cloud-connector"
+            ouiaId="inventory-documentation-button"
             rel="noopener noreferrer"
             target="_blank"
           >
@@ -39,6 +41,7 @@ exports[`PageTitle rendering render without Props 1`] = `
           </DropdownItem>,
           <DropdownItem
             onClick={[Function]}
+            ouiaId="dropdownItem-cloud-ping"
           >
             Connectivity test
           </DropdownItem>,
@@ -47,6 +50,7 @@ exports[`PageTitle rendering render without Props 1`] = `
       isOpen={false}
       isPlain={true}
       onSelect={[Function]}
+      ouiaId="title-dropdown"
       position="right"
       toggle={
         <KebabToggle

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

@@ -26,7 +26,7 @@ export const CloudConnectorButton = ({ status, onClick, jobLink }) => {
           className="cloud-connector-pending-button"
           onMouseEnter={() => setIsPopoverVisible(true)}
         >
-          <Button variant="secondary" isDisabled>
+          <Button variant="secondary" ouiaId="button-in-progress" isDisabled>
             <Spinner size="sm" /> {__('Cloud Connector is in progress')}
           </Button>
         </div>
@@ -36,14 +36,14 @@ export const CloudConnectorButton = ({ status, onClick, jobLink }) => {
 
   if (status === CONNECTOR_STATUS.RESOLVED) {
     return (
-      <Button variant="secondary" onClick={onClick}>
+      <Button variant="secondary" ouiaId="button-reconfigure" onClick={onClick}>
         {__('Reconfigure cloud connector')}
       </Button>
     );
   }
 
   return (
-    <Button variant="secondary" onClick={onClick}>
+    <Button variant="secondary" ouiaId="button-configure" onClick={onClick}>
       {__('Configure cloud connector')}
     </Button>
   );

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

@@ -3,6 +3,7 @@
 exports[`CloudConnectorButton render no cloud connector 1`] = `
 <Button
   onClick={[MockFunction]}
+  ouiaId="button-configure"
   variant="secondary"
 >
   Configure cloud connector
@@ -34,6 +35,7 @@ exports[`CloudConnectorButton render pending connector 1`] = `
   >
     <Button
       isDisabled={true}
+      ouiaId="button-in-progress"
       variant="secondary"
     >
       <Spinner
@@ -49,6 +51,7 @@ exports[`CloudConnectorButton render pending connector 1`] = `
 exports[`CloudConnectorButton render resolved cloud connector 1`] = `
 <Button
   onClick={[MockFunction]}
+  ouiaId="button-reconfigure"
   variant="secondary"
 >
   Reconfigure cloud connector

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

@@ -79,26 +79,32 @@ const CloudPingModal = ({ title, isOpen, toggle }) => {
     <>
       <Modal
         id="cloud-ping-modal"
+        ouiaId="cloud-ping-modal"
         appendTo={document.getElementsByClassName('react-container')[0]}
         variant={ModalVariant.large}
         title={title}
         isOpen={isOpen}
         onClose={toggle}
       >
-        <Card className="certs-status">
+        <Card className="certs-status" ouiaId="card-org-status">
           <CardTitle>{__('Organization status')}</CardTitle>
           <CardBody>
-            <Text>
+            <Text ouiaId="text-description">
               {__('Displays manifest statuses per accessible organizations.')}
             </Text>
             {isPending ? (
               <Spinner size="xl" />
             ) : (
               <>
-                <Text className="pull-right">
+                <Text className="pull-right" ouiaId="text-org-count">
                   {sprintf(__('%s organizations'), rows.length)}
                 </Text>
-                <Table aria-label="Simple Table" cells={['']} rows={rows}>
+                <Table
+                  aria-label="Simple Table"
+                  ouiaId="simple-table"
+                  cells={['']}
+                  rows={rows}
+                >
                   <TableHeader />
                   <TableBody />
                 </Table>{' '}

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

@@ -13,18 +13,18 @@ export const PageDescription = () => {
 
   return (
     <div id="inventory_page_description">
-      <Text>
+      <Text ouiaId="text-cloud-console">
         {__(
           '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>
+      <Text ouiaId="text-inventory-upload">
         {__(
           '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>
       {subscriptionConnectionEnabled && (
-        <Text>
+        <Text ouiaId="text-enable-report">
           <FormattedMessage
             id="enable-upload-hint"
             defaultMessage={__(
@@ -39,7 +39,7 @@ export const PageDescription = () => {
         </Text>
       )}
       {subscriptionConnectionEnabled && (
-        <Text>
+        <Text ouiaId="text-restart-button">
           <FormattedMessage
             id="restart-button-hint"
             defaultMessage={__(
@@ -53,7 +53,7 @@ export const PageDescription = () => {
           />
         </Text>
       )}
-      <Text>
+      <Text ouiaId="text-more-info-subscription">
         {__('For more information about the Subscriptions service, see:')}
         &nbsp;
         <a
@@ -64,7 +64,7 @@ export const PageDescription = () => {
           {__('About subscription watch')}
         </a>
       </Text>
-      <Text>
+      <Text ouiaId="text-more-info-insights">
         {__('For more information about Insights and Cloud Connector, see:')}
         &nbsp;
         <a

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

@@ -22,6 +22,7 @@ export const SettingsWarning = ({
     alerts.push(
       <Alert
         key="auto-upload"
+        ouiaId="auto-upload"
         variant="warning"
         title={__(
           "Cloud Connector has been configured however the inventory auto-upload is disabled, it's recommended to enable it"
@@ -36,6 +37,7 @@ export const SettingsWarning = ({
     alerts.push(
       <Alert
         key="obfuscating-host"
+        ouiaId="obfuscating-host"
         variant="warning"
         title={__(
           "Cloud Connector has been configured however obfuscating host names setting is enabled, it's recommended to disable it"

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

@@ -11,6 +11,7 @@ exports[`SettingsWarning with 2 alerts 1`] = `
       />
     }
     key="auto-upload"
+    ouiaId="auto-upload"
     title="Cloud Connector has been configured however the inventory auto-upload is disabled, it's recommended to enable it"
     variant="warning"
   />
@@ -21,6 +22,7 @@ exports[`SettingsWarning with 2 alerts 1`] = `
       />
     }
     key="obfuscating-host"
+    ouiaId="obfuscating-host"
     title="Cloud Connector has been configured however obfuscating host names setting is enabled, it's recommended to disable it"
     variant="warning"
   />

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

@@ -21,6 +21,7 @@ class SyncButton extends React.Component {
       <React.Fragment>
         <Button
           className="sync_button"
+          ouiaId="sync-button"
           onClick={handleClick}
           isDisabled={status === STATUS.PENDING}
           variant="secondary"

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

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

data/webpack/ForemanInventoryUpload/SubscriptionsPageExtension/InventoryAutoUpload/InventoryAutoUpload.js

@@ -48,6 +48,7 @@ const InventoryAutoUploadSwitcher = ({
               position="right"
             >
               <Button
+                ouiaId="button-advanced-settings"
                 variant="secondary"
                 style={{ fontSize: 'small', marginTop: '-4px' }}
               >
@@ -59,9 +60,10 @@ const InventoryAutoUploadSwitcher = ({
         <br />
         <Grid.Row>
           <Grid.Col sm={12}>
-            <Text component={TextVariants.p}>
+            <Text component={TextVariants.p} ouiaId="text-more-details">
               <InfoAltIcon /> {__('More details can be found in')}{' '}
               <Text
+                ouiaId="text-details-link"
                 component={TextVariants.a}
                 href={foremanUrl('/foreman_rh_cloud/inventory_upload')}
                 target="_blank"