foreman_rh_cloud 12.1.4 → 12.2.0

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

@@ -42,7 +42,7 @@ class CloudRequestForwarderTest < ActiveSupport::TestCase
     }
 
     paths.each do |key, value|
-      actual_params = @forwarder.path_params(key, generate_certs_hash)
+      actual_params = @forwarder.path_params(key)
       assert_equal value, actual_params[:url]
     end
   end
@@ -167,7 +167,7 @@ class CloudRequestForwarderTest < ActiveSupport::TestCase
       'action_dispatch.request.query_parameters' => params
     )
 
-    actual = @forwarder.prepare_request_opts(req, 'TEST PAYLOAD', params, generate_certs_hash, @host)
+    actual = @forwarder.prepare_request_opts(req, 'TEST PAYLOAD', params, @host)
 
     assert_match /foo/, actual[:headers][:user_agent]
     assert_match /bar/, actual[:headers][:user_agent]
@@ -192,7 +192,7 @@ class CloudRequestForwarderTest < ActiveSupport::TestCase
       'action_dispatch.request.query_parameters' => params
     )
 
-    actual = @forwarder.prepare_request_opts(req, 'TEST PAYLOAD', params, generate_certs_hash, @host)
+    actual = @forwarder.prepare_request_opts(req, 'TEST PAYLOAD', params, @host)
 
     assert_match /text\/html/, actual[:headers][:content_type]
   end

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

@@ -0,0 +1,20 @@
+import React from 'react';
+import { useParams } from 'react-router-dom';
+import { ScalprumComponent, ScalprumProvider } from '@scalprum/react-core';
+import { providerOptions } from '../common/ScalprumModule/ScalprumContext';
+
+const CveDetailsPage = () => {
+  const { cveId } = useParams();
+  const scope = 'vulnerability';
+  const module = './CveDetailPage';
+
+  return (
+    <ScalprumProvider {...providerOptions}>
+      <div className="rh-cloud-cve-details-page">
+        <ScalprumComponent scope={scope} module={module} cveId={cveId} />
+      </div>
+    </ScalprumProvider>
+  );
+};
+
+export default CveDetailsPage;

data/webpack/CveDetailsPage/CveDetailsPage.test.js

@@ -0,0 +1,31 @@
+import React from 'react';
+import { render } from '@testing-library/react';
+import '@testing-library/jest-dom';
+import CveDetailsPage from './CveDetailsPage';
+
+// Mock react-router-dom
+jest.mock('react-router-dom', () => ({
+  useParams: jest.fn(() => ({ cveId: 'CVE-2021-1234' })),
+}));
+
+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('CveDetailsPage component', () => {
+  it('renders the container with correct class', () => {
+    const { container } = render(<CveDetailsPage />);
+    expect(
+      container.querySelector('.rh-cloud-cve-details-page')
+    ).toBeTruthy();
+  });
+
+  it('passes cveId from URL params to ScalprumComponent', () => {
+    const { getByTestId } = render(<CveDetailsPage />);
+    const mockComponent = getByTestId('mock-scalprum-component');
+    expect(mockComponent.textContent).toContain('CVE-2021-1234');
+  });
+});

data/webpack/CveDetailsPage/index.js

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

data/webpack/ForemanColumnExtensions/index.js

@@ -1,19 +1,15 @@
 import React from 'react';
+import { ScalprumComponent, ScalprumProvider } from '@scalprum/react-core';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { propsToCamelCase } from 'foremanReact/common/helpers';
+import { CVECountCell } from '../InsightsVulnerabilityHostIndexExtensions/CVECountCell';
+import { providerOptions } from '../common/ScalprumModule/ScalprumContext';
 
-const RecommendationsCell = hostDetails => {
+const HostedRecommendationsCell = hostDetails => {
   const insightsAttributes = propsToCamelCase(
     // eslint-disable-next-line camelcase
     hostDetails?.insights_attributes ?? {}
   );
-  // Local insights advisor
-  if (insightsAttributes.useLocalAdvisorEngine) {
-    // TODO: Replace this placeholder with the actual local advisor integration
-    return <span>Local advisor placeholder</span>;
-  }
-
-  // Hosted insights advisor
   const { insightsHitsCount: hitsCount } = insightsAttributes;
   if (hitsCount === undefined || hitsCount === null) return '—';
   const hostname = hostDetails?.name;
@@ -22,6 +18,38 @@ const RecommendationsCell = hostDetails => {
   return <a href={hitsUrl}>{hitsCount}</a>;
 };
 
+const IopRecommendationsCell = hostDetails => {
+  const scope = 'advisor';
+  const module = './RecommendationsCellWrapped';
+
+  return (
+    <span className="rh-cloud-insights-recommendations-cell">
+      <ScalprumComponent scope={scope} module={module} />
+    </span>
+  );
+};
+
+const IopRecommendationsCellWrapped = hostDetails => (
+  <ScalprumProvider {...providerOptions}>
+    <IopRecommendationsCell hostDetails={hostDetails} />
+  </ScalprumProvider>
+);
+
+const RecommendationsCell = hostDetails => {
+  const insightsAttributes = propsToCamelCase(
+    // eslint-disable-next-line camelcase
+    hostDetails?.insights_attributes ?? {}
+  );
+
+  return insightsAttributes.useLocalAdvisorEngine ? (
+    <IopRecommendationsCellWrapped hostDetails={hostDetails} />
+  ) : (
+    <HostedRecommendationsCell hostDetails={hostDetails} />
+  );
+};
+
+const insightsCategoryName = __('Insights');
+
 const hostsIndexColumnExtensions = [
   {
     columnName: 'insights_recommendations_count',
@@ -29,13 +57,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>{' '}