foreman_rh_cloud 12.2.13 → 12.2.15

data/webpack/InsightsCloudSync/Components/InsightsTable/InsightsTable.js

@@ -16,7 +16,7 @@ import TableEmptyState from '../../../common/table/EmptyState';
 import { modifySelectedRows, getSortColumnIndex } from './InsightsTableHelpers';
 import Pagination from './Pagination';
 import './table.scss';
-import { useAdvisorEngineConfig } from '../../../common/Hooks/ConfigHooks';
+import { useIopConfig } from '../../../common/Hooks/ConfigHooks';
 
 const InsightsTable = ({
   page,
@@ -48,17 +48,11 @@ const InsightsTable = ({
     fetchInsights({ page, perPage, query, sortBy, sortOrder });
   }, [hostname]);
 
-  const isLocalAdvisorEngine = useAdvisorEngineConfig();
+  const isIop = useIopConfig();
 
   useEffect(() => {
     setRows(
-      modifySelectedRows(
-        hits,
-        selectedIds,
-        showSelectAllAlert,
-        hideHost,
-        isLocalAdvisorEngine
-      )
+      modifySelectedRows(hits, selectedIds, showSelectAllAlert, hideHost, isIop)
     );
 
     if (hideHost) setColumns(getColumnsWithoutHostname());

data/webpack/InsightsCloudSync/Components/InsightsTable/__tests__/InsightsTable.test.js

@@ -1,13 +1,33 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-
+import React from 'react';
+import { rtlHelpers } from 'foremanReact/common/rtlTestHelpers';
 import InsightsTable from '../InsightsTable';
 import { tableProps } from './fixtures';
 
+jest.mock('foremanReact/Root/Context/ForemanContext', () => ({
+  useForemanContext: () => ({
+    metadata: {
+      foreman_rh_cloud: {
+        iop: true,
+      },
+    },
+  }),
+  useForemanSettings: () => ({
+    perPage: 20,
+  }),
+}));
+
+const { renderWithStore } = rtlHelpers;
+
 const fixtures = {
   'render with Props': tableProps,
 };
 
 describe('InsightsTable', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(InsightsTable, fixtures));
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+
+  it('renders without crashing', () => {
+    renderWithStore(<InsightsTable {...tableProps} />);
+  });
 });

data/webpack/InsightsCloudSync/Components/RemediationModal/RemediationModal.js

@@ -16,7 +16,7 @@ import { modifyRows } from './RemediationHelpers';
 import ModalFooter from './RemediationModalFooter';
 import TableEmptyState from '../../../common/table/EmptyState';
 import './RemediationModal.scss';
-import { useAdvisorEngineConfig } from '../../../common/Hooks/ConfigHooks';
+import { useIopConfig } from '../../../common/Hooks/ConfigHooks';
 
 /* eslint-disable spellcheck/spell-checker */
 
@@ -82,7 +82,7 @@ const RemediationModal = ({
   const [rows, setRows] = React.useState([]);
   const toggleModal = () => setOpen(prevValue => !prevValue);
 
-  const isIop = useAdvisorEngineConfig();
+  const isIop = useIopConfig();
   useEffect(() => {
     // only fetch for Hosted. IoP provides via props.
     if (!isIop && open)

data/webpack/InsightsCloudSync/Components/ToolbarDropdown.js

@@ -8,12 +8,12 @@ import {
 } from '@patternfly/react-core/deprecated';
 import { ExternalLinkAltIcon } from '@patternfly/react-icons';
 import { redHatAdvisorSystems } from '../InsightsCloudSyncHelpers';
-import { useAdvisorEngineConfig } from '../../common/Hooks/ConfigHooks';
+import { useIopConfig } from '../../common/Hooks/ConfigHooks';
 
 const ToolbarDropdown = ({ onRecommendationSync }) => {
   const [isDropdownOpen, setIsDropdownOpen] = useState(false);
-  const isLocalAdvisorEngine = useAdvisorEngineConfig();
-  if (isLocalAdvisorEngine) {
+  const isIop = useIopConfig();
+  if (isIop) {
     return null;
   }
   const dropdownItems = [

data/webpack/InsightsCloudSync/InsightsCloudSync.js

@@ -3,7 +3,7 @@ import PropTypes from 'prop-types';
 import PageLayout from 'foremanReact/routes/common/PageLayout/PageLayout';
 import { ScalprumComponent, ScalprumProvider } from '@scalprum/react-core';
 import InsightsTable from './Components/InsightsTable';
-import { useAdvisorEngineConfig } from '../common/Hooks/ConfigHooks';
+import { useIopConfig } from '../common/Hooks/ConfigHooks';
 import { foremanUrl } from '../ForemanRhCloudHelpers';
 import RemediationModal from './Components/RemediationModal';
 import {
@@ -85,9 +85,9 @@ const IopRecommendationsPageWrapped = props => (
 );
 
 const RecommendationsPage = props => {
-  const isLocalAdvisorEngine = useAdvisorEngineConfig();
+  const isIop = useIopConfig();
 
-  return isLocalAdvisorEngine ? (
+  return isIop ? (
     <IopRecommendationsPageWrapped {...props} />
   ) : (
     <InsightsCloudSync {...props} />

data/webpack/InsightsCloudSync/InsightsCloudSync.test.js

@@ -2,6 +2,16 @@ import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
 import { noop } from 'foremanReact/common/helpers';
 import InsightsCloudSync from './InsightsCloudSync';
 
+jest.mock('foremanReact/Root/Context/ForemanContext', () => ({
+  useForemanContext: () => ({
+    metadata: {
+      foreman_rh_cloud: {
+        iop: true,
+      },
+    },
+  }),
+}));
+
 const fixtures = {
   render: {
     status: 'RESOLVED',

data/webpack/InsightsCloudSync/__snapshots__/InsightsCloudSync.test.js.snap

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`InsightsCloudSync render 1`] = `
-<InsightsCloudSync
+<IopRecommendationsPageWrapped
   fetchInsights={[Function]}
   query=""
   status="RESOLVED"

data/webpack/InsightsHostDetailsTab/NewHostDetailsTab.js

@@ -21,7 +21,7 @@ import {
   selectHits,
 } from '../InsightsCloudSync/Components/InsightsTable/InsightsTableSelectors';
 import { redHatAdvisorSystems } from '../InsightsCloudSync/InsightsCloudSyncHelpers';
-import { useAdvisorEngineConfig } from '../common/Hooks/ConfigHooks';
+import { useIopConfig } from '../common/Hooks/ConfigHooks';
 import { generateRuleUrl } from '../InsightsCloudSync/InsightsCloudSync';
 import { providerOptions } from '../common/ScalprumModule/ScalprumContext';
 
@@ -30,7 +30,7 @@ const NewHostDetailsTab = ({ hostName, router }) => {
   const dispatch = useDispatch();
   const query = useSelector(selectSearch);
   const hits = useSelector(selectHits);
-  const isLocalAdvisorEngine = useAdvisorEngineConfig();
+  const isIop = useIopConfig();
 
   useEffect(() => () => router.replace({ search: null }), [router]);
 
@@ -46,7 +46,7 @@ const NewHostDetailsTab = ({ hostName, router }) => {
     </DropdownItem>,
   ];
 
-  if (hits.length && !isLocalAdvisorEngine) {
+  if (hits.length && !isIop) {
     const { host_uuid: uuid } = hits[0];
     dropdownItems.push(
       <DropdownItem key="insights-advisor-link" ouiaId="insights-advisor-link">
@@ -131,9 +131,9 @@ const IopInsightsTabWrapped = props => (
 );
 
 const InsightsTab = props => {
-  const isLocalIop = useAdvisorEngineConfig();
+  const isIop = useIopConfig();
 
-  return isLocalIop ? (
+  return isIop ? (
     <IopInsightsTabWrapped {...props} />
   ) : (
     <NewHostDetailsTab {...props} />

data/webpack/InsightsVulnerabilityHostIndexExtensions/CVECountCell.js

@@ -4,13 +4,13 @@ import { UnknownIcon } from '@patternfly/react-icons';
 import { Link } from 'react-router-dom';
 import { useAPI } from 'foremanReact/common/hooks/API/APIHooks';
 import { insightsCloudUrl } from '../InsightsCloudSync/InsightsCloudSyncHelpers';
-import { useAdvisorEngineConfig } from '../common/Hooks/ConfigHooks';
+import { useIopConfig } from '../common/Hooks/ConfigHooks';
 
 const vulnerabilityApiPath = path =>
   insightsCloudUrl(`api/vulnerability/v1/${path}`);
 
 export const CVECountCell = ({ hostDetails }) => {
-  const isIopEnabled = useAdvisorEngineConfig();
+  const isIopEnabled = useIopConfig();
 
   // eslint-disable-next-line camelcase
   const uuid = hostDetails?.subscription_facet_attributes?.uuid;

data/webpack/InsightsVulnerabilityHostIndexExtensions/__tests__/CVECountCell.test.js

@@ -1,12 +1,14 @@
 import React from 'react';
-import { render, screen } from '@testing-library/react';
+import { screen } from '@testing-library/react';
+import { rtlHelpers } from 'foremanReact/common/rtlTestHelpers';
 import { API } from 'foremanReact/redux/API';
 import { CVECountCell } from '../CVECountCell';
-import * as ConfigHooks from '../../common/Hooks/ConfigHooks';
 
 jest.mock('foremanReact/redux/API');
 jest.mock('../../common/Hooks/ConfigHooks');
 
+const { renderWithStore } = rtlHelpers;
+
 API.get.mockImplementation(async () => ({
   data: [
     {
@@ -18,17 +20,13 @@ API.get.mockImplementation(async () => ({
 }));
 
 const hostDetailsMock = {
-      name: 'test-host.example.com',
-      subscription_facet_attributes: {
-        uuid: 'test-uuid-123',
-      },
-    };
+  name: 'test-host.example.com',
+  subscription_facet_attributes: {
+    uuid: 'test-uuid-123',
+  },
+};
 
 describe('CVECountCell', () => {
-  beforeEach(() => {
-    ConfigHooks.useAdvisorEngineConfig.mockReturnValue(true);
-  });
-
   afterEach(() => {
     jest.clearAllMocks();
   });
@@ -40,35 +38,92 @@ describe('CVECountCell', () => {
         uuid: null, // no subscription
       },
     };
-    render(<CVECountCell hostDetails={hostDetailsMockIoP} />);
+    renderWithStore(<CVECountCell hostDetails={hostDetailsMockIoP} />, {
+      router: {
+        location: {
+          pathname: '/',
+          search: '',
+          hash: '',
+          query: {},
+        },
+      },
+      API: {
+        ADVISOR_ENGINE_CONFIG: {
+          response: { use_iop_mode: true },
+          status: 'RESOLVED',
+        },
+      },
+    });
     expect(screen.getByRole('img', { hidden: true })).toBeTruthy();
   });
 
   it('renders UnknownIcon when IoP is not enabled', () => {
-    ConfigHooks.useAdvisorEngineConfig.mockReturnValue(false);
-    render(<CVECountCell hostDetails={hostDetailsMock} />);
+    renderWithStore(<CVECountCell hostDetails={hostDetailsMock} />, {
+      router: {
+        location: {
+          pathname: '/',
+          search: '',
+          hash: '',
+          query: {},
+        },
+      },
+      API: {
+        ADVISOR_ENGINE_CONFIG: {
+          response: { use_iop_mode: false },
+          status: 'RESOLVED',
+        },
+      },
+    });
     expect(screen.getByRole('img', { hidden: true })).toBeTruthy();
-    expect(ConfigHooks.useAdvisorEngineConfig).toHaveBeenCalled();
   });
 
   it('renders UnknownIcon when IoP is enabled but CVE API call fails', () => {
-    // Mock successful IoP config
-    ConfigHooks.useAdvisorEngineConfig.mockReturnValue(true);
     // Mock CVE API failure - override the global mock for this test
     API.get.mockImplementationOnce(async () => {
       throw new Error('CVE API call failed');
     });
 
-    render(<CVECountCell hostDetails={hostDetailsMock} />);
+    renderWithStore(<CVECountCell hostDetails={hostDetailsMock} />, {
+      router: {
+        location: {
+          pathname: '/',
+          search: '',
+          hash: '',
+          query: {},
+        },
+      },
+      API: {
+        ADVISOR_ENGINE_CONFIG: {
+          response: { use_iop_mode: true },
+          status: 'RESOLVED',
+        },
+        // Mock the API failure state for the CVE endpoint
+        [`HOST_CVE_COUNT_${hostDetailsMock.subscription_facet_attributes.uuid}`]: {
+          status: 'ERROR',
+          error: 'CVE API call failed',
+        },
+      },
+    });
     // Should render UnknownIcon when CVE API fails
     expect(screen.getByRole('img', { hidden: true })).toBeTruthy();
-    expect(ConfigHooks.useAdvisorEngineConfig).toHaveBeenCalled();
   });
 
   it('renders UnknownIcon when IoP is undefined (API call pending)', () => {
-    ConfigHooks.useAdvisorEngineConfig.mockReturnValue(undefined);
-    render(<CVECountCell hostDetails={hostDetailsMock} />);
+    renderWithStore(<CVECountCell hostDetails={hostDetailsMock} />, {
+      router: {
+        location: {
+          pathname: '/',
+          search: '',
+          hash: '',
+          query: {},
+        },
+      },
+      API: {
+        ADVISOR_ENGINE_CONFIG: {
+          status: 'PENDING',
+        },
+      },
+    });
     expect(screen.getByRole('img', { hidden: true })).toBeTruthy();
-    expect(ConfigHooks.useAdvisorEngineConfig).toHaveBeenCalled();
   });
 });

data/webpack/common/Hooks/ConfigHooks.js

@@ -1,18 +1,5 @@
-import { useAPI } from 'foremanReact/common/hooks/API/APIHooks';
-import {
-  ADVISOR_ENGINE_CONFIG_KEY,
-  ADVISOR_ENGINE_CONFIG_PATH,
-} from '../../InsightsCloudSync/Components/InsightsTable/InsightsTableConstants';
-
-export const useAdvisorEngineConfig = () => {
-  const { response: advisorEngineConfig } = useAPI(
-    'get',
-    ADVISOR_ENGINE_CONFIG_PATH,
-    {
-      key: ADVISOR_ENGINE_CONFIG_KEY,
-    }
-  );
+import { useForemanContext } from 'foremanReact/Root/Context/ForemanContext';
 
+export const useIopConfig = () =>
   // eslint-disable-next-line camelcase
-  return advisorEngineConfig?.use_iop_mode;
-};
+  useForemanContext().metadata?.foreman_rh_cloud?.iop;

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_rh_cloud
 version: !ruby/object:Gem::Version
-  version: 12.2.13
+  version: 12.2.15
 platform: ruby
 authors:
 - Foreman Red Hat Cloud team
@@ -196,7 +196,7 @@ files:
 - lib/foreman_inventory_upload/async/remove_insights_hosts_job.rb
 - lib/foreman_inventory_upload/async/shell_process.rb
 - lib/foreman_inventory_upload/async/single_host_report_job.rb
-- lib/foreman_inventory_upload/async/upload_report_job.rb
+- lib/foreman_inventory_upload/async/upload_report_direct_job.rb
 - lib/foreman_inventory_upload/generators/archived_report.rb
 - lib/foreman_inventory_upload/generators/fact_helpers.rb
 - lib/foreman_inventory_upload/generators/json_stream.rb
@@ -205,7 +205,6 @@ files:
 - lib/foreman_inventory_upload/generators/slice.rb
 - lib/foreman_inventory_upload/generators/tags.rb
 - lib/foreman_inventory_upload/notifications/manifest_import_success_notification_override.rb
-- lib/foreman_inventory_upload/scripts/uploader.sh.erb
 - lib/foreman_rh_cloud.rb
 - lib/foreman_rh_cloud/async/exponential_backoff.rb
 - lib/foreman_rh_cloud/engine.rb
@@ -278,16 +277,18 @@ files:
 - test/jobs/queue_for_upload_job_test.rb
 - test/jobs/remove_insights_hosts_job_test.rb
 - test/jobs/single_host_report_job_test.rb
-- test/jobs/upload_report_job_test.rb
+- test/jobs/upload_report_direct_job_test.rb
 - test/models/insights_client_report_status_test.rb
 - test/test_plugin_helper.rb
 - test/unit/archived_report_generator_test.rb
 - test/unit/fact_helpers_test.rb
+- test/unit/foreman_rh_cloud_iop_metadata_test.rb
 - test/unit/foreman_rh_cloud_self_host_test.rb
 - test/unit/insights_facet_test.rb
 - test/unit/lib/foreman_rh_cloud/registration_manager_extensions_test.rb
 - test/unit/metadata_generator_test.rb
 - test/unit/playbook_progress_generator_test.rb
+- test/unit/rh_cloud_host_test.rb
 - test/unit/rh_cloud_http_proxy_test.rb
 - test/unit/rh_cloud_permissions_test.rb
 - test/unit/services/foreman_rh_cloud/branch_info_test.rb
@@ -441,7 +442,6 @@ files:
 - webpack/ForemanInventoryUpload/Components/PageHeader/PageTitle.js
 - webpack/ForemanInventoryUpload/Components/PageHeader/__tests__/PageHeader.test.js
 - webpack/ForemanInventoryUpload/Components/PageHeader/__tests__/PageTitle.test.js
-- webpack/ForemanInventoryUpload/Components/PageHeader/__tests__/__snapshots__/PageHeader.test.js.snap
 - webpack/ForemanInventoryUpload/Components/PageHeader/__tests__/__snapshots__/PageTitle.test.js.snap
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/CloudConnectorButton/CloudConnectorActions.js
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/CloudConnectorButton/CloudConnectorButton.js
@@ -586,7 +586,6 @@ files:
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/InsightsTable.test.js
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/InsightsTableActions.test.js
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/InsightsTableSelectors.test.js
-- webpack/InsightsCloudSync/Components/InsightsTable/__tests__/__snapshots__/InsightsTable.test.js.snap
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/__snapshots__/InsightsTableActions.test.js.snap
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/__snapshots__/InsightsTableSelectors.test.js.snap
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/fixtures.js
@@ -646,7 +645,6 @@ files:
 - webpack/__mocks__/foremanReact/common/I18n.js
 - webpack/__mocks__/foremanReact/common/MountingService.js
 - webpack/__mocks__/foremanReact/common/helpers.js
-- webpack/__mocks__/foremanReact/common/hooks/API/APIHooks.js
 - webpack/__mocks__/foremanReact/components/ConfirmModal/index.js
 - webpack/__mocks__/foremanReact/components/Head.js
 - webpack/__mocks__/foremanReact/components/Layout/LayoutConstants.js
@@ -703,7 +701,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.3
 specification_version: 4
 summary: Summary of ForemanRhCloud.
 test_files:
@@ -738,16 +736,18 @@ test_files:
 - test/jobs/queue_for_upload_job_test.rb
 - test/jobs/remove_insights_hosts_job_test.rb
 - test/jobs/single_host_report_job_test.rb
-- test/jobs/upload_report_job_test.rb
+- test/jobs/upload_report_direct_job_test.rb
 - test/models/insights_client_report_status_test.rb
 - test/test_plugin_helper.rb
 - test/unit/archived_report_generator_test.rb
 - test/unit/fact_helpers_test.rb
+- test/unit/foreman_rh_cloud_iop_metadata_test.rb
 - test/unit/foreman_rh_cloud_self_host_test.rb
 - test/unit/insights_facet_test.rb
 - test/unit/lib/foreman_rh_cloud/registration_manager_extensions_test.rb
 - test/unit/metadata_generator_test.rb
 - test/unit/playbook_progress_generator_test.rb
+- test/unit/rh_cloud_host_test.rb
 - test/unit/rh_cloud_http_proxy_test.rb
 - test/unit/rh_cloud_permissions_test.rb
 - test/unit/services/foreman_rh_cloud/branch_info_test.rb

data/lib/foreman_inventory_upload/async/upload_report_job.rb

@@ -1,97 +0,0 @@
-require 'tempfile'
-
-module ForemanInventoryUpload
-  module Async
-    class UploadReportJob < ShellProcess
-      def self.output_label(label)
-        "upload_for_#{label}"
-      end
-
-      def plan(filename, organization_id)
-        label = UploadReportJob.output_label(organization_id)
-        super(label, filename: filename, organization_id: organization_id)
-      end
-
-      def try_execute
-        if content_disconnected?
-          progress_output do |progress_output|
-            progress_output.write_line("Report was not moved and upload was canceled because connection to Insights is not enabled. Report location: #{filename}.")
-            progress_output.status = "Task aborted, exit 1"
-            done!
-          end
-          return
-        end
-
-        unless organization.owner_details&.fetch('upstreamConsumer')&.fetch('idCert')
-          logger.info("Skipping organization '#{organization}', no candlepin certificate defined.")
-          progress_output do |progress_output|
-            progress_output.write_line("Skipping organization #{organization}, no candlepin certificate defined.")
-            progress_output.status = "Task aborted, exit 1"
-            done!
-          end
-          return
-        end
-
-        Tempfile.create([organization.name, '.pem']) do |cer_file|
-          cer_file.write(certificate[:cert])
-          cer_file.write(certificate[:key])
-          cer_file.flush
-          @cer_path = cer_file.path
-          super
-        end
-      end
-
-      def command
-        ['/bin/bash', File.join(File.dirname(filename), ForemanInventoryUpload.upload_script_file)]
-      end
-
-      def env
-        env_vars = super.merge(
-          'FILES' => filename,
-          'CER_PATH' => @cer_path,
-          'ORG_ID' => organization.label
-        )
-
-        http_proxy_string = ForemanRhCloud.http_proxy_string
-        if http_proxy_string
-          env_vars['http_proxy'] = http_proxy_string
-          env_vars['https_proxy'] = http_proxy_string
-        end
-        env_vars
-      end
-
-      def certificate
-        ForemanRhCloud.with_iop_smart_proxy? ? foreman_certificate : manifest_certificate
-      end
-
-      def manifest_certificate
-        @manifest_certificate ||= begin
-          candlepin_id_certificate = organization.owner_details['upstreamConsumer']['idCert']
-          {
-            cert: candlepin_id_certificate['cert'],
-            key: candlepin_id_certificate['key'],
-          }
-        end
-      end
-
-      def foreman_certificate
-        @foreman_certificate ||= {
-          cert: File.read(Setting[:ssl_certificate]),
-          key: File.read(Setting[:ssl_priv_key]),
-        }
-      end
-
-      def filename
-        input[:filename]
-      end
-
-      def organization
-        @organization ||= Organization.find(input[:organization_id])
-      end
-
-      def content_disconnected?
-        !Setting[:subscription_connection_enabled]
-      end
-    end
-  end
-end

data/lib/foreman_inventory_upload/scripts/uploader.sh.erb

@@ -1,55 +0,0 @@
-#! /bin/bash
-
-DEST=<%= @upload_url %>
-RH_USERNAME=<%= @rh_username %>
-
-if [ -z "$FILES" ]
-then
-        FILES=./*.tar.gz
-fi
-
-if [ -n "$CER_PATH" ]
-then
-        AUTH_KEY="--cert"
-        AUTH_VAL="$CER_PATH"
-else
-        if [ -z "$RH_USERNAME" ]
-        then
-                IFS= read -rp "Enter username: " RH_USERNAME
-        fi
-
-        if [ -z "$RH_PASSWORD" ]
-        then
-                IFS= read -rsp "Enter password: " RH_PASSWORD
-        fi
-
-        AUTH_KEY="-u"
-        AUTH_VAL="\"$RH_USERNAME\":\"$RH_PASSWORD\""
-fi
-
-ORG_HEADER=()
-if [ -n "$ORG_ID" ]
-then
-        ORG_HEADER=("-H" "X-Org-Id: $ORG_ID")
-fi
-
-# /tmp/a b/x.pem
-# curl --cert /tmp/a\ b/x.pem
-
-SCRIPT_DIR=$(realpath "$(dirname "${BASH_SOURCE:-0}")")
-DONE_DIR="$SCRIPT_DIR/done/"
-mkdir -p $DONE_DIR
-
-for f in $FILES
-do
-  curl -k -vvv -# --fail -F "file=@$f;type=application/vnd.redhat.qpc.tar+tgz" $DEST "$AUTH_KEY" "$AUTH_VAL" "${ORG_HEADER[@]}"
-  status=$?
-  if [ $status -eq 0 ]; then
-    mv $f $DONE_DIR
-    echo "Done: $f"
-  fi
-done
-echo "Uploaded files moved to done/ folder"
-
-# return the error code from the curl command
-exit $status

data/test/jobs/upload_report_job_test.rb

@@ -1,38 +0,0 @@
-require 'test_plugin_helper'
-require 'foreman_tasks/test_helpers'
-
-class UploadReportJobTest < ActiveSupport::TestCase
-  include Dynflow::Testing::Factories
-
-  test 'returns aborted state when disconnected' do
-    organization = FactoryBot.create(:organization)
-    Organization.any_instance.stubs(:owner_details).returns(
-      'upstreamConsumer' => {
-        'idCert' => 'TEST_CERT',
-      }
-    )
-    ForemanInventoryUpload::Async::UploadReportJob.any_instance.expects(:content_disconnected?).returns(true)
-
-    action = create_and_plan_action(ForemanInventoryUpload::Async::UploadReportJob, '', organization.id)
-    run_action(action)
-
-    label = ForemanInventoryUpload::Async::UploadReportJob.output_label(organization.id)
-    progress_output = ForemanInventoryUpload::Async::ProgressOutput.get(label)
-    assert_match(/upload was canceled because connection to Insights is not enabled/, progress_output.full_output)
-    assert_match(/Report location:/, progress_output.full_output)
-    assert_match(/exit 1/, progress_output.status)
-  end
-
-  test 'returns aborted state when no certificate defined on organization' do
-    organization = FactoryBot.create(:organization)
-    Organization.any_instance.expects(:owner_details).returns(nil)
-
-    action = create_and_plan_action(ForemanInventoryUpload::Async::UploadReportJob, '', organization.id)
-    run_action(action)
-
-    label = ForemanInventoryUpload::Async::UploadReportJob.output_label(organization.id)
-    progress_output = ForemanInventoryUpload::Async::ProgressOutput.get(label)
-    assert_match(/Skipping organization/, progress_output.full_output)
-    assert_match(/exit 1/, progress_output.status)
-  end
-end