foreman_rh_cloud 12.2.17 → 13.0.0

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_rh_cloud
 version: !ruby/object:Gem::Version
-  version: 12.2.17
+  version: 13.0.0
 platform: ruby
 authors:
 - Foreman Red Hat Cloud team
@@ -43,14 +43,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.18'
+        version: 4.14.0.rc1.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.18'
+        version: 4.14.0.rc1.1
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
@@ -99,7 +99,6 @@ files:
 - app/controllers/api/v2/rh_cloud/cloud_request_controller.rb
 - app/controllers/api/v2/rh_cloud/inventory_controller.rb
 - 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/client_authentication.rb
 - app/controllers/concerns/insights_cloud/package_profile_upload_extensions.rb
@@ -140,6 +139,7 @@ files:
 - app/services/foreman_rh_cloud/cloud_presence.rb
 - app/services/foreman_rh_cloud/cloud_request.rb
 - app/services/foreman_rh_cloud/cloud_request_forwarder.rb
+- app/services/foreman_rh_cloud/gateway_request.rb
 - app/services/foreman_rh_cloud/hit_remediations_retriever.rb
 - app/services/foreman_rh_cloud/hits_uploader.rb
 - app/services/foreman_rh_cloud/insights_api_forwarder.rb
@@ -185,18 +185,14 @@ files:
 - db/seeds.d/50_job_templates.rb
 - lib/foreman_inventory_upload.rb
 - lib/foreman_inventory_upload/async/async_helpers.rb
-- lib/foreman_inventory_upload/async/create_missing_insights_facets.rb
 - lib/foreman_inventory_upload/async/delayed_start.rb
 - lib/foreman_inventory_upload/async/generate_all_reports_job.rb
-- lib/foreman_inventory_upload/async/generate_host_report.rb
 - lib/foreman_inventory_upload/async/generate_report_job.rb
-- lib/foreman_inventory_upload/async/host_inventory_report_job.rb
 - lib/foreman_inventory_upload/async/progress_output.rb
 - lib/foreman_inventory_upload/async/queue_for_upload_job.rb
 - lib/foreman_inventory_upload/async/remove_insights_hosts_job.rb
 - lib/foreman_inventory_upload/async/shell_process.rb
-- lib/foreman_inventory_upload/async/single_host_report_job.rb
-- lib/foreman_inventory_upload/async/upload_report_direct_job.rb
+- lib/foreman_inventory_upload/async/upload_report_job.rb
 - lib/foreman_inventory_upload/generators/archived_report.rb
 - lib/foreman_inventory_upload/generators/fact_helpers.rb
 - lib/foreman_inventory_upload/generators/json_stream.rb
@@ -205,6 +201,7 @@ files:
 - lib/foreman_inventory_upload/generators/slice.rb
 - lib/foreman_inventory_upload/generators/tags.rb
 - lib/foreman_inventory_upload/notifications/manifest_import_success_notification_override.rb
+- lib/foreman_inventory_upload/scripts/uploader.sh.erb
 - lib/foreman_rh_cloud.rb
 - lib/foreman_rh_cloud/async/exponential_backoff.rb
 - lib/foreman_rh_cloud/engine.rb
@@ -261,11 +258,7 @@ files:
 - test/factories/inventory_upload_factories.rb
 - test/jobs/cloud_connector_announce_task_test.rb
 - test/jobs/connector_playbook_execution_reporter_task_test.rb
-- test/jobs/create_missing_insights_facets_test.rb
 - test/jobs/exponential_backoff_test.rb
-- test/jobs/generate_host_report_test.rb
-- test/jobs/generate_report_job_test.rb
-- 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_resolutions_sync_test.rb
@@ -274,21 +267,16 @@ files:
 - test/jobs/inventory_hosts_sync_test.rb
 - test/jobs/inventory_scheduled_sync_test.rb
 - test/jobs/inventory_self_host_sync_test.rb
-- test/jobs/queue_for_upload_job_test.rb
 - test/jobs/remove_insights_hosts_job_test.rb
-- test/jobs/single_host_report_job_test.rb
-- test/jobs/upload_report_direct_job_test.rb
+- test/jobs/upload_report_job_test.rb
 - test/models/insights_client_report_status_test.rb
 - test/test_plugin_helper.rb
 - test/unit/archived_report_generator_test.rb
 - test/unit/fact_helpers_test.rb
-- test/unit/foreman_rh_cloud_iop_metadata_test.rb
 - test/unit/foreman_rh_cloud_self_host_test.rb
 - test/unit/insights_facet_test.rb
-- test/unit/lib/foreman_rh_cloud/registration_manager_extensions_test.rb
 - test/unit/metadata_generator_test.rb
 - test/unit/playbook_progress_generator_test.rb
-- test/unit/rh_cloud_host_test.rb
 - test/unit/rh_cloud_http_proxy_test.rb
 - test/unit/rh_cloud_permissions_test.rb
 - test/unit/services/foreman_rh_cloud/branch_info_test.rb
@@ -442,6 +430,7 @@ files:
 - webpack/ForemanInventoryUpload/Components/PageHeader/PageTitle.js
 - webpack/ForemanInventoryUpload/Components/PageHeader/__tests__/PageHeader.test.js
 - webpack/ForemanInventoryUpload/Components/PageHeader/__tests__/PageTitle.test.js
+- webpack/ForemanInventoryUpload/Components/PageHeader/__tests__/__snapshots__/PageHeader.test.js.snap
 - webpack/ForemanInventoryUpload/Components/PageHeader/__tests__/__snapshots__/PageTitle.test.js.snap
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/CloudConnectorButton/CloudConnectorActions.js
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/CloudConnectorButton/CloudConnectorButton.js
@@ -586,6 +575,7 @@ files:
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/InsightsTable.test.js
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/InsightsTableActions.test.js
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/InsightsTableSelectors.test.js
+- webpack/InsightsCloudSync/Components/InsightsTable/__tests__/__snapshots__/InsightsTable.test.js.snap
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/__snapshots__/InsightsTableActions.test.js.snap
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/__snapshots__/InsightsTableSelectors.test.js.snap
 - webpack/InsightsCloudSync/Components/InsightsTable/__tests__/fixtures.js
@@ -628,8 +618,6 @@ files:
 - webpack/InsightsHostDetailsTab/__tests__/InsightsTabIntegration.test.js
 - 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
@@ -646,6 +634,7 @@ files:
 - webpack/__mocks__/foremanReact/common/I18n.js
 - webpack/__mocks__/foremanReact/common/MountingService.js
 - webpack/__mocks__/foremanReact/common/helpers.js
+- webpack/__mocks__/foremanReact/common/hooks/API/APIHooks.js
 - webpack/__mocks__/foremanReact/components/ConfirmModal/index.js
 - webpack/__mocks__/foremanReact/components/Head.js
 - webpack/__mocks__/foremanReact/components/Layout/LayoutConstants.js
@@ -702,7 +691,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.6
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Summary of ForemanRhCloud.
 test_files:
@@ -721,11 +710,7 @@ test_files:
 - test/factories/inventory_upload_factories.rb
 - test/jobs/cloud_connector_announce_task_test.rb
 - test/jobs/connector_playbook_execution_reporter_task_test.rb
-- test/jobs/create_missing_insights_facets_test.rb
 - test/jobs/exponential_backoff_test.rb
-- test/jobs/generate_host_report_test.rb
-- test/jobs/generate_report_job_test.rb
-- 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_resolutions_sync_test.rb
@@ -734,21 +719,16 @@ test_files:
 - test/jobs/inventory_hosts_sync_test.rb
 - test/jobs/inventory_scheduled_sync_test.rb
 - test/jobs/inventory_self_host_sync_test.rb
-- test/jobs/queue_for_upload_job_test.rb
 - test/jobs/remove_insights_hosts_job_test.rb
-- test/jobs/single_host_report_job_test.rb
-- test/jobs/upload_report_direct_job_test.rb
+- test/jobs/upload_report_job_test.rb
 - test/models/insights_client_report_status_test.rb
 - test/test_plugin_helper.rb
 - test/unit/archived_report_generator_test.rb
 - test/unit/fact_helpers_test.rb
-- test/unit/foreman_rh_cloud_iop_metadata_test.rb
 - test/unit/foreman_rh_cloud_self_host_test.rb
 - test/unit/insights_facet_test.rb
-- test/unit/lib/foreman_rh_cloud/registration_manager_extensions_test.rb
 - test/unit/metadata_generator_test.rb
 - test/unit/playbook_progress_generator_test.rb
-- test/unit/rh_cloud_host_test.rb
 - test/unit/rh_cloud_http_proxy_test.rb
 - test/unit/rh_cloud_permissions_test.rb
 - test/unit/services/foreman_rh_cloud/branch_info_test.rb

data/app/controllers/concerns/foreman_rh_cloud/registration_manager_extensions.rb

@@ -1,39 +0,0 @@
-module ForemanRhCloud
-  module RegistrationManagerExtensions
-    include ForemanRhCloud::CertAuth
-
-    def logger
-      Rails.logger
-    end
-
-    def unregister_host(host, options = {})
-      organization_destroy = options.fetch(:organization_destroy, false)
-
-      # Reload to ensure we have fresh association data
-      host.reload
-
-      # Only delete from HBI in IoP mode (hosted mode uses async job for cleanup)
-      hbi_host_destroy(host) if ForemanRhCloud.with_iop_smart_proxy? && !organization_destroy && host.insights_uuid.presence
-      host.insights&.destroy!
-      super(host, options)
-    end
-
-    def hbi_host_destroy(host)
-      uuid = host.insights_uuid
-      logger.debug "Unregistering host #{uuid} from HBI"
-      execute_cloud_request(
-        organization: host.organization,
-        method: :delete,
-        url: ForemanInventoryUpload.host_by_id_url(uuid),
-        headers: {
-          content_type: :json,
-        }
-      )
-    rescue RestClient::NotFound
-      Rails.logger.warn(_("Attempted to destroy HBI host %s, but host does not exist in HBI") % uuid)
-    rescue StandardError => e
-      # TODO: Improve error handling - don't break registration if HBI delete fails
-      Rails.logger.error(format(_("Failed to destroy HBI host %s: %s"), uuid, e.message))
-    end
-  end
-end

data/lib/foreman_inventory_upload/async/create_missing_insights_facets.rb

@@ -1,36 +0,0 @@
-module ForemanInventoryUpload
-  module Async
-    class CreateMissingInsightsFacets < ::Actions::EntryAction
-      def plan(organization_id)
-        plan_self(organization_id: organization_id)
-      end
-
-      def run
-        organization = ::Organization.find(input[:organization_id])
-        # Find hosts with subscription facets but without insights facets
-        # Note: We can't use scoped_search 'null? insights_uuid' because the null? operator
-        # doesn't work with ext_methods - it would check hosts.id IS NULL instead of the facet
-        hosts_without_facets = ::ForemanInventoryUpload::Generators::Queries.for_org(organization, use_batches: false)
-                                                                            .left_outer_joins(:insights)
-                                                                            .where(insights_facets: { id: nil })
-
-        facet_count = 0
-        hosts_without_facets.in_batches(of: ForemanInventoryUpload.slice_size) do |batch|
-          facets = batch.pluck(:id, 'katello_subscription_facets.uuid').map do |host_id, uuid|
-            {
-              host_id: host_id,
-              uuid: uuid,
-            }
-          end
-          # We don't need to validate the facets here as we create the necessary fields.
-          # rubocop:disable Rails/SkipsModelValidations
-          InsightsFacet.upsert_all(facets, unique_by: :host_id) unless facets.empty?
-          # rubocop:enable Rails/SkipsModelValidations
-          facet_count += facets.size
-        end
-        output[:result] = facet_count.zero? ? _("There were no missing Insights facets") : format(_("Missing Insights facets created: %s"), facet_count)
-        Rails.logger.debug output[:result]
-      end
-    end
-  end
-end

data/lib/foreman_inventory_upload/async/generate_host_report.rb

@@ -1,20 +0,0 @@
-module ForemanInventoryUpload
-  module Async
-    class GenerateHostReport < ::Actions::EntryAction
-      def plan(base_folder, organization_id, filter)
-        plan_self(
-          base_folder: base_folder,
-          organization_id: organization_id,
-          filter: filter
-        )
-        input[:target] = File.join(base_folder, ForemanInventoryUpload.facts_archive_name(input[:organization_id], input[:filter]))
-      end
-
-      def run
-        archived_report_generator = ForemanInventoryUpload::Generators::ArchivedReport.new(input[:target])
-        archived_report_generator.render(organization: input[:organization_id], filter: input[:filter])
-        output[:result] = "Generated #{input[:target]} for organization id #{input[:organization_id]}"
-      end
-    end
-  end
-end

data/lib/foreman_inventory_upload/async/host_inventory_report_job.rb

@@ -1,39 +0,0 @@
-module ForemanInventoryUpload
-  module Async
-    class HostInventoryReportJob < ::Actions::EntryAction
-      def plan(base_folder, organization_id, hosts_filter = "", upload = true)
-        sequence do
-          plan_action(
-            GenerateHostReport,
-            base_folder,
-            organization_id,
-            hosts_filter
-          )
-          if upload
-            plan_action(
-              QueueForUploadJob,
-              base_folder,
-              ForemanInventoryUpload.facts_archive_name(organization_id, hosts_filter),
-              organization_id
-            )
-          end
-
-          if ForemanRhCloud.with_iop_smart_proxy?
-            plan_action(
-              CreateMissingInsightsFacets,
-              organization_id
-            )
-          end
-        end
-      end
-
-      def humanized_name
-        _("Host inventory report job")
-      end
-
-      def organization_id
-        input[:organization_id]
-      end
-    end
-  end
-end

data/lib/foreman_inventory_upload/async/single_host_report_job.rb

@@ -1,20 +0,0 @@
-module ForemanInventoryUpload
-  module Async
-    class SingleHostReportJob < HostInventoryReportJob
-      def plan(base_folder, organization_id, host_id)
-        input[:host_id] = host_id
-        super(base_folder, organization_id, "id=#{input[:host_id]}")
-      end
-
-      def hostname(host_id)
-        host = ::Host.find_by(id: host_id)
-        host&.name
-      end
-
-      def humanized_name
-        hostname_result = hostname(input[:host_id])
-        hostname_result.present? ? format(_("Single-host report job for host %s"), hostname_result) : _("Single-host report job")
-      end
-    end
-  end
-end

data/lib/foreman_inventory_upload/async/upload_report_direct_job.rb

@@ -1,200 +0,0 @@
-require 'tempfile'
-require 'rest-client'
-
-module ForemanInventoryUpload
-  module Async
-    class UploadReportDirectJob < ::Actions::EntryAction
-      include AsyncHelpers
-      include ::ForemanRhCloud::Async::ExponentialBackoff
-      include ::ForemanRhCloud::CloudRequest
-
-      # Wrapper class to avoid monkey-patching File for multipart uploads
-      class FileUpload
-        attr_reader :file, :content_type
-
-        def initialize(file, content_type:)
-          @file = file
-          @content_type = content_type
-        end
-
-        def read(*args)
-          @file.read(*args)
-        end
-
-        def path
-          @file.path
-        end
-
-        def respond_to_missing?(method_name, include_private = false)
-          @file.respond_to?(method_name, include_private) || super
-        end
-
-        def method_missing(method_name, *args, &block)
-          if @file.respond_to?(method_name)
-            @file.send(method_name, *args, &block)
-          else
-            super
-          end
-        end
-      end
-
-      def self.output_label(label)
-        "upload_for_#{label}"
-      end
-
-      def plan(filename, organization_id)
-        # NOTE: This implementation assumes a single organization will not trigger multiple
-        # concurrent uploads. The instance_label is derived from organization_id alone, which
-        # means concurrent uploads for the same org would share ProgressOutput storage.
-        # This matches the pattern in GenerateReportJob. A full fix for thread-safety
-        # requires UI changes to display multiple concurrent tasks per org (tracked for PR #2).
-        label = UploadReportDirectJob.output_label(organization_id)
-        clear_task_output(label)
-        plan_self(
-          instance_label: label,
-          filename: filename,
-          organization_id: organization_id
-        )
-      end
-
-      def try_execute
-        if content_disconnected?
-          progress_output do |progress_output|
-            progress_output.write_line("Report was not moved and upload was canceled because connection to Insights is not enabled. Report location: #{filename}.")
-            progress_output.status = "Task aborted, exit 1"
-            done!
-          end
-          return
-        end
-
-        unless organization.owner_details&.dig('upstreamConsumer', 'idCert')
-          logger.info("Skipping organization '#{organization}', no candlepin certificate defined.")
-          progress_output do |progress_output|
-            progress_output.write_line("Skipping organization #{organization}, no candlepin certificate defined.")
-            progress_output.status = "Task aborted, exit 1"
-            done!
-          end
-          return
-        end
-
-        Tempfile.create([organization.name, '.pem']) do |cer_file|
-          cer_file.write(certificate[:cert])
-          cer_file.write(certificate[:key])
-          cer_file.flush
-          upload_report(cer_file.path)
-        end
-
-        done!
-      end
-
-      def upload_report(cer_path)
-        progress_output do |progress_output|
-          progress_output.write_line("Uploading report for organization #{organization.label}...")
-          progress_output.status = "Running upload"
-
-          begin
-            upload_file(cer_path)
-            progress_output.write_line("Upload completed successfully")
-            move_to_done_folder
-            progress_output.write_line("Uploaded file moved to done/ folder")
-            progress_output.status = "pid #{Process.pid} exit 0"
-          rescue StandardError => e
-            progress_output.write_line("Upload failed: #{e.message}")
-            progress_output.status = "pid #{Process.pid} exit 1"
-            raise
-          end
-        end
-      end
-
-      def upload_file(cer_path)
-        cert_content = File.read(cer_path)
-
-        File.open(filename, 'rb') do |file|
-          # Wrap file with FileUpload class for RestClient multipart handling
-          # RestClient requires objects with :read, :path, and :content_type methods
-          wrapped_file = FileUpload.new(file, content_type: 'application/vnd.redhat.qpc.tar+tgz')
-
-          response = execute_cloud_request(
-            method: :post,
-            url: ForemanInventoryUpload.upload_url,
-            payload: {
-              multipart: true,
-              file: wrapped_file,
-            },
-            headers: {
-              'X-Org-Id' => organization.label,
-            },
-            ssl_client_cert: OpenSSL::X509::Certificate.new(cert_content),
-            ssl_client_key: OpenSSL::PKey::RSA.new(cert_content),
-            timeout: 600,
-            open_timeout: 60
-          )
-
-          logger.debug("Upload response code: #{response.code}")
-        end
-      end
-
-      def move_to_done_folder
-        FileUtils.mkdir_p(ForemanInventoryUpload.done_folder)
-        done_file = ForemanInventoryUpload.done_file_path(File.basename(filename))
-        FileUtils.mv(filename, done_file)
-        logger.debug("Moved #{filename} to #{done_file}")
-      end
-
-      def certificate
-        ForemanRhCloud.with_iop_smart_proxy? ? foreman_certificate : manifest_certificate
-      end
-
-      def manifest_certificate
-        candlepin_id_certificate = organization.owner_details['upstreamConsumer']['idCert']
-        {
-          cert: candlepin_id_certificate['cert'],
-          key: candlepin_id_certificate['key'],
-        }
-      end
-
-      def foreman_certificate
-        {
-          cert: File.read(Setting[:ssl_certificate]),
-          key: File.read(Setting[:ssl_priv_key]),
-        }
-      end
-
-      def filename
-        input[:filename]
-      end
-
-      def organization
-        Organization.find(input[:organization_id])
-      end
-
-      def content_disconnected?
-        !Setting[:subscription_connection_enabled]
-      end
-
-      def progress_output
-        progress_output = ProgressOutput.register(instance_label)
-        yield(progress_output)
-      ensure
-        progress_output.close
-      end
-
-      def instance_label
-        input[:instance_label]
-      end
-
-      def logger
-        Foreman::Logging.logger('background')
-      end
-
-      def rescue_strategy_for_self
-        Dynflow::Action::Rescue::Fail
-      end
-
-      def clear_task_output(label)
-        TaskOutputLine.where(label: label).delete_all
-        TaskOutputStatus.where(label: label).delete_all
-      end
-    end
-  end
-end

data/test/jobs/create_missing_insights_facets_test.rb

@@ -1,151 +0,0 @@
-require 'test_plugin_helper'
-require 'foreman_tasks/test_helpers'
-
-class CreateMissingInsightsFacetsTest < ActiveSupport::TestCase
-  include Dynflow::Testing::Factories
-  include KatelloCVEHelper
-
-  setup do
-    User.current = User.find_by(login: 'secret_admin')
-    @cve = make_cve
-    @env = @cve.lifecycle_environment
-    @organization = @env.organization
-
-    # Create a host with subscription facet but no insights facet
-    @host_without_facet = FactoryBot.create(
-      :host,
-      :with_subscription,
-      :with_content,
-      content_view: @cve.content_view,
-      lifecycle_environment: @env,
-      organization: @organization
-    )
-
-    # Create a host with both subscription and insights facets
-    @host_with_facet = FactoryBot.create(
-      :host,
-      :with_subscription,
-      :with_content,
-      content_view: @cve.content_view,
-      lifecycle_environment: @env,
-      organization: @organization
-    )
-    @host_with_facet.build_insights(uuid: @host_with_facet.subscription_facet.uuid)
-    @host_with_facet.insights.save!
-  end
-
-  test 'creates insights facets for hosts without them' do
-    assert_nil @host_without_facet.insights
-
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::CreateMissingInsightsFacets,
-      @organization.id
-    )
-    action = run_action(action)
-
-    @host_without_facet.reload
-    assert_not_nil @host_without_facet.insights
-    assert_equal @host_without_facet.subscription_facet.uuid, @host_without_facet.insights.uuid
-    assert_match(/Missing Insights facets created: 1/, action.output[:result])
-  end
-
-  test 'does not create duplicate facets for hosts that already have them' do
-    original_uuid = @host_with_facet.insights.uuid
-    original_id = @host_with_facet.insights.id
-
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::CreateMissingInsightsFacets,
-      @organization.id
-    )
-    run_action(action)
-
-    @host_with_facet.reload
-    assert_equal original_id, @host_with_facet.insights.id
-    assert_equal original_uuid, @host_with_facet.insights.uuid
-  end
-
-  test 'handles organization with no missing facets' do
-    # Create insights facet for the host that was missing one
-    @host_without_facet.build_insights(uuid: @host_without_facet.subscription_facet.uuid)
-    @host_without_facet.insights.save!
-
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::CreateMissingInsightsFacets,
-      @organization.id
-    )
-    action = run_action(action)
-
-    assert_match(/There were no missing Insights facets/, action.output[:result])
-  end
-
-  test 'creates multiple facets when multiple hosts are missing them' do
-    # Remove the insights facet from the host that has one
-    @host_with_facet.insights.destroy
-    @host_with_facet.reload
-
-    assert_nil @host_without_facet.insights
-    assert_nil @host_with_facet.insights
-
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::CreateMissingInsightsFacets,
-      @organization.id
-    )
-    action = run_action(action)
-
-    @host_without_facet.reload
-    @host_with_facet.reload
-    assert_not_nil @host_without_facet.insights
-    assert_not_nil @host_with_facet.insights
-    # After the bug fix, the count should correctly show 2 hosts
-    assert_match(/Missing Insights facets created: 2/, action.output[:result])
-  end
-
-  test 'logs result message' do
-    Rails.logger.expects(:debug).with(regexp_matches(/Missing Insights facets created/))
-
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::CreateMissingInsightsFacets,
-      @organization.id
-    )
-    run_action(action)
-  end
-
-  test 'correctly counts facets across multiple batches' do
-    # Remove existing insights facet
-    @host_with_facet.insights.destroy
-    @host_with_facet.reload
-
-    # Stub the batch size to force multiple batches with just 2 hosts
-    ForemanInventoryUpload.stubs(:slice_size).returns(1)
-
-    assert_nil @host_without_facet.insights
-    assert_nil @host_with_facet.insights
-
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::CreateMissingInsightsFacets,
-      @organization.id
-    )
-    action = run_action(action)
-
-    @host_without_facet.reload
-    @host_with_facet.reload
-    assert_not_nil @host_without_facet.insights
-    assert_not_nil @host_with_facet.insights
-    # Count should be 2 even though processed in 2 separate batches
-    assert_match(/Missing Insights facets created: 2/, action.output[:result])
-  end
-
-  test 'handles error when InsightsFacet.upsert_all fails' do
-    # Stub upsert_all to raise an exception
-    InsightsFacet.stubs(:upsert_all).raises(StandardError.new('upsert failed'))
-
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::CreateMissingInsightsFacets,
-      @organization.id
-    )
-
-    assert_raises(StandardError) do
-      run_action(action)
-    end
-  end
-end