foreman_rh_cloud 13.2.9 → 14.0.1

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_rh_cloud
 version: !ruby/object:Gem::Version
-  version: 13.2.9
+  version: 14.0.1
 platform: ruby
 authors:
 - Foreman Red Hat Cloud team
+autorequire:
 bindir: bin
 cert_chain: []
-date: 1980-01-02 00:00:00.000000000 Z
+date: 2026-02-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: foreman_ansible
@@ -101,7 +102,6 @@ files:
 - app/controllers/concerns/foreman_rh_cloud/iop_smart_proxy_access.rb
 - app/controllers/concerns/foreman_rh_cloud/registration_manager_extensions.rb
 - app/controllers/concerns/insights_cloud/candlepin_cache.rb
-- app/controllers/concerns/insights_cloud/candlepin_proxies_extensions.rb
 - app/controllers/concerns/insights_cloud/client_authentication.rb
 - app/controllers/concerns/insights_cloud/package_profile_upload_extensions.rb
 - app/controllers/concerns/inventory_upload/report_actions.rb
@@ -234,22 +234,31 @@ files:
 - locale/Makefile
 - locale/foreman_rh_cloud.pot
 - locale/fr/LC_MESSAGES/foreman_rh_cloud.mo
+- locale/fr/foreman_rh_cloud.edit.po
 - locale/fr/foreman_rh_cloud.po
+- locale/fr/foreman_rh_cloud.po.time_stamp
 - locale/gemspec.rb
 - locale/ja/LC_MESSAGES/foreman_rh_cloud.mo
+- locale/ja/foreman_rh_cloud.edit.po
 - locale/ja/foreman_rh_cloud.po
+- locale/ja/foreman_rh_cloud.po.time_stamp
 - locale/ka/LC_MESSAGES/foreman_rh_cloud.mo
+- locale/ka/foreman_rh_cloud.edit.po
 - locale/ka/foreman_rh_cloud.po
+- locale/ka/foreman_rh_cloud.po.time_stamp
 - locale/ko/LC_MESSAGES/foreman_rh_cloud.mo
+- locale/ko/foreman_rh_cloud.edit.po
 - locale/ko/foreman_rh_cloud.po
+- locale/ko/foreman_rh_cloud.po.time_stamp
 - locale/zh_CN/LC_MESSAGES/foreman_rh_cloud.mo
+- locale/zh_CN/foreman_rh_cloud.edit.po
 - locale/zh_CN/foreman_rh_cloud.po
+- locale/zh_CN/foreman_rh_cloud.po.time_stamp
 - package.json
 - test/controllers/accounts_controller_test.rb
 - test/controllers/insights_cloud/api/advisor_engine_controller_test.rb
 - test/controllers/insights_cloud/api/cloud_request_controller_test.rb
 - test/controllers/insights_cloud/api/machine_telemetries_controller_test.rb
-- test/controllers/insights_cloud/candlepin_proxies_extensions_test.rb
 - test/controllers/insights_cloud/ui_requests_controller_test.rb
 - test/controllers/insights_sync/settings_controller_test.rb
 - test/controllers/inventory_upload/api/inventory_controller_test.rb
@@ -276,7 +285,6 @@ files:
 - test/jobs/single_host_report_job_test.rb
 - test/jobs/upload_report_direct_job_test.rb
 - test/models/insights_client_report_status_test.rb
-- test/models/inventory_sync/inventory_status_test.rb
 - test/test_plugin_helper.rb
 - test/unit/archived_report_generator_test.rb
 - test/unit/fact_helpers_test.rb
@@ -451,7 +459,6 @@ files:
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/__tests__/__snapshots__/integrations.test.js.snap
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/__tests__/integrations.test.js
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/components/Toast.js
-- webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/components/__tests__/Toast.test.js
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/index.js
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/ToolbarButtons/ToolbarButtons.js
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/ToolbarButtons/__tests__/ToolbarButtons.test.js
@@ -590,7 +597,6 @@ files:
 - webpack/InsightsHostDetailsTab/__tests__/InsightsTabReducer.test.js
 - webpack/InsightsHostDetailsTab/__tests__/InsightsTabSelectors.test.js
 - webpack/InsightsHostDetailsTab/__tests__/InsightsTotalRiskChart.test.js
-- webpack/InsightsHostDetailsTab/__tests__/NewHostDetailsTab.test.js
 - webpack/InsightsHostDetailsTab/__tests__/__snapshots__/InsightsTab.test.js.snap
 - webpack/InsightsHostDetailsTab/__tests__/__snapshots__/InsightsTabActions.test.js.snap
 - webpack/InsightsHostDetailsTab/__tests__/__snapshots__/InsightsTabReducer.test.js.snap
@@ -656,6 +662,7 @@ homepage: https://github.com/theforeman/foreman_rh_cloud
 licenses:
 - GPL-3.0
 metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -673,7 +680,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.6
+rubygems_version: 3.2.33
+signing_key:
 specification_version: 4
 summary: Summary of ForemanRhCloud.
 test_files:
@@ -681,7 +689,6 @@ test_files:
 - test/controllers/insights_cloud/api/advisor_engine_controller_test.rb
 - test/controllers/insights_cloud/api/cloud_request_controller_test.rb
 - test/controllers/insights_cloud/api/machine_telemetries_controller_test.rb
-- test/controllers/insights_cloud/candlepin_proxies_extensions_test.rb
 - test/controllers/insights_cloud/ui_requests_controller_test.rb
 - test/controllers/insights_sync/settings_controller_test.rb
 - test/controllers/inventory_upload/api/inventory_controller_test.rb
@@ -708,7 +715,6 @@ test_files:
 - test/jobs/single_host_report_job_test.rb
 - test/jobs/upload_report_direct_job_test.rb
 - test/models/insights_client_report_status_test.rb
-- test/models/inventory_sync/inventory_status_test.rb
 - test/test_plugin_helper.rb
 - test/unit/archived_report_generator_test.rb
 - test/unit/fact_helpers_test.rb

data/app/controllers/concerns/insights_cloud/candlepin_proxies_extensions.rb

@@ -1,23 +0,0 @@
-module InsightsCloud
-  module CandlepinProxiesExtensions
-    extend ActiveSupport::Concern
-
-    included do
-      # Update insights client status when hosts check in via subscription-manager facts upload
-      # rubocop:disable Rails/LexicallyScopedActionFilter
-      after_action :update_insights_client_status, only: [:facts]
-      # rubocop:enable Rails/LexicallyScopedActionFilter
-    end
-
-    def update_insights_client_status
-      # Update InsightsClientReportStatus whenever host checks in via subscription-manager
-      # This ensures USER_OMITTED status gets set even when insights-client isn't installed
-      # (parameter=false means insights-client won't be installed, so it won't hit MachineTelemetriesController)
-      hoststatus = @host.get_status(InsightsClientReportStatus)
-      Rails.logger.debug "Current status: #{hoststatus.to_label}"
-      @host.get_status(InsightsClientReportStatus).refresh!
-      @host.refresh_global_status!
-      Rails.logger.debug "New status: #{hoststatus.to_label}"
-    end
-  end
-end

data/test/controllers/insights_cloud/candlepin_proxies_extensions_test.rb

@@ -1,70 +0,0 @@
-require 'test_plugin_helper'
-
-module InsightsCloud
-  class CandlepinProxiesExtensionsTest < ActiveSupport::TestCase
-    setup do
-      @organization = FactoryBot.create(:organization)
-      @host = FactoryBot.create(:host, :with_subscription, :managed, organization: @organization)
-    end
-
-    test 'updates InsightsClientReportStatus to USER_OMITTED when parameter is false' do
-      # Set parameter to false so status should be USER_OMITTED
-      FactoryBot.create(:common_parameter, name: 'host_registration_insights', key_type: 'boolean', value: false)
-
-      # Simulate what the callback does
-      @host.get_status(InsightsClientReportStatus).refresh!
-      @host.refresh_global_status!
-
-      # Verify the status was updated
-      @host.reload
-      insights_status = @host.get_status(InsightsClientReportStatus)
-      assert insights_status.persisted?, 'InsightsClientReportStatus should be persisted'
-      assert_equal InsightsClientReportStatus::USER_OMITTED, insights_status.status,
-        'Status should be USER_OMITTED when host_registration_insights=false'
-    end
-
-    test 'sets USER_OMITTED status when parameter is inherited from hostgroup' do
-      # Create hostgroup with parameter
-      hostgroup = FactoryBot.create(:hostgroup)
-      hostgroup.group_parameters << GroupParameter.create(
-        name: 'host_registration_insights',
-        value: 'false',
-        key_type: 'boolean'
-      )
-      hostgroup.save!
-
-      @host.hostgroup = hostgroup
-      @host.save!
-
-      # Simulate what the callback does
-      @host.get_status(InsightsClientReportStatus).refresh!
-      @host.refresh_global_status!
-
-      # Verify status respects inherited parameter
-      @host.reload
-      insights_status = @host.get_status(InsightsClientReportStatus)
-      assert_equal InsightsClientReportStatus::USER_OMITTED, insights_status.status,
-        'Status should be USER_OMITTED when host_registration_insights=false is inherited from hostgroup'
-    end
-
-    test 'refreshes global status' do
-      FactoryBot.create(:common_parameter, name: 'host_registration_insights', key_type: 'boolean', value: true)
-
-      # Simulate what the callback does
-      @host.get_status(InsightsClientReportStatus).refresh!
-      @host.refresh_global_status!
-
-      # Verify global status was updated
-      @host.reload
-      assert_not_nil @host.global_status, 'Global status should be set'
-    end
-
-    test 'CandlepinProxiesController includes the update_insights_client_status method' do
-      # Verify the concern is included and method is available
-      controller = Katello::Api::Rhsm::CandlepinProxiesController.new
-
-      assert_respond_to controller, :update_insights_client_status,
-        'CandlepinProxiesController should have update_insights_client_status method from concern'
-    end
-  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,154 +0,0 @@
-import React from 'react';
-import { render } from '@testing-library/react';
-import '@testing-library/jest-dom';
-import { Provider } from 'react-redux';
-import configureMockStore from 'redux-mock-store';
-import thunk from 'redux-thunk';
-import NewHostDetailsTab from '../NewHostDetailsTab';
-
-jest.mock('../../common/Hooks/ConfigHooks', () => ({
-  useIopConfig: jest.fn(() => false),
-}));
-
-jest.mock('foremanReact/common/I18n', () => ({
-  translate: jest.fn(str => str),
-}));
-
-const mockStore = configureMockStore([thunk]);
-
-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}>
-          <NewHostDetailsTab
-            hostName="test-host.example.com"
-            router={mockRouter}
-          />
-        </Provider>
-      );
-
-      // Unmount the component to trigger cleanup
-      unmount();
-
-      // Verify router.replace was called with both search: null AND the existing hash
-      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}>
-          <NewHostDetailsTab
-            hostName="test-host.example.com"
-            router={mockRouter}
-          />
-        </Provider>
-      );
-
-      unmount();
-
-      // When there's no hash, should only pass search: null
-      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}>
-          <NewHostDetailsTab
-            hostName="test-host.example.com"
-            router={routerWithoutLocation}
-          />
-        </Provider>
-      );
-
-      unmount();
-
-      // Should still call replace with search: null even if location is undefined
-      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}>
-          <NewHostDetailsTab
-            hostName="test-host.example.com"
-            router={mockRouter}
-          />
-        </Provider>
-      );
-
-      // Change the hash after mount, before unmount
-      mockRouter.location.hash = '#/Overview';
-
-      unmount();
-
-      // Verify router.replace was called with the UPDATED hash, not the initial '#/Insights'
-      expect(mockRouter.replace).toHaveBeenCalledWith({
-        search: null,
-        hash: '#/Overview',
-      });
-    });
-  });
-});