foreman_rh_cloud 12.2.13 → 13.0.0

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

@@ -1,192 +0,0 @@
-require 'test_plugin_helper'
-
-module ForemanRhCloud
-  class RegistrationManagerExtensionsTest < ActiveSupport::TestCase
-    setup do
-      @org = FactoryBot.create(:organization)
-      @host = FactoryBot.create(:host, :managed, organization: @org)
-      @insights_facet = ::InsightsFacet.create!(host: @host, uuid: 'test-uuid-123')
-
-      # 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 facet with UUID' do
-        ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-        expected_url = ForemanInventoryUpload.host_by_id_url('test-uuid-123')
-
-        # 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 NOT call HBI delete when host has no insights_facet' do
-        host_without_facet = FactoryBot.create(:host, :managed, organization: @org)
-        ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-
-        Katello::RegistrationManager.expects(:execute_cloud_request).never
-
-        assert_nothing_raised do
-          Katello::RegistrationManager.unregister_host(host_without_facet, unregistering: true)
-        end
-      end
-
-      test 'should NOT call HBI delete when host has insights_facet with empty UUID' do
-        host_with_empty_uuid_facet = FactoryBot.create(:host, :managed, organization: @org)
-        empty_uuid_facet = InsightsFacet.create!(host: host_with_empty_uuid_facet, uuid: '')
-        ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-
-        Katello::RegistrationManager.expects(:execute_cloud_request).never
-
-        assert_nothing_raised do
-          Katello::RegistrationManager.unregister_host(host_with_empty_uuid_facet, unregistering: true)
-        end
-
-        assert_nil InsightsFacet.find_by(id: empty_uuid_facet.id)
-      end
-
-      test 'should NOT call HBI delete when insights_facet has no UUID' do
-        facet_id = @insights_facet.id
-        @insights_facet.update(uuid: nil)
-        ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-
-        Katello::RegistrationManager.expects(:execute_cloud_request).never
-
-        Katello::RegistrationManager.unregister_host(@host, unregistering: true)
-
-        # Verify facet was still destroyed
-        assert_nil InsightsFacet.find_by(id: 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/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)
-    );
-  });
-});