foreman_rh_cloud 13.2.11 → 14.0.1

data/test/jobs/insights_generate_notifications_test.rb

@@ -1,26 +0,0 @@
-require 'test_plugin_helper'
-require 'foreman_tasks/test_helpers'
-
-class InsightsGenerateNotificationsTest < ActiveSupport::TestCase
-  include Dynflow::Testing::Factories
-
-  setup do
-    User.current = User.find_by(login: 'secret_admin')
-  end
-
-  test 'skips notifications when foreman_host is nil' do
-    ForemanRhCloud.stubs(:foreman_host).returns(nil)
-
-    # Ensure blueprint exists or create it
-    NotificationBlueprint.find_or_create_by(name: 'insights_satellite_hits') do |bp|
-      bp.message = 'Test message'
-      bp.level = 'info'
-      bp.expires_in = 7.days
-    end
-
-    plan = ForemanTasks.sync_task(InsightsCloud::Async::InsightsGenerateNotifications)
-
-    # Should not raise an error, task completes successfully
-    assert_includes ['success', 'stopped'], plan.state, "Task should complete without error"
-  end
-end

data/test/models/inventory_sync/inventory_status_test.rb

@@ -1,85 +0,0 @@
-require 'test_plugin_helper'
-
-module InventorySync
-  class InventoryStatusTest < ActiveSupport::TestCase
-    setup do
-      @host = FactoryBot.create(:host, :managed)
-    end
-
-    test 'status constants are defined correctly' do
-      assert_equal 0, InventorySync::InventoryStatus::DISCONNECT
-      assert_equal 1, InventorySync::InventoryStatus::SYNC
-      assert_equal 2, InventorySync::InventoryStatus::USER_OMITTED
-    end
-
-    test 'to_global returns OK for USER_OMITTED status' do
-      status = @host.get_status(InventorySync::InventoryStatus)
-      status.status = InventorySync::InventoryStatus::USER_OMITTED
-      status.save!
-
-      assert_equal HostStatus::Global::OK, status.to_global
-    end
-
-    test 'to_global returns WARN for DISCONNECT status' do
-      status = @host.get_status(InventorySync::InventoryStatus)
-      status.status = InventorySync::InventoryStatus::DISCONNECT
-      status.save!
-
-      assert_equal HostStatus::Global::WARN, status.to_global
-    end
-
-    test 'to_global returns OK for SYNC status' do
-      status = @host.get_status(InventorySync::InventoryStatus)
-      status.status = InventorySync::InventoryStatus::SYNC
-      status.save!
-
-      assert_equal HostStatus::Global::OK, status.to_global
-    end
-
-    test 'to_label returns appropriate messages for each status' do
-      status = @host.get_status(InventorySync::InventoryStatus)
-
-      # Test DISCONNECT label
-      status.status = InventorySync::InventoryStatus::DISCONNECT
-      status.save!
-      label = status.to_label
-      assert_match(/not present/, label.downcase)
-      assert_match(/console\.redhat\.com/, label)
-
-      # Test SYNC label
-      status.status = InventorySync::InventoryStatus::SYNC
-      status.save!
-      label = status.to_label
-      assert_match(/uploaded.*present/, label.downcase)
-      assert_match(/console\.redhat\.com/, label)
-
-      # Test USER_OMITTED label
-      status.status = InventorySync::InventoryStatus::USER_OMITTED
-      status.save!
-      label = status.to_label
-      assert_match(/excluded/, label.downcase)
-      assert_match(/host parameter/, label.downcase)
-      assert_match(/console\.redhat\.com/, label)
-    end
-
-    test 'relevant? returns true in regular (non-IoP) mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-
-      status = @host.get_status(InventorySync::InventoryStatus)
-      status.status = InventorySync::InventoryStatus::SYNC
-      status.save!
-
-      assert status.relevant?, 'Inventory status should be relevant in regular mode'
-    end
-
-    test 'relevant? returns false in IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-
-      status = @host.get_status(InventorySync::InventoryStatus)
-      status.status = InventorySync::InventoryStatus::SYNC
-      status.save!
-
-      refute status.relevant?, 'Inventory status should NOT be relevant in IoP mode'
-    end
-  end
-end

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/components/__tests__/Toast.test.js

@@ -1,82 +0,0 @@
-import React from 'react';
-import { shallow } from '@theforeman/test';
-import Toast from '../Toast';
-
-describe('Toast', () => {
-  it('renders with all three status counts including user_omitted', () => {
-    const wrapper = shallow(
-      <Toast syncHosts={5} disconnectHosts={3} userOmittedHosts={2} />
-    );
-
-    const links = wrapper.find('HostsWithStatusLink');
-    expect(links).toHaveLength(3);
-
-    // Check the children (numbers) of each link
-    expect(
-      links
-        .at(0)
-        .children()
-        .text()
-    ).toBe('5');
-    expect(
-      links
-        .at(1)
-        .children()
-        .text()
-    ).toBe('3');
-    expect(
-      links
-        .at(2)
-        .children()
-        .text()
-    ).toBe('2');
-  });
-
-  it('does not render user_omitted section when count is 0', () => {
-    const wrapper = shallow(
-      <Toast syncHosts={5} disconnectHosts={3} userOmittedHosts={0} />
-    );
-
-    // Should have only 2 HostsWithStatusLink components (sync and disconnect)
-    const links = wrapper.find('HostsWithStatusLink');
-    expect(links).toHaveLength(2);
-
-    // Should not contain the user_omitted explanation text
-    expect(wrapper.text()).not.toContain(
-      'host_registration_insights_inventory parameter value is false'
-    );
-  });
-
-  it('renders without crashing when userOmittedHosts is not provided (default)', () => {
-    const wrapper = shallow(<Toast syncHosts={5} disconnectHosts={3} />);
-
-    // Should use default value of 0, so only 2 links
-    const links = wrapper.find('HostsWithStatusLink');
-    expect(links).toHaveLength(2);
-
-    // Verify the count values
-    expect(
-      links
-        .at(0)
-        .children()
-        .text()
-    ).toBe('5');
-    expect(
-      links
-        .at(1)
-        .children()
-        .text()
-    ).toBe('3');
-  });
-
-  it('renders correct status links for each category', () => {
-    const wrapper = shallow(
-      <Toast syncHosts={5} disconnectHosts={3} userOmittedHosts={2} />
-    );
-
-    const links = wrapper.find('HostsWithStatusLink');
-    expect(links.at(0).prop('statusName')).toBe('sync');
-    expect(links.at(1).prop('statusName')).toBe('disconnect');
-    expect(links.at(2).prop('statusName')).toBe('user_omitted');
-  });
-});

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

@@ -1,266 +0,0 @@
-import React from 'react';
-import { render } from '@testing-library/react';
-import '@testing-library/jest-dom';
-import { Provider } from 'react-redux';
-import { MemoryRouter } from 'react-router-dom';
-import configureMockStore from 'redux-mock-store';
-import thunk from 'redux-thunk';
-import NewHostDetailsTab from '../NewHostDetailsTab';
-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('../../common/Hooks/ConfigHooks', () => ({
-  useIopConfig: jest.fn(() => false),
-}));
-
-jest.mock('foremanReact/common/I18n', () => ({
-  translate: jest.fn(str => str),
-}));
-
-jest.mock(
-  'foremanReact/components/SearchBar',
-  () => () => <div>SearchBar</div>,
-  { virtual: true }
-);
-
-jest.mock('../../InsightsCloudSync/Components/InsightsTable', () => () => (
-  <div>InsightsTable</div>
-));
-
-jest.mock('../../InsightsCloudSync/Components/RemediationModal', () => () => (
-  <div>RemediationModal</div>
-));
-
-jest.mock(
-  '../../InsightsCloudSync/Components/InsightsTable/Pagination',
-  () => () => <div>Pagination</div>
-);
-
-const mockStore = configureMockStore([thunk]);
-
-const defaultResponse = {
-  id: 1,
-  operatingsystem_name: 'Red Hat Enterprise Linux 8',
-  insights_attributes: { uuid: 'test-uuid' },
-};
-
-describe('NewHostDetailsTab', () => {
-  let store;
-  let mockRouter;
-
-  beforeEach(() => {
-    mockRouter = {
-      push: jest.fn(),
-      replace: jest.fn(),
-      location: {
-        pathname: '/new/hosts/test-host.example.com',
-        search: '?page=1&per_page=20',
-        hash: '#/Insights',
-        query: { page: '1', per_page: '20' },
-      },
-    };
-
-    store = mockStore({
-      API: {},
-      ForemanRhCloud: {
-        InsightsCloudSync: {
-          table: {
-            selectedIds: {},
-            isAllSelected: false,
-            showSelectAllAlert: false,
-          },
-        },
-      },
-      insightsHostDetailsTab: {
-        query: '',
-        hits: [],
-        selectedIds: {},
-        error: null,
-      },
-      router: {
-        location: {
-          pathname: '/new/hosts/test-host.example.com',
-          search: '?page=1&per_page=20',
-          hash: '#/Insights',
-          query: { page: '1', per_page: '20' },
-        },
-      },
-    });
-  });
-
-  afterEach(() => {
-    jest.clearAllMocks();
-  });
-
-  describe('cleanup effect', () => {
-    it('should preserve hash when clearing search params on unmount', () => {
-      const { unmount } = render(
-        <Provider store={store}>
-          <MemoryRouter>
-            <NewHostDetailsTab
-              hostName="test-host.example.com"
-              router={mockRouter}
-              response={defaultResponse}
-            />
-          </MemoryRouter>
-        </Provider>
-      );
-
-      unmount();
-
-      expect(mockRouter.replace).toHaveBeenCalledWith({
-        search: null,
-        hash: '#/Insights',
-      });
-    });
-
-    it('should only clear search params when no hash exists', () => {
-      mockRouter.location.hash = '';
-
-      const { unmount } = render(
-        <Provider store={store}>
-          <MemoryRouter>
-            <NewHostDetailsTab
-              hostName="test-host.example.com"
-              router={mockRouter}
-              response={defaultResponse}
-            />
-          </MemoryRouter>
-        </Provider>
-      );
-
-      unmount();
-
-      expect(mockRouter.replace).toHaveBeenCalledWith({
-        search: null,
-      });
-    });
-
-    it('should handle router.location being undefined gracefully', () => {
-      const routerWithoutLocation = {
-        push: jest.fn(),
-        replace: jest.fn(),
-        location: undefined,
-      };
-
-      const { unmount } = render(
-        <Provider store={store}>
-          <MemoryRouter>
-            <NewHostDetailsTab
-              hostName="test-host.example.com"
-              router={routerWithoutLocation}
-              response={defaultResponse}
-            />
-          </MemoryRouter>
-        </Provider>
-      );
-
-      unmount();
-
-      expect(routerWithoutLocation.replace).toHaveBeenCalledWith({
-        search: null,
-      });
-    });
-
-    it('should use the latest hash value at unmount time, not a stale captured value', () => {
-      const { unmount } = render(
-        <Provider store={store}>
-          <MemoryRouter>
-            <NewHostDetailsTab
-              hostName="test-host.example.com"
-              router={mockRouter}
-              response={defaultResponse}
-            />
-          </MemoryRouter>
-        </Provider>
-      );
-
-      mockRouter.location.hash = '#/Overview';
-
-      unmount();
-
-      expect(mockRouter.replace).toHaveBeenCalledWith({
-        search: null,
-        hash: '#/Overview',
-      });
-    });
-  });
-
-  describe('tab visibility', () => {
-    it('should redirect to Overview when host is not RHEL', () => {
-      const nonRhelResponse = {
-        id: 2,
-        operatingsystem_name: 'Ubuntu 20.04',
-        insights_attributes: { uuid: 'test-uuid' },
-      };
-
-      render(
-        <Provider store={store}>
-          <MemoryRouter>
-            <NewHostDetailsTab
-              hostName="test-host.example.com"
-              response={nonRhelResponse}
-            />
-          </MemoryRouter>
-        </Provider>
-      );
-
-      expect(mockHistoryReplace).toHaveBeenCalledWith(OVERVIEW_TAB_PATH);
-    });
-
-    it('should redirect to Overview when insights facet is missing', () => {
-      const responseWithoutInsights = {
-        id: 3,
-        operatingsystem_name: 'Red Hat Enterprise Linux 8',
-      };
-
-      render(
-        <Provider store={store}>
-          <MemoryRouter>
-            <NewHostDetailsTab
-              hostName="test-host.example.com"
-              response={responseWithoutInsights}
-            />
-          </MemoryRouter>
-        </Provider>
-      );
-
-      expect(mockHistoryReplace).toHaveBeenCalledWith(OVERVIEW_TAB_PATH);
-    });
-
-    it('should not redirect when host is valid RHEL with insights facet', () => {
-      render(
-        <Provider store={store}>
-          <MemoryRouter>
-            <NewHostDetailsTab
-              hostName="test-host.example.com"
-              response={defaultResponse}
-            />
-          </MemoryRouter>
-        </Provider>
-      );
-
-      expect(mockHistoryReplace).not.toHaveBeenCalled();
-    });
-
-    it('should not redirect when host data is not yet loaded', () => {
-      render(
-        <Provider store={store}>
-          <MemoryRouter>
-            <NewHostDetailsTab hostName="test-host.example.com" response={{}} />
-          </MemoryRouter>
-        </Provider>
-      );
-
-      expect(mockHistoryReplace).not.toHaveBeenCalled();
-    });
-  });
-});