foreman_rh_cloud 12.2.17 → 13.0.0

data/test/unit/lib/foreman_rh_cloud/registration_manager_extensions_test.rb

@@ -1,154 +0,0 @@
-require 'test_plugin_helper'
-
-module ForemanRhCloud
-  class RegistrationManagerExtensionsTest < ActiveSupport::TestCase
-    setup do
-      @org = FactoryBot.create(:organization)
-      @host = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      @insights_facet = ::InsightsFacet.create!(host: @host, uuid: @host.subscription_facet.uuid)
-
-      # Stub Candlepin interaction (from Katello)
-      ::Katello::Resources::Candlepin::Consumer.stubs(:destroy)
-
-      # Stub the cloud request to avoid actual HTTP calls
-      Katello::RegistrationManager.stubs(:execute_cloud_request).returns(true)
-    end
-
-    context 'unregister_host' do
-      test 'should call HBI delete in IoP mode when host has insights UUID' do
-        ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-        expected_url = ForemanInventoryUpload.host_by_id_url(@host.subscription_facet.uuid)
-
-        # Expect the cloud request to be made
-        Katello::RegistrationManager.expects(:execute_cloud_request).with do |params|
-          params[:organization] == @org &&
-            params[:method] == :delete &&
-            params[:url] == expected_url &&
-            params[:headers][:content_type] == :json
-        end.returns(true)
-
-        Katello::RegistrationManager.unregister_host(@host, unregistering: true)
-
-        # Verify insights_facet was destroyed
-        assert_nil InsightsFacet.find_by(id: @insights_facet.id)
-      end
-
-      test 'should NOT call HBI delete in non-IoP mode' do
-        ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-
-        # Should NOT attempt to delete from HBI
-        Katello::RegistrationManager.expects(:execute_cloud_request).never
-
-        Katello::RegistrationManager.unregister_host(@host, unregistering: true)
-
-        # Verify insights_facet was still destroyed
-        assert_nil InsightsFacet.find_by(id: @insights_facet.id)
-      end
-
-      test 'should always destroy insights_facet regardless of IoP mode' do
-        facet_id = @insights_facet.id
-        ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-
-        assert_not_nil InsightsFacet.find_by(id: facet_id)
-
-        Katello::RegistrationManager.unregister_host(@host, unregistering: true)
-
-        assert_nil InsightsFacet.find_by(id: facet_id)
-      end
-    end
-
-    context 'hbi_host_destroy error handling' do
-      setup do
-        ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-        @expected_url = ForemanInventoryUpload.host_by_id_url('test-uuid-123')
-        @facet_id = @insights_facet.id
-        # Unstub execute_cloud_request for error tests
-        Katello::RegistrationManager.unstub(:execute_cloud_request)
-      end
-
-      test 'should handle RestClient::NotFound gracefully' do
-        Katello::RegistrationManager.stubs(:execute_cloud_request).raises(RestClient::NotFound)
-
-        # Should log warning but not raise
-        Rails.logger.expects(:warn).with(regexp_matches(/host does not exist in HBI/))
-
-        assert_nothing_raised do
-          Katello::RegistrationManager.unregister_host(@host, unregistering: true)
-        end
-
-        # Facet should still be destroyed
-        assert_nil InsightsFacet.find_by(id: @facet_id)
-      end
-
-      test 'should handle server errors gracefully' do
-        error = RestClient::InternalServerError.new
-        Katello::RegistrationManager.stubs(:execute_cloud_request).raises(error)
-
-        # Should log error but not raise
-        Rails.logger.expects(:error).with(regexp_matches(/Failed to destroy HBI host/))
-
-        assert_nothing_raised do
-          Katello::RegistrationManager.unregister_host(@host, unregistering: true)
-        end
-
-        # Facet should still be destroyed
-        assert_nil InsightsFacet.find_by(id: @facet_id)
-      end
-
-      test 'should handle timeout errors gracefully' do
-        error = RestClient::Exceptions::ReadTimeout.new
-        Katello::RegistrationManager.stubs(:execute_cloud_request).raises(error)
-
-        # Should log error but not raise
-        Rails.logger.expects(:error).with(regexp_matches(/Failed to destroy HBI host/))
-
-        assert_nothing_raised do
-          Katello::RegistrationManager.unregister_host(@host, unregistering: true)
-        end
-
-        # Facet should still be destroyed
-        assert_nil InsightsFacet.find_by(id: @facet_id)
-      end
-
-      test 'should handle connection errors gracefully' do
-        error = Errno::ECONNREFUSED.new
-        Katello::RegistrationManager.stubs(:execute_cloud_request).raises(error)
-
-        # Should log error but not raise
-        Rails.logger.expects(:error).with(regexp_matches(/Failed to destroy HBI host/))
-
-        assert_nothing_raised do
-          Katello::RegistrationManager.unregister_host(@host, unregistering: true)
-        end
-
-        # Facet should still be destroyed
-        assert_nil InsightsFacet.find_by(id: @facet_id)
-      end
-    end
-
-    context 'integration' do
-      test 'should be properly prepended to RegistrationManager' do
-        assert_includes Katello::RegistrationManager.singleton_class.ancestors,
-          ForemanRhCloud::RegistrationManagerExtensions
-      end
-
-      test 'should preserve original unregister_host behavior' do
-        ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-
-        # Just verify the extension is properly integrated
-        # Detailed Katello behavior is tested in Katello's own tests
-        assert_nothing_raised do
-          Katello::RegistrationManager.unregister_host(@host, unregistering: true)
-        end
-      end
-    end
-
-    context 'URL generation' do
-      test 'host_by_id_url should return correct format' do
-        url = ForemanInventoryUpload.host_by_id_url('test-uuid-123')
-
-        assert_match %r{/hosts/test-uuid-123$}, url
-      end
-    end
-  end
-end

data/test/unit/rh_cloud_host_test.rb

@@ -1,345 +0,0 @@
-require 'test_plugin_helper'
-
-class RhCloudHostTest < ActiveSupport::TestCase
-  setup do
-    @org = FactoryBot.create(:organization)
-  end
-
-  teardown do
-    ForemanRhCloud.unstub(:with_iop_smart_proxy?)
-  end
-
-  context 'insights_uuid method' do
-    test 'returns insights_facet uuid in non-IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      @host = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      @host.insights = FactoryBot.create(:insights_facet, host_id: @host.id, uuid: 'insights-123')
-
-      assert_equal 'insights-123', @host.insights_uuid
-      assert_not_equal @host.subscription_facet.uuid, @host.insights_uuid
-    end
-
-    test 'returns subscription_facet uuid in IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      @host = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      @host.insights = FactoryBot.create(:insights_facet, host_id: @host.id, uuid: 'insights-456')
-
-      assert_equal @host.subscription_facet.uuid, @host.insights_uuid
-      assert_not_equal 'insights-456', @host.insights_uuid
-    end
-
-    test 'returns nil when insights_facet missing in non-IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      @host = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-
-      assert_nil @host.insights_uuid
-    end
-
-    test 'returns nil when subscription_facet missing in IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      @host = FactoryBot.create(:host, :managed, organization: @org)
-      @host.insights = FactoryBot.create(:insights_facet, host_id: @host.id, uuid: 'insights-789')
-
-      assert_nil @host.insights_uuid
-    end
-
-    test 'returns nil when both facets missing' do
-      @host = FactoryBot.create(:host, :managed, organization: @org)
-
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      assert_nil @host.insights_uuid
-
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      assert_nil @host.insights_uuid
-    end
-
-    test 'never returns stale insights_facet uuid in IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      @host = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      current_uuid = @host.subscription_facet.uuid
-      @host.insights = FactoryBot.create(:insights_facet, host_id: @host.id, uuid: 'stale-456')
-
-      # Should return subscription_facet UUID, never the stale insights_facet UUID
-      assert_equal current_uuid, @host.insights_uuid
-      assert_not_equal 'stale-456', @host.insights_uuid
-
-      # Verify multiple calls always return subscription_facet UUID
-      3.times do
-        assert_equal current_uuid, @host.insights_uuid
-      end
-    end
-
-    test 'dynamically responds to IoP mode changes' do
-      @host = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      @host.insights = FactoryBot.create(:insights_facet, host_id: @host.id, uuid: 'insights-123')
-      subscription_uuid = @host.subscription_facet.uuid
-
-      # Non-IoP mode: should return insights_facet UUID
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      assert_equal 'insights-123', @host.insights_uuid
-
-      # IoP mode: should return subscription_facet UUID
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      assert_equal subscription_uuid, @host.insights_uuid
-
-      # Toggle back to non-IoP mode
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      assert_equal 'insights-123', @host.insights_uuid
-    end
-  end
-
-  context 'ensure_iop_insights_uuid method' do
-    test 'updates insights_facet uuid when different from subscription_facet' do
-      @host = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      correct_uuid = @host.subscription_facet.uuid
-      @host.insights = FactoryBot.create(:insights_facet, host_id: @host.id, uuid: 'wrong-456')
-
-      assert_equal 'wrong-456', @host.insights_facet.uuid
-
-      @host.ensure_iop_insights_uuid
-
-      assert_equal correct_uuid, @host.insights_facet.reload.uuid
-    end
-
-    test 'does nothing when uuids already match' do
-      @host = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      matching_uuid = @host.subscription_facet.uuid
-      @host.insights = FactoryBot.create(:insights_facet, host_id: @host.id, uuid: matching_uuid)
-
-      # Expect no update call since UUIDs already match
-      @host.insights_facet.expects(:update!).never
-
-      @host.ensure_iop_insights_uuid
-
-      assert_equal matching_uuid, @host.insights_facet.uuid
-    end
-
-    test 'does nothing when insights_facet missing' do
-      @host = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-
-      assert_nothing_raised do
-        @host.ensure_iop_insights_uuid
-      end
-    end
-
-    test 'does nothing when subscription_facet missing' do
-      @host = FactoryBot.create(:host, :managed, organization: @org)
-      @host.insights = FactoryBot.create(:insights_facet, host_id: @host.id, uuid: 'insights-999')
-      original_uuid = @host.insights_facet.uuid
-
-      assert_nothing_raised do
-        @host.ensure_iop_insights_uuid
-      end
-
-      assert_equal original_uuid, @host.insights_facet.uuid
-    end
-
-    test 'does nothing when both facets missing' do
-      @host = FactoryBot.create(:host, :managed, organization: @org)
-
-      assert_nothing_raised do
-        @host.ensure_iop_insights_uuid
-      end
-    end
-
-    test 'handles nil uuids gracefully' do
-      @host = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-
-      # Scenario 1: subscription_facet uuid is nil (shouldn't happen but test gracefully)
-      @host.subscription_facet.update(uuid: nil)
-      @host.insights = FactoryBot.create(:insights_facet, host_id: @host.id, uuid: 'some-uuid')
-
-      assert_nothing_raised do
-        @host.ensure_iop_insights_uuid
-      end
-
-      # Scenario 2: insights_facet uuid is nil
-      @host.subscription_facet.update(uuid: 'valid-uuid')
-      @host.insights_facet.update(uuid: nil)
-
-      assert_nothing_raised do
-        @host.ensure_iop_insights_uuid
-      end
-
-      # Should update to match subscription_facet
-      assert_equal 'valid-uuid', @host.insights_facet.reload.uuid
-    end
-
-    test 'corrects stale uuid after cloud registration' do
-      # Simulate a host that was previously registered to cloud with cloud-assigned UUID
-      @host = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      local_uuid = @host.subscription_facet.uuid
-      @host.insights = FactoryBot.create(:insights_facet, host_id: @host.id, uuid: 'cloud-456')
-
-      # Before sync: insights_facet has stale cloud UUID
-      assert_equal 'cloud-456', @host.insights_facet.uuid
-
-      # In IoP mode, insights_uuid should return subscription_facet UUID (not stale)
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      assert_equal local_uuid, @host.insights_uuid
-
-      # Call sync to correct the stale UUID
-      @host.ensure_iop_insights_uuid
-
-      # After sync: insights_facet UUID should match subscription_facet UUID
-      assert_equal local_uuid, @host.insights_facet.reload.uuid
-
-      # Verify insights_uuid still returns correct value
-      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
-end

data/webpack/InsightsHostDetailsTab/__tests__/InsightsTotalRiskChart.test.js

@@ -1,194 +0,0 @@
-import React from 'react';
-import { render, screen, waitFor } from '@testing-library/react';
-import '@testing-library/jest-dom';
-import { Provider } from 'react-redux';
-import { ConnectedRouter } from 'connected-react-router';
-import { createMemoryHistory } from 'history';
-import configureMockStore from 'redux-mock-store';
-import { STATUS } from 'foremanReact/constants';
-import * as APIHooks from 'foremanReact/common/hooks/API/APIHooks';
-import InsightsTotalRiskCard from '../InsightsTotalRiskChart';
-
-jest.mock('foremanReact/common/hooks/API/APIHooks');
-jest.mock('foremanReact/common/I18n', () => ({
-  translate: jest.fn(str => str),
-}));
-
-const mockStore = configureMockStore();
-const history = createMemoryHistory();
-const store = mockStore({
-  router: {
-    location: {
-      pathname: '/',
-      search: '',
-      hash: '',
-      state: null,
-    },
-    action: 'POP',
-  },
-});
-
-const defaultHostDetails = {
-  id: 1,
-  insights_attributes: {
-    uuid: 'test-uuid',
-    use_iop_mode: false,
-  },
-};
-
-const renderComponent = (props = {}) => {
-  const allProps = {
-    hostDetails: defaultHostDetails,
-    ...props,
-  };
-
-  return render(
-    <Provider store={store}>
-      <ConnectedRouter history={history}>
-        <InsightsTotalRiskCard {...allProps} />
-      </ConnectedRouter>
-    </Provider>
-  );
-};
-
-describe('InsightsTotalRiskChart', () => {
-  beforeEach(() => {
-    store.clearActions();
-    jest.clearAllMocks();
-  });
-
-  it('should show loading state initially', () => {
-    APIHooks.useAPI.mockReturnValue({
-      status: STATUS.PENDING,
-      response: null,
-    });
-
-    renderComponent();
-    // SkeletonLoader shows loading state when status is PENDING
-    expect(screen.queryByText('No results found')).not.toBeInTheDocument();
-    expect(
-      screen.queryByTestId('rh-cloud-total-risk-card')
-    ).not.toBeInTheDocument();
-  });
-
-  it('should display error state when API fails', async () => {
-    APIHooks.useAPI.mockReturnValue({
-      status: STATUS.ERROR,
-      response: null,
-    });
-
-    renderComponent();
-    expect(screen.getByText('No results found')).toBeInTheDocument();
-    expect(
-      screen.queryByTestId('rh-cloud-total-risk-card')
-    ).not.toBeInTheDocument();
-  });
-
-  it('should handle non-IoP mode API response correctly', async () => {
-    const mockResponse = {
-      hits: [
-        { total_risk: 1 },
-        { total_risk: 2 },
-        { total_risk: 2 },
-        { total_risk: 3 },
-        { total_risk: 4 },
-      ],
-    };
-
-    APIHooks.useAPI.mockReturnValue({
-      status: STATUS.RESOLVED,
-      response: mockResponse,
-    });
-
-    renderComponent();
-
-    await waitFor(() => {
-      // Check if total number of recommendations is displayed
-      expect(screen.getByText('5')).toBeInTheDocument();
-      // Check if risk levels are displayed correctly
-      expect(screen.getByText(/Low: 1/)).toBeInTheDocument();
-      expect(screen.getByText(/Moderate: 2/)).toBeInTheDocument();
-      expect(screen.getByText(/Important: 1/)).toBeInTheDocument();
-      expect(screen.getByText(/Critical: 1/)).toBeInTheDocument();
-    });
-  });
-
-  it('should handle IOP mode API response correctly', async () => {
-    const mockResponse = {
-      low_hits: 2,
-      moderate_hits: 3,
-      important_hits: 1,
-      critical_hits: 2,
-      hits: 8,
-    };
-
-    APIHooks.useAPI.mockReturnValue({
-      status: STATUS.RESOLVED,
-      response: mockResponse,
-    });
-
-    renderComponent({
-      hostDetails: {
-        ...defaultHostDetails,
-        insights_attributes: {
-          ...defaultHostDetails.insights_attributes,
-          use_iop_mode: true,
-        },
-      },
-    });
-
-    await waitFor(() => {
-      // Check if total number of recommendations is displayed
-      expect(screen.getByText('8')).toBeInTheDocument();
-      // Check if risk levels are displayed correctly
-      expect(screen.getByText(/Low: 2/)).toBeInTheDocument();
-      expect(screen.getByText(/Moderate: 3/)).toBeInTheDocument();
-      expect(screen.getByText(/Important: 1/)).toBeInTheDocument();
-      expect(screen.getByText(/Critical: 2/)).toBeInTheDocument();
-    });
-  });
-
-  it('should show empty state when no recommendations exist', async () => {
-    APIHooks.useAPI.mockReturnValue({
-      status: STATUS.RESOLVED,
-      response: { hits: [] },
-    });
-
-    renderComponent();
-
-    await waitFor(() => {
-      expect(screen.getByText(/Low: 0/)).toBeInTheDocument();
-      expect(screen.getByText(/Moderate: 0/)).toBeInTheDocument();
-      expect(screen.getByText(/Important: 0/)).toBeInTheDocument();
-      expect(screen.getByText(/Critical: 0/)).toBeInTheDocument();
-    });
-  });
-
-  it('should use correct API endpoint based on IOP mode', () => {
-    renderComponent({
-      hostDetails: {
-        ...defaultHostDetails,
-        insights_attributes: {
-          ...defaultHostDetails.insights_attributes,
-          use_iop_mode: true,
-        },
-      },
-    });
-
-    expect(APIHooks.useAPI).toHaveBeenCalledWith(
-      'get',
-      expect.stringContaining('/api/insights/v1/system/test-uuid'),
-      expect.any(Object)
-    );
-
-    jest.clearAllMocks();
-
-    renderComponent();
-
-    expect(APIHooks.useAPI).toHaveBeenCalledWith(
-      'get',
-      expect.stringContaining('/hits/1'),
-      expect.any(Object)
-    );
-  });
-});