foreman_rh_cloud 13.2.10 → 14.0.1

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/PageDescription/PageDescription.js

@@ -7,8 +7,6 @@ import { getDocsURL } from 'foremanReact/common/helpers';
 import { FormattedMessage } from 'react-intl';
 import { selectSubscriptionConnectionEnabled } from '../../../InventorySettings/InventorySettingsSelectors';
 
-import { getSubscriptionServiceDocsUrl } from '../../../../ForemanInventoryHelpers';
-
 export const PageDescription = () => {
   const subscriptionConnectionEnabled = useSelector(
     selectSubscriptionConnectionEnabled
@@ -82,7 +80,7 @@ export const PageDescription = () => {
         {__('For more information about the Subscriptions service, see:')}
          
         <a
-          href={getSubscriptionServiceDocsUrl()}
+          href="https://docs.redhat.com/en/documentation/subscription_central/1-latest/html/getting_started_with_the_subscriptions_service/index"
           target="_blank"
           rel="noopener noreferrer"
         >

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/SyncButtonActions.js

@@ -39,20 +39,14 @@ export const setupInventorySyncTaskPolling = (id, dispatch) =>
     key: INVENTORY_SYNC_TASK_UPDATE,
     onTaskSuccess: ({
       output: {
-        host_statuses: { sync, disconnect, user_omitted: userOmitted },
+        host_statuses: { sync, disconnect },
       },
     }) =>
       dispatch(
         addToast({
           sticky: true,
           type: 'success',
-          message: (
-            <Toast
-              syncHosts={sync}
-              disconnectHosts={disconnect}
-              userOmittedHosts={userOmitted}
-            />
-          ),
+          message: <Toast syncHosts={sync} disconnectHosts={disconnect} />,
         })
       ),
     dispatch,

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/__tests__/__snapshots__/integrations.test.js.snap

@@ -9,7 +9,6 @@ Array [
           "message": <Toast
             disconnectHosts={2}
             syncHosts={0}
-            userOmittedHosts={1}
           />,
           "sticky": true,
           "type": "success",

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

@@ -30,7 +30,6 @@ describe('SyncButton integration test', () => {
                 host_statuses: {
                   sync: 0,
                   disconnect: 2,
-                  user_omitted: 1,
                 },
               },
               result: 'success',

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

@@ -1,55 +1,33 @@
 import React from 'react';
-import { Link } from 'react-router-dom';
 import PropTypes from 'prop-types';
 import { translate as __ } from 'foremanReact/common/I18n';
+import { foremanUrl } from '../../../../../../ForemanRhCloudHelpers';
 
-const statusSearchParams = statusName =>
-  `/new/hosts?search=insights_inventory_sync_status+%3D+${statusName}&page=1`;
-const DISCONNECT = 'disconnect';
-const SYNC = 'sync';
-const USER_OMITTED = 'user_omitted';
-const HostsWithStatusLink = ({ statusName, children }) => (
-  <Link to={statusSearchParams(statusName)}>{children}</Link>
-);
-HostsWithStatusLink.propTypes = {
-  statusName: PropTypes.string.isRequired,
-  children: PropTypes.node.isRequired,
-};
-
-const Toast = ({ syncHosts, disconnectHosts, userOmittedHosts }) => {
-  const totalHosts = syncHosts + disconnectHosts + userOmittedHosts;
+const Toast = ({ syncHosts, disconnectHosts }) => {
+  const totalHosts = syncHosts + disconnectHosts;
   return (
     <span>
       <p>
-        {__('Registered hosts in organization: ')}
-        <Link to="/new/hosts?search=set%3F+subscription_uuid&page=1">
-          {totalHosts}
-        </Link>
+        {__('Hosts with subscription in organization: ')}
+        <strong>{totalHosts}</strong>
       </p>
       <p>
-        {__('Uploaded and present on console.redhat.com Inventory service: ')}
-        <HostsWithStatusLink statusName={SYNC}>{syncHosts}</HostsWithStatusLink>
+        {__('Successfully synced hosts: ')}
+        <strong>{syncHosts}</strong>
       </p>
       <p>
-        {__('Not present on console.redhat.com Inventory service: ')}
-        <HostsWithStatusLink statusName={DISCONNECT}>
-          {disconnectHosts}
-        </HostsWithStatusLink>
+        {__('Disconnected hosts: ')}
+        <strong>{disconnectHosts}</strong>
       </p>
-      {!!userOmittedHosts && (
-        <p>
-          {__(
-            'Excluded from upload to console.redhat.com Inventory service because host_registration_insights_inventory parameter value is false: '
-          )}
-          <HostsWithStatusLink statusName={USER_OMITTED}>
-            {userOmittedHosts}
-          </HostsWithStatusLink>
-        </p>
-      )}
       <p>
-        {__(
-          'You can review this information later by looking at the Inventory status of each host.'
-        )}
+        {__('For more info, please visit the')}{' '}
+        <a
+          href={foremanUrl('/hosts')}
+          target="_blank"
+          rel="noopener noreferrer"
+        >
+          {__('hosts page')}
+        </a>
       </p>
     </span>
   );
@@ -58,10 +36,6 @@ const Toast = ({ syncHosts, disconnectHosts, userOmittedHosts }) => {
 Toast.propTypes = {
   syncHosts: PropTypes.number.isRequired,
   disconnectHosts: PropTypes.number.isRequired,
-  userOmittedHosts: PropTypes.number,
-};
-Toast.defaultProps = {
-  userOmittedHosts: 0,
 };
 
 export default Toast;

data/webpack/ForemanInventoryUpload/ForemanInventoryHelpers.js

@@ -11,13 +11,6 @@ export const getInventoryDocsUrl = () =>
     )}`
   );
 
-export const getSubscriptionServiceDocsUrl = () =>
-  foremanUrl(
-    `/links/manual/?root_url=${URI.encode(
-      'https://docs.redhat.com/en/documentation/subscription_central/1-latest/html-single/getting_started_with_the_subscriptions_service/index'
-    )}`
-  );
-
 export const getActionsHistoryUrl = () =>
   foremanUrl(
     '/foreman_tasks/tasks?search=label+%3D+ForemanInventoryUpload%3A%3AAsync%3A%3AHostInventoryReportJob+or+label+%3D+ForemanInventoryUpload%3A%3AAsync%3A%3AGenerateAllReportsJob&page=1'

data/webpack/ForemanRhCloudHelpers.js

@@ -1,6 +1,3 @@
-import { useEffect } from 'react';
-import { useHistory } from 'react-router-dom';
-
 /**
  * copied from core, since it's not in the ReactApp folder,
  * it's complicated to import it and mock it in tests.
@@ -8,25 +5,6 @@ import { useHistory } from 'react-router-dom';
  */
 export const foremanUrl = path => `${window.URL_PREFIX}${path}`;
 
-export const OVERVIEW_TAB_PATH = '/Overview';
-
-/**
- * Redirects to Overview tab when the current tab should be hidden
- * @param {boolean} shouldRedirect - Whether to redirect (e.g., host loaded AND tab should hide)
- * @returns {boolean} - Returns shouldRedirect for convenience
- */
-export const useTabRedirect = shouldRedirect => {
-  const history = useHistory();
-
-  useEffect(() => {
-    if (shouldRedirect && history) {
-      history.replace(OVERVIEW_TAB_PATH);
-    }
-  }, [shouldRedirect, history]);
-
-  return shouldRedirect;
-};
-
 export const isNotRhelHost = ({ hostDetails }) =>
   // This regex tries matches sane variations of "RedHat", "RHEL" and "RHCOS"
   !new RegExp('red[\\s\\-]?hat|rh[\\s\\-]?el|rhc[\\s\\-]?os', 'i').test(

data/webpack/InsightsHostDetailsTab/NewHostDetailsTab.js

@@ -25,11 +25,6 @@ import { useIopConfig } from '../common/Hooks/ConfigHooks';
 import { generateRuleUrl } from '../InsightsCloudSync/InsightsCloudSync';
 import { createProviderOptions } from '../common/ScalprumModule/ScalprumContext';
 import { useInsightsPermissions } from '../common/Hooks/PermissionsHooks';
-import {
-  isNotRhelHost,
-  hasNoInsightsFacet,
-  useTabRedirect,
-} from '../ForemanRhCloudHelpers';
 
 // Hosted Insights advisor
 const NewHostDetailsTab = ({ hostName, router }) => {
@@ -38,28 +33,13 @@ const NewHostDetailsTab = ({ hostName, router }) => {
   const hits = useSelector(selectHits);
   const isIop = useIopConfig();
 
-  useEffect(
-    () => () => {
-      // Preserve hash when clearing search params to prevent tab navigation bugs
-      if (router && typeof router.replace === 'function') {
-        const replaceOptions = { search: null };
-        if (router.location && router.location.hash) {
-          replaceOptions.hash = router.location.hash;
-        }
-        router.replace(replaceOptions);
-      }
-    },
-    [router]
-  );
+  useEffect(() => () => router.replace({ search: null }), [router]);
 
   const onSearch = q => dispatch(fetchInsights({ query: q, page: 1 }));
 
   const [isDropdownOpen, setIsDropdownOpen] = useState(false);
-  const onSatInsightsClick = () => {
-    if (router && typeof router.push === 'function') {
-      router.push({ pathname: '/foreman_rh_cloud/insights_cloud' });
-    }
-  };
+  const onSatInsightsClick = () =>
+    router.push({ pathname: '/foreman_rh_cloud/insights_cloud' });
 
   const dropdownItems = [
     <DropdownItem key="insights-link" ouiaId="insights-link">
@@ -133,32 +113,17 @@ NewHostDetailsTab.defaultProps = {
 const scope = 'advisor';
 const module = './SystemDetailWrapped';
 
-const IopInsightsTab = props => {
-  // eslint-disable-next-line camelcase
-  const systemId = props.response?.subscription_facet_attributes?.uuid;
-  return (
-    <div className="advisor">
-      <ScalprumComponent
-        key={systemId || props.hostName}
-        scope={scope}
-        module={module}
-        IopRemediationModal={RemediationModal}
-        generateRuleUrl={generateRuleUrl}
-        {...props}
-      />
-    </div>
-  );
-};
-
-IopInsightsTab.propTypes = {
-  hostName: PropTypes.string,
-  response: PropTypes.object,
-};
-
-IopInsightsTab.defaultProps = {
-  hostName: '',
-  response: {},
-};
+const IopInsightsTab = props => (
+  <div className="advisor">
+    <ScalprumComponent
+      scope={scope}
+      module={module}
+      IopRemediationModal={RemediationModal}
+      generateRuleUrl={generateRuleUrl}
+      {...props}
+    />
+  </div>
+);
 
 const IopInsightsTabWrapped = props => {
   const permissions = useInsightsPermissions();
@@ -170,18 +135,7 @@ const IopInsightsTabWrapped = props => {
 };
 
 const InsightsTab = props => {
-  const { response } = props;
   const isIop = useIopConfig();
-  const isHostDataLoaded = Boolean(response?.id);
-  const shouldHideTab = useTabRedirect(
-    isHostDataLoaded &&
-      (isNotRhelHost({ hostDetails: response }) ||
-        hasNoInsightsFacet({ response, hostDetails: response }))
-  );
-
-  if (shouldHideTab) {
-    return null;
-  }
 
   return isIop ? (
     <IopInsightsTabWrapped {...props} />
@@ -190,16 +144,6 @@ const InsightsTab = props => {
   );
 };
 
-InsightsTab.propTypes = {
-  response: PropTypes.shape({
-    id: PropTypes.number,
-    operatingsystem_name: PropTypes.string,
-    insights_attributes: PropTypes.object,
-  }),
-};
-
-InsightsTab.defaultProps = {
-  response: {},
-};
+InsightsTab.defaultProps = {};
 
 export default InsightsTab;

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_rh_cloud
 version: !ruby/object:Gem::Version
-  version: 13.2.10
+  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
@@ -30,9 +31,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 10.0.0
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: 13.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -40,9 +38,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 10.0.0
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: 13.0.0
 - !ruby/object:Gem::Dependency
   name: katello
   requirement: !ruby/object:Gem::Requirement
@@ -107,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
@@ -240,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
@@ -271,7 +274,6 @@ files:
 - test/jobs/host_inventory_report_job_test.rb
 - test/jobs/insights_client_status_aging_test.rb
 - test/jobs/insights_full_sync_test.rb
-- test/jobs/insights_generate_notifications_test.rb
 - test/jobs/insights_resolutions_sync_test.rb
 - test/jobs/insights_rules_sync_test.rb
 - test/jobs/inventory_full_sync_test.rb
@@ -283,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
@@ -458,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
@@ -597,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
@@ -663,6 +662,7 @@ homepage: https://github.com/theforeman/foreman_rh_cloud
 licenses:
 - GPL-3.0
 metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -680,7 +680,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.16
+rubygems_version: 3.2.33
+signing_key:
 specification_version: 4
 summary: Summary of ForemanRhCloud.
 test_files:
@@ -688,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
@@ -704,7 +704,6 @@ test_files:
 - test/jobs/host_inventory_report_job_test.rb
 - test/jobs/insights_client_status_aging_test.rb
 - test/jobs/insights_full_sync_test.rb
-- test/jobs/insights_generate_notifications_test.rb
 - test/jobs/insights_resolutions_sync_test.rb
 - test/jobs/insights_rules_sync_test.rb
 - test/jobs/inventory_full_sync_test.rb
@@ -716,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/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