foreman_rh_cloud 13.2.10 → 14.0.1

data/test/unit/lib/insights_cloud/async/vmaas_reposcan_sync_test.rb

@@ -5,17 +5,19 @@ class VmaasReposcanSyncTest < ActiveSupport::TestCase
   include ForemanTasks::TestHelpers::WithInThreadExecutor
 
   setup do
-    @organization = FactoryBot.create(:organization)
-    # Create a simple repository - we only need id and organization_id for the action
-    @repo = ::Katello::Repository.new(id: 1)
-    @repo.stubs(:organization_id).returns(@organization.id)
-    ::Katello::Repository.stubs(:find).with(1).returns(@repo)
-
+    @root = FactoryBot.build(:katello_root_repository, :fedora_17_x86_64_dev_root)
+    @root.save(validate: false)
+    @repo = FactoryBot.create(
+      :katello_repository,
+      :with_product,
+      distribution_family: 'Red Hat',
+      distribution_version: '7.5',
+      root: @root
+    )
     @repo_payload = { id: @repo.id }
     @expected_url = 'https://example.com/api/v1/vmaas/reposcan/sync'
     InsightsCloud.stubs(:vmaas_reposcan_sync_url).returns(@expected_url)
     ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-    Organization.stubs(:find).with(@organization.id).returns(@organization)
   end
 
   teardown do
@@ -81,7 +83,7 @@ class VmaasReposcanSyncTest < ActiveSupport::TestCase
         params[:url] == @expected_url &&
         params[:headers].is_a?(Hash) &&
         params[:headers]['Content-Type'] == 'application/json' &&
-        params[:organization] == @organization
+        params[:organization] == @repo.organization
     end
                                            .returns(mock_response)
 
@@ -114,94 +116,37 @@ class VmaasReposcanSyncTest < ActiveSupport::TestCase
                                            .stubs(:execute_cloud_request)
                                            .raises(exception)
 
-    task = ForemanTasks.sync_task(InsightsCloud::Async::VmaasReposcanSync, @repo_payload)
+    error = assert_raises(ForemanTasks::TaskError) do
+      ForemanTasks.sync_task(InsightsCloud::Async::VmaasReposcanSync, @repo_payload)
+    end
 
-    assert_equal 'VMaaS reposcan sync failed: 500 - Server Error', task.output[:message]
+    assert_equal 'VMaaS reposcan sync failed: 500 - Server Error', error.task.output[:message]
   end
 
   test 'run sets error message in task output for StandardError exception' do
-    mock_logger = mock('logger')
-    mock_logger.expects(:error).with('Error triggering VMaaS reposcan sync: Network timeout')
-    InsightsCloud::Async::VmaasReposcanSync.any_instance.stubs(:logger).returns(mock_logger)
-
     InsightsCloud::Async::VmaasReposcanSync.any_instance
                                            .stubs(:execute_cloud_request)
                                            .raises(StandardError.new('Network timeout'))
 
-    task = ForemanTasks.sync_task(InsightsCloud::Async::VmaasReposcanSync, @repo_payload)
-
-    assert_equal 'Error triggering VMaaS reposcan sync: Network timeout', task.output[:message]
-  end
-
-  test 'run logs and handles error response without raising' do
-    error_response = mock('error_response')
-    error_response.stubs(:code).returns(500)
-    error_response.stubs(:body).returns('error')
-    exception = RestClient::ExceptionWithResponse.new(error_response)
-
-    InsightsCloud::Async::VmaasReposcanSync.any_instance
-                                           .stubs(:execute_cloud_request)
-                                           .raises(exception)
-
-    task = ForemanTasks.sync_task(InsightsCloud::Async::VmaasReposcanSync, @repo_payload)
-
-    assert_equal 'VMaaS reposcan sync failed: 500 - error', task.output[:message]
-  end
-
-  test 'run handles 429 error with warning log level' do
-    error_response = mock('error_response')
-    error_response.stubs(:code).returns(429)
-    error_response.stubs(:body).returns('{"msg": "Another task already in progress"}')
-    exception = RestClient::ExceptionWithResponse.new(error_response)
-
-    mock_logger = mock('logger')
-    mock_logger.expects(:warn).with('VMaaS reposcan sync skipped: another sync already in progress (429)')
-    InsightsCloud::Async::VmaasReposcanSync.any_instance.stubs(:logger).returns(mock_logger)
-
-    InsightsCloud::Async::VmaasReposcanSync.any_instance
-                                           .stubs(:execute_cloud_request)
-                                           .raises(exception)
-
-    task = ForemanTasks.sync_task(InsightsCloud::Async::VmaasReposcanSync, @repo_payload)
+    error = assert_raises(ForemanTasks::TaskError) do
+      ForemanTasks.sync_task(InsightsCloud::Async::VmaasReposcanSync, @repo_payload)
+    end
 
-    assert_equal 'VMaaS reposcan sync skipped: another sync already in progress (429)',
-      task.output[:message]
+    # The task is available via main_action
+    assert_match(/Error triggering VMaaS reposcan sync: Network timeout, response: /,
+      error.task.main_action.output[:message])
   end
 
-  test 'run handles non-429 errors with error log level' do
-    error_response = mock('error_response')
-    error_response.stubs(:code).returns(500)
-    error_response.stubs(:body).returns('Internal Server Error')
+  test 'run logs and re-raises when cloud request returns error response' do
+    error_response = mock('error_response', code: 500, body: 'error')
     exception = RestClient::ExceptionWithResponse.new(error_response)
 
-    mock_logger = mock('logger')
-    mock_logger.expects(:error).with('VMaaS reposcan sync failed: 500 - Internal Server Error')
-    InsightsCloud::Async::VmaasReposcanSync.any_instance.stubs(:logger).returns(mock_logger)
-
     InsightsCloud::Async::VmaasReposcanSync.any_instance
                                            .stubs(:execute_cloud_request)
                                            .raises(exception)
 
-    task = ForemanTasks.sync_task(InsightsCloud::Async::VmaasReposcanSync, @repo_payload)
-
-    assert_equal 'VMaaS reposcan sync failed: 500 - Internal Server Error',
-      task.output[:message]
-  end
-
-  test 'run handles RestClient::ExceptionWithResponse with nil response' do
-    exception = RestClient::ExceptionWithResponse.new(nil)
-
-    mock_logger = mock('logger')
-    mock_logger.expects(:error).with('VMaaS reposcan sync failed:  - ')
-    InsightsCloud::Async::VmaasReposcanSync.any_instance.stubs(:logger).returns(mock_logger)
-
-    InsightsCloud::Async::VmaasReposcanSync.any_instance
-                                           .stubs(:execute_cloud_request)
-                                           .raises(exception)
-
-    task = ForemanTasks.sync_task(InsightsCloud::Async::VmaasReposcanSync, @repo_payload)
-
-    refute_nil task.output[:message]
-    assert_equal 'VMaaS reposcan sync failed:  - ', task.output[:message]
+    assert_raises(ForemanTasks::TaskError) do
+      ForemanTasks.sync_task(InsightsCloud::Async::VmaasReposcanSync, @repo_payload)
+    end
   end
 end

data/test/unit/metadata_generator_test.rb

@@ -2,8 +2,7 @@ require 'test_plugin_helper'
 
 class MetadataGeneratorTest < ActiveSupport::TestCase
   setup do
-    # reset cached value - must remove the variable entirely, not just set to nil
-    ForemanRhCloud.remove_instance_variable(:@foreman_host) if ForemanRhCloud.instance_variable_defined?(:@foreman_host)
+    ForemanRhCloud.instance_variable_set(:@foreman_host, nil)
   end
 
   test 'generates an empty report' do
@@ -65,26 +64,4 @@ class MetadataGeneratorTest < ActiveSupport::TestCase
     assert_not_nil(slice = slices['test_12345'])
     assert_equal 3, slice['number_hosts']
   end
-
-  test 'generates metadata when foreman_host is nil' do
-    ForemanRhCloud.stubs(:foreman_host).returns(nil)
-    ForemanRhCloud.stubs(:foreman_host_name).returns('satellite.example.com')
-
-    generator = ForemanInventoryUpload::Generators::Metadata.new
-
-    # Should not raise an error
-    json_str = nil
-    assert_nothing_raised do
-      json_str = generator.render do
-      end
-    end
-
-    # Verify hostname is from foreman_host_name
-    actual = JSON.parse(json_str.join("\n"))
-    assert_equal 'satellite.example.com', actual['reporting_host_name']
-    # Verify IP and BIOS UUID fields are nil when host is nil
-    # This is acceptable per SAT-25889 - cloud services don't rely on these fields
-    assert_nil actual['reporting_host_ips']
-    assert_nil actual['reporting_host_bios_uuid']
-  end
 end

data/test/unit/rh_cloud_host_test.rb

@@ -188,218 +188,4 @@ class RhCloudHostTest < ActiveSupport::TestCase
       assert_equal local_uuid, @host.insights_uuid
     end
   end
-
-  context 'scoped search on insights_uuid' do
-    setup do
-      @org = FactoryBot.create(:organization)
-    end
-
-    teardown do
-      ForemanRhCloud.unstub(:with_iop_smart_proxy?)
-    end
-
-    test 'searches insights_facet.uuid in non-IoP mode with = operator' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      host1 = FactoryBot.create(:host, :managed, organization: @org)
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'insights-uuid-123')
-      host2 = FactoryBot.create(:host, :managed, organization: @org)
-      host2.insights = FactoryBot.create(:insights_facet, host_id: host2.id, uuid: 'insights-uuid-456')
-
-      results = Host::Managed.search_for('insights_uuid = insights-uuid-123')
-
-      assert_includes results, host1
-      assert_not_includes results, host2
-    end
-
-    test 'searches subscription_facet.uuid in IoP mode with = operator' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      host1 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host2 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-
-      # Even if insights_facet has different UUID, should use subscription_facet UUID
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'stale-123')
-
-      results = Host::Managed.search_for("insights_uuid = #{host1.subscription_facet.uuid}")
-
-      assert_includes results, host1
-      assert_not_includes results, host2
-    end
-
-    test 'searches with ^ operator (IN) in non-IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      host1 = FactoryBot.create(:host, :managed, organization: @org)
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'uuid-1')
-      host2 = FactoryBot.create(:host, :managed, organization: @org)
-      host2.insights = FactoryBot.create(:insights_facet, host_id: host2.id, uuid: 'uuid-2')
-      host3 = FactoryBot.create(:host, :managed, organization: @org)
-      host3.insights = FactoryBot.create(:insights_facet, host_id: host3.id, uuid: 'uuid-3')
-
-      results = Host::Managed.search_for('insights_uuid ^ (uuid-1,uuid-2)')
-
-      assert_includes results, host1
-      assert_includes results, host2
-      assert_not_includes results, host3
-    end
-
-    test 'searches with ^ operator (IN) in IoP mode - THE BUG FIX' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      host1 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host2 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host3 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-
-      # Create insights facets with stale UUIDs to verify we're using subscription_facet
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'stale-1')
-      host2.insights = FactoryBot.create(:insights_facet, host_id: host2.id, uuid: 'stale-2')
-      host3.insights = FactoryBot.create(:insights_facet, host_id: host3.id, uuid: 'stale-3')
-
-      uuid1 = host1.subscription_facet.uuid
-      uuid2 = host2.subscription_facet.uuid
-
-      # This is the search query that remediation modal creates
-      results = Host::Managed.search_for("insights_uuid ^ (#{uuid1},#{uuid2})")
-
-      # Should find hosts by subscription_facet UUID, not insights_facet UUID
-      assert_includes results, host1
-      assert_includes results, host2
-      assert_not_includes results, host3
-    end
-
-    test 'searches with !^ operator (NOT IN) in non-IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      host1 = FactoryBot.create(:host, :managed, organization: @org)
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'uuid-1')
-      host2 = FactoryBot.create(:host, :managed, organization: @org)
-      host2.insights = FactoryBot.create(:insights_facet, host_id: host2.id, uuid: 'uuid-2')
-      host3 = FactoryBot.create(:host, :managed, organization: @org)
-      host3.insights = FactoryBot.create(:insights_facet, host_id: host3.id, uuid: 'uuid-3')
-
-      results = Host::Managed.search_for('insights_uuid !^ (uuid-1,uuid-2)')
-
-      assert_not_includes results, host1
-      assert_not_includes results, host2
-      assert_includes results, host3
-    end
-
-    test 'searches with !^ operator (NOT IN) in IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      host1 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host2 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host3 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-
-      uuid1 = host1.subscription_facet.uuid
-      uuid2 = host2.subscription_facet.uuid
-
-      results = Host::Managed.search_for("insights_uuid !^ (#{uuid1},#{uuid2})")
-
-      assert_not_includes results, host1
-      assert_not_includes results, host2
-      assert_includes results, host3
-    end
-
-    test 'handles hosts without facets in non-IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      host_without_facet = FactoryBot.create(:host, :managed, organization: @org)
-      host_with_facet = FactoryBot.create(:host, :managed, organization: @org)
-      host_with_facet.insights = FactoryBot.create(:insights_facet, host_id: host_with_facet.id, uuid: 'uuid-1')
-
-      results = Host::Managed.search_for('insights_uuid = uuid-1')
-
-      assert_includes results, host_with_facet
-      assert_not_includes results, host_without_facet
-    end
-
-    test 'handles hosts without subscription_facet in IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      host_without_sub = FactoryBot.create(:host, :managed, organization: @org)
-      host_with_sub = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-
-      uuid = host_with_sub.subscription_facet.uuid
-
-      results = Host::Managed.search_for("insights_uuid = #{uuid}")
-
-      assert_includes results, host_with_sub
-      assert_not_includes results, host_without_sub
-    end
-
-    test 'mode changes are reflected in searches' do
-      host1 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'insights-uuid-abc')
-      insights_uuid = 'insights-uuid-abc'
-      subscription_uuid = host1.subscription_facet.uuid
-
-      # Non-IoP mode: should find by insights_facet UUID
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      results = Host::Managed.search_for("insights_uuid = #{insights_uuid}")
-      assert_includes results, host1
-
-      # IoP mode: should find by subscription_facet UUID
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      results = Host::Managed.search_for("insights_uuid = #{subscription_uuid}")
-      assert_includes results, host1
-
-      # Should NOT find by old insights_facet UUID in IoP mode
-      results = Host::Managed.search_for("insights_uuid = #{insights_uuid}")
-      assert_not_includes results, host1
-    end
-  end
-
-  test 'scoped search for user_omitted inventory status works' do
-    host1 = FactoryBot.create(:host, :managed)
-    host2 = FactoryBot.create(:host, :managed)
-    host3 = FactoryBot.create(:host, :managed)
-
-    # Create different inventory statuses
-    InventorySync::InventoryStatus.create!(
-      host_id: host1.id,
-      status: InventorySync::InventoryStatus::SYNC,
-      reported_at: Time.zone.now
-    )
-
-    InventorySync::InventoryStatus.create!(
-      host_id: host2.id,
-      status: InventorySync::InventoryStatus::DISCONNECT,
-      reported_at: Time.zone.now
-    )
-
-    InventorySync::InventoryStatus.create!(
-      host_id: host3.id,
-      status: InventorySync::InventoryStatus::USER_OMITTED,
-      reported_at: Time.zone.now
-    )
-
-    # Search for user_omitted status
-    results = Host.search_for('insights_inventory_sync_status = user_omitted')
-    result_ids = results.pluck(:id)
-
-    assert_includes result_ids, host3.id, 'Host with USER_OMITTED status should be in search results'
-    assert_not_includes result_ids, host1.id, 'Host with SYNC status should not be in search results'
-    assert_not_includes result_ids, host2.id, 'Host with DISCONNECT status should not be in search results'
-  end
-
-  test 'scoped search for user_omitted insights client report status works' do
-    host1 = FactoryBot.create(:host, :managed)
-    host2 = FactoryBot.create(:host, :managed)
-    host3 = FactoryBot.create(:host, :managed)
-
-    # Create different insights client report statuses
-    status1 = host1.get_status(InsightsClientReportStatus)
-    status1.status = InsightsClientReportStatus::REPORTING
-    status1.save!
-
-    status2 = host2.get_status(InsightsClientReportStatus)
-    status2.status = InsightsClientReportStatus::NO_REPORT
-    status2.save!
-
-    status3 = host3.get_status(InsightsClientReportStatus)
-    status3.status = InsightsClientReportStatus::USER_OMITTED
-    status3.save!
-
-    # Search for user_omitted status
-    results = Host.search_for('insights_client_report_status = user_omitted')
-    result_ids = results.pluck(:id)
-
-    assert_includes result_ids, host3.id, 'Host with USER_OMITTED status should be in search results'
-    assert_not_includes result_ids, host1.id, 'Host with REPORTING status should not be in search results'
-    assert_not_includes result_ids, host2.id, 'Host with NO_REPORT status should not be in search results'
-  end
 end

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

@@ -19,7 +19,7 @@ class BranchInfoTest < ActiveSupport::TestCase
       organization: env.organization
     )
 
-    @host.subscription_facet.pools << FactoryBot.create(:katello_pool, account_number: '5678', cp_id: 1)
+    @host.organization.pools << FactoryBot.create(:katello_pool, account_number: '5678', cp_id: 1)
   end
 
   test 'should generate branch info for host' do

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

@@ -29,7 +29,7 @@ class CloudRequestForwarderTest < ActiveSupport::TestCase
       organization: env.organization
     )
 
-    @host.subscription_facet.pools << FactoryBot.create(:katello_pool, account_number: '5678', cp_id: 1)
+    @host.organization.pools << FactoryBot.create(:katello_pool, account_number: '5678', cp_id: 1)
   end
 
   test 'should prepare correct cloud url' do

data/test/unit/slice_generator_test.rb

@@ -641,6 +641,7 @@ class SliceGeneratorTest < ActiveSupport::TestCase
     assert_tag('1', actual_host, 'int_param', 'satellite_parameter')
 
     assert_equal false, satellite_facts['is_hostname_obfuscated']
+    assert_equal true, satellite_facts['is_simple_content_access']
 
     version = satellite_facts['satellite_version']
     if defined?(ForemanThemeSatellite)
@@ -702,22 +703,6 @@ class SliceGeneratorTest < ActiveSupport::TestCase
     assert_equal 'test_sla', fact_values['system_purpose_sla']
   end
 
-  test 'generates a report for a golden ticket' do
-    batch = Host.where(id: @host.id).in_batches.first
-    generator = create_generator(batch) do |generator|
-      generator.stubs(:golden_ticket?).returns(true)
-    end
-
-    json_str = generator.render
-    actual = JSON.parse(json_str.join("\n"))
-
-    assert_equal '00000000-0000-0000-0000-000000000000', actual['report_slice_id']
-    assert_not_nil(actual_host = actual['hosts'].first)
-    assert_equal @host.fqdn, actual_host['fqdn']
-    assert_equal '1234', actual_host['account']
-    assert_equal 1, generator.hosts_count
-  end
-
   test 'skips hosts without subscription' do
     a_host = FactoryBot.create(
       :host,
@@ -1131,11 +1116,7 @@ class SliceGeneratorTest < ActiveSupport::TestCase
 
   def create_generator(batch, name = '00000000-0000-0000-0000-000000000000')
     generator = ForemanInventoryUpload::Generators::Slice.new(batch, [], name)
-    if block_given?
-      yield(generator)
-    else
-      generator.stubs(:golden_ticket?).returns(false)
-    end
+    yield(generator) if block_given?
     generator
   end
 

data/webpack/CVEsHostDetailsTab/CVEsHostDetailsTab.js

@@ -3,10 +3,6 @@ import PropTypes from 'prop-types';
 import { ScalprumComponent, ScalprumProvider } from '@scalprum/react-core';
 import { createProviderOptions } from '../common/ScalprumModule/ScalprumContext';
 import { useInsightsPermissions } from '../common/Hooks/PermissionsHooks';
-import {
-  vulnerabilityDisabled,
-  useTabRedirect,
-} from '../ForemanRhCloudHelpers';
 import './CVEsHostDetailsTab.scss';
 
 const CVEsHostDetailsTab = ({ systemId }) => {
@@ -14,12 +10,7 @@ const CVEsHostDetailsTab = ({ systemId }) => {
   const module = './SystemDetailTable';
   return (
     <div className="rh-cloud-insights-vulnerability-host-details-component vulnerability">
-      <ScalprumComponent
-        key={systemId}
-        scope={scope}
-        module={module}
-        systemId={systemId}
-      />
+      <ScalprumComponent scope={scope} module={module} systemId={systemId} />
     </div>
   );
 };
@@ -30,15 +21,6 @@ CVEsHostDetailsTab.propTypes = {
 
 const CVEsHostDetailsTabWrapper = ({ response }) => {
   const permissions = useInsightsPermissions();
-  const isHostDataLoaded = Boolean(response?.id);
-  const shouldHideTab = useTabRedirect(
-    isHostDataLoaded && vulnerabilityDisabled({ hostDetails: response })
-  );
-
-  if (shouldHideTab) {
-    return null;
-  }
-
   return (
     <ScalprumProvider {...createProviderOptions(permissions)}>
       <CVEsHostDetailsTab
@@ -51,19 +33,10 @@ const CVEsHostDetailsTabWrapper = ({ response }) => {
 
 CVEsHostDetailsTabWrapper.propTypes = {
   response: PropTypes.shape({
-    id: PropTypes.number,
-    operatingsystem_name: PropTypes.string,
-    vulnerability: PropTypes.shape({
-      enabled: PropTypes.bool,
-    }),
     subscription_facet_attributes: PropTypes.shape({
-      uuid: PropTypes.string,
+      uuid: PropTypes.string.isRequired,
     }),
-  }),
-};
-
-CVEsHostDetailsTabWrapper.defaultProps = {
-  response: {},
+  }).isRequired,
 };
 
 export default CVEsHostDetailsTabWrapper;

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

@@ -1,17 +1,6 @@
 import React from 'react';
 import { render } from '@testing-library/react';
-import { MemoryRouter } from 'react-router-dom';
 import CVEsHostDetailsTabWrapper from '../CVEsHostDetailsTab';
-import { OVERVIEW_TAB_PATH } from '../../ForemanRhCloudHelpers';
-
-const mockHistoryReplace = jest.fn();
-
-jest.mock('react-router-dom', () => ({
-  ...jest.requireActual('react-router-dom'),
-  useHistory: () => ({
-    replace: mockHistoryReplace,
-  }),
-}));
 
 jest.mock('foremanReact/Root/Context/ForemanContext', () => ({
   useForemanContext: () => ({
@@ -22,115 +11,24 @@ jest.mock('foremanReact/Root/Context/ForemanContext', () => ({
   useForemanPermissions: () => new Set(['view_vulnerability']),
 }));
 
-const mockUnmountTracker = jest.fn();
-jest.mock('@scalprum/react-core', () => {
-  const ReactMock = require('react');
-  return {
-    ScalprumComponent: jest.fn(props => {
-      ReactMock.useEffect(() => mockUnmountTracker, []);
-      return (
-        <div data-testid="mock-scalprum-component">{JSON.stringify(props)}</div>
-      );
-    }),
-    ScalprumProvider: jest.fn(({ children }) => <div>{children}</div>),
-  };
-});
-
-const defaultResponse = {
-  id: 1,
-  operatingsystem_name: 'Red Hat Enterprise Linux 8',
-  vulnerability: { enabled: true },
-  subscription_facet_attributes: { uuid: '1-2-3' },
-};
+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', () => {
-  beforeEach(() => {
-    jest.clearAllMocks();
-  });
-
-  it('renders without crashing and does not redirect for valid host', () => {
+  it('renders without crashing', () => {
     const { container } = render(
-      <MemoryRouter>
-        <CVEsHostDetailsTabWrapper response={defaultResponse} />
-      </MemoryRouter>
+      <CVEsHostDetailsTabWrapper
+        response={{ subscription_facet_attributes: { uuid: '1-2-3' } }}
+      />
     );
     expect(
       container.querySelector(
         '.rh-cloud-insights-vulnerability-host-details-component'
       )
     ).toBeTruthy();
-    expect(mockHistoryReplace).not.toHaveBeenCalled();
-  });
-
-  it('remounts ScalprumComponent when systemId changes', () => {
-    const { ScalprumComponent } = require('@scalprum/react-core');
-
-    const responseHostA = {
-      ...defaultResponse,
-      subscription_facet_attributes: { uuid: 'uuid-host-A' },
-    };
-
-    const { rerender } = render(
-      <MemoryRouter>
-        <CVEsHostDetailsTabWrapper response={responseHostA} />
-      </MemoryRouter>
-    );
-
-    expect(mockUnmountTracker).not.toHaveBeenCalled();
-    expect(ScalprumComponent).toHaveBeenLastCalledWith(
-      expect.objectContaining({ systemId: 'uuid-host-A' }),
-      expect.anything()
-    );
-
-    const responseHostB = {
-      ...defaultResponse,
-      subscription_facet_attributes: { uuid: 'uuid-host-B' },
-    };
-
-    rerender(
-      <MemoryRouter>
-        <CVEsHostDetailsTabWrapper response={responseHostB} />
-      </MemoryRouter>
-    );
-
-    expect(mockUnmountTracker).toHaveBeenCalledTimes(1);
-    expect(ScalprumComponent).toHaveBeenLastCalledWith(
-      expect.objectContaining({ systemId: 'uuid-host-B' }),
-      expect.anything()
-    );
-  });
-
-  it('redirects to Overview when tab should be hidden', () => {
-    const nonRhelResponse = {
-      id: 2,
-      operatingsystem_name: 'Ubuntu 20.04',
-      vulnerability: { enabled: false },
-      subscription_facet_attributes: { uuid: '1-2-3' },
-    };
-
-    const { container } = render(
-      <MemoryRouter>
-        <CVEsHostDetailsTabWrapper response={nonRhelResponse} />
-      </MemoryRouter>
-    );
-
-    expect(mockHistoryReplace).toHaveBeenCalledWith(OVERVIEW_TAB_PATH);
-    expect(
-      container.querySelector(
-        '.rh-cloud-insights-vulnerability-host-details-component'
-      )
-    ).toBeNull();
-  });
-
-  it('does not redirect when host data is not yet loaded', () => {
-    const emptyResponse = { subscription_facet_attributes: { uuid: '1-2-3' } };
-
-    render(
-      <MemoryRouter>
-        <CVEsHostDetailsTabWrapper response={emptyResponse} />
-      </MemoryRouter>
-    );
-
-    expect(mockHistoryReplace).not.toHaveBeenCalled();
   });
 });