foreman_rh_cloud 13.0.7 → 13.0.9

data/test/jobs/insights_full_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InsightsFullSyncTest < ActiveSupport::TestCase
-  include ForemanTasks::TestHelpers::WithInThreadExecutor
+  include Dynflow::Testing::Factories
   include MockCerts
 
   setup do
@@ -72,7 +72,8 @@ class InsightsFullSyncTest < ActiveSupport::TestCase
     InsightsCloud::Async::InsightsFullSync.any_instance.expects(:plan_hosts_sync)
     InsightsCloud::Async::InsightsFullSync.any_instance.expects(:plan_rules_sync)
     InsightsCloud::Async::InsightsFullSync.any_instance.expects(:plan_notifications)
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsFullSync, [@host1.organization, @host2.organization])
+    action = create_and_plan_action(InsightsCloud::Async::InsightsFullSync, [@host1.organization, @host2.organization])
+    run_action(action)
 
     @host1.reload
     @host2.reload
@@ -89,7 +90,8 @@ class InsightsFullSyncTest < ActiveSupport::TestCase
     InsightsCloud::Async::InsightsFullSync.any_instance.expects(:plan_hosts_sync).never
     InsightsCloud::Async::InsightsFullSync.any_instance.expects(:plan_self).never
 
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsFullSync, [@host1.organization])
+    action = create_and_plan_action(InsightsCloud::Async::InsightsFullSync, [@host1.organization])
+    run_action(action)
   end
 
   test 'Manifest is deleted do not run task steps' do
@@ -100,7 +102,8 @@ class InsightsFullSyncTest < ActiveSupport::TestCase
     InsightsCloud::Async::InsightsFullSync.any_instance.expects(:plan_hosts_sync).never
     InsightsCloud::Async::InsightsFullSync.any_instance.expects(:plan_self).never
 
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsFullSync, [@host1.organization])
+    action = create_and_plan_action(InsightsCloud::Async::InsightsFullSync, [@host1.organization])
+    run_action(action)
   end
 
   test 'Hits counters are reset correctly' do
@@ -110,9 +113,11 @@ class InsightsFullSyncTest < ActiveSupport::TestCase
 
     InsightsCloud::Async::InsightsFullSync.any_instance.stubs(:plan_hosts_sync)
 
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsFullSync, [@host1.organization, @host2.organization])
+    action = create_and_plan_action(InsightsCloud::Async::InsightsFullSync, [@host1.organization, @host2.organization])
+    run_action(action)
     # Invoke again
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsFullSync, [@host1.organization, @host2.organization])
+    action = create_and_plan_action(InsightsCloud::Async::InsightsFullSync, [@host1.organization, @host2.organization])
+    run_action(action)
 
     @host1.reload
     @host2.reload
@@ -146,7 +151,8 @@ class InsightsFullSyncTest < ActiveSupport::TestCase
     InsightsCloud::Async::InsightsFullSync.any_instance.stubs(:plan_hosts_sync)
     InsightsCloud::Async::InsightsFullSync.any_instance.expects(:query_insights_hits).returns(hits)
 
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsFullSync, [@host1.organization, @host2.organization])
+    action = create_and_plan_action(InsightsCloud::Async::InsightsFullSync, [@host1.organization, @host2.organization])
+    run_action(action)
 
     @host1.reload
     @host2.reload

data/test/jobs/insights_resolutions_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InsightsResolutionsSyncTest < ActiveSupport::TestCase
-  include ForemanTasks::TestHelpers::WithInThreadExecutor
+  include Dynflow::Testing::Factories
   include MockCerts
 
   setup do
@@ -76,7 +76,8 @@ class InsightsResolutionsSyncTest < ActiveSupport::TestCase
     Katello::UpstreamConnectionChecker.any_instance.stubs(:can_connect?).returns(true)
     InsightsCloud::Async::InsightsResolutionsSync.any_instance.stubs(:query_insights_resolutions).returns(@resolutions)
 
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsResolutionsSync)
+    action = create_and_plan_action(InsightsCloud::Async::InsightsResolutionsSync)
+    run_action(action)
     @rule.reload
 
     assert_equal 5, InsightsResolution.all.count
@@ -88,7 +89,8 @@ class InsightsResolutionsSyncTest < ActiveSupport::TestCase
     Katello::UpstreamConnectionChecker.any_instance.stubs(:can_connect?).returns(false)
     InsightsCloud::Async::InsightsResolutionsSync.any_instance.expects(:query_insights_resolutions).never
 
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsResolutionsSync)
+    action = create_and_plan_action(InsightsCloud::Async::InsightsResolutionsSync)
+    run_action(action)
 
     assert_equal 0, InsightsResolution.all.count
   end
@@ -98,7 +100,8 @@ class InsightsResolutionsSyncTest < ActiveSupport::TestCase
     Katello::UpstreamConnectionChecker.any_instance.stubs(:can_connect?).returns(true)
     InsightsCloud::Async::InsightsResolutionsSync.any_instance.expects(:query_insights_resolutions).never
 
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsResolutionsSync)
+    action = create_and_plan_action(InsightsCloud::Async::InsightsResolutionsSync)
+    run_action(action)
 
     assert_equal 0, InsightsResolution.all.count
   end
@@ -108,7 +111,8 @@ class InsightsResolutionsSyncTest < ActiveSupport::TestCase
     InsightsCloud::Async::InsightsResolutionsSync.any_instance.expects(:query_insights_resolutions).never
     InsightsRule.all.delete_all
 
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsResolutionsSync)
+    action = create_and_plan_action(InsightsCloud::Async::InsightsResolutionsSync)
+    run_action(action)
 
     assert_equal 0, InsightsResolution.all.count
   end

data/test/jobs/insights_rules_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InsightsRulesSyncTest < ActiveSupport::TestCase
-  include ForemanTasks::TestHelpers::WithInThreadExecutor
+  include Dynflow::Testing::Factories
 
   setup do
     rules_json = <<-'RULES_JSON'
@@ -119,7 +119,8 @@ class InsightsRulesSyncTest < ActiveSupport::TestCase
     # do not cleanup unused rules for tests
     InsightsCloud::Async::InsightsRulesSync.any_instance.stubs(:cleanup_rules)
 
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsRulesSync, Organization.all)
+    action = create_and_plan_action(InsightsCloud::Async::InsightsRulesSync, Organization.all)
+    run_action(action)
     @hit.reload
 
     assert_equal 2, InsightsRule.all.count
@@ -201,7 +202,8 @@ class InsightsRulesSyncTest < ActiveSupport::TestCase
     # do not cleanup unused rules for tests
     InsightsCloud::Async::InsightsRulesSync.any_instance.stubs(:cleanup_rules)
 
-    ForemanTasks.sync_task(InsightsCloud::Async::InsightsRulesSync, Organization.all)
+    action = create_and_plan_action(InsightsCloud::Async::InsightsRulesSync, Organization.all)
+    run_action(action)
 
     assert_equal 3, InsightsRule.all.count
   end

data/test/jobs/inventory_full_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InventoryFullSyncTest < ActiveSupport::TestCase
-  include ForemanTasks::TestHelpers::WithInThreadExecutor
+  include Dynflow::Testing::Factories
   include MockCerts
   include KatelloCVEHelper
 
@@ -265,7 +265,8 @@ class InventoryFullSyncTest < ActiveSupport::TestCase
 
     InventorySync::Async::InventoryFullSync.any_instance.expects(:query_inventory).returns(@inventory)
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryFullSync, @host2.organization)
+    action = create_and_plan_action(InventorySync::Async::InventoryFullSync, @host2.organization)
+    run_action(action)
 
     @host2.reload
 
@@ -280,7 +281,8 @@ class InventoryFullSyncTest < ActiveSupport::TestCase
     InventorySync::Async::InventoryFullSync.any_instance.expects(:query_inventory).returns(@inventory)
     FactoryBot.create(:fact_value, fact_name: fact_names['virt::uuid'], value: '1234', host: @host2)
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryFullSync, @host1.organization)
+    action = create_and_plan_action(InventorySync::Async::InventoryFullSync, @host1.organization)
+    run_action(action)
     @host2.reload
 
     assert_equal InventorySync::InventoryStatus::DISCONNECT, InventorySync::InventoryStatus.where(host_id: @host1.id).first.status
@@ -291,7 +293,8 @@ class InventoryFullSyncTest < ActiveSupport::TestCase
 
     InventorySync::Async::InventoryFullSync.any_instance.expects(:plan_self).never
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryFullSync, @host1.organization)
+    action = create_and_plan_action(InventorySync::Async::InventoryFullSync, @host1.organization)
+    run_action(action)
   end
 
   test 'Should skip hosts that are not returned in query' do
@@ -304,7 +307,8 @@ class InventoryFullSyncTest < ActiveSupport::TestCase
     InventorySync::Async::InventoryFullSync.any_instance.expects(:affected_host_ids).returns([@host1.id, @host2.id])
     FactoryBot.create(:fact_value, fact_name: fact_names['virt::uuid'], value: '1234', host: @host2)
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryFullSync, @host1.organization)
+    action = create_and_plan_action(InventorySync::Async::InventoryFullSync, @host1.organization)
+    run_action(action)
     @host2.reload
 
     assert_nil InventorySync::InventoryStatus.where(host_id: @host3.id).first

data/test/jobs/inventory_hosts_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InventoryHostsSyncTest < ActiveSupport::TestCase
-  include ForemanTasks::TestHelpers::WithInThreadExecutor
+  include Dynflow::Testing::Factories
   include MockCerts
   include KatelloCVEHelper
 
@@ -302,7 +302,8 @@ class InventoryHostsSyncTest < ActiveSupport::TestCase
 
     @host2.build_insights.save
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
+    action = create_and_plan_action(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
+    run_action(action)
 
     @host2.reload
 
@@ -317,7 +318,8 @@ class InventoryHostsSyncTest < ActiveSupport::TestCase
       InventorySync::Async::InventoryHostsSync.any_instance.stubs(:candlepin_id_cert)
     end
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
+    action = create_and_plan_action(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
+    run_action(action)
 
     @host2.reload
 
@@ -336,7 +338,8 @@ class InventoryHostsSyncTest < ActiveSupport::TestCase
 
     assert_nil @host2.insights
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
+    action = create_and_plan_action(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
+    run_action(action)
 
     @host2.reload
 
@@ -353,7 +356,8 @@ class InventoryHostsSyncTest < ActiveSupport::TestCase
       InventorySync::Async::InventoryHostsSync.any_instance.stubs(:candlepin_id_cert)
     end
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryHostsSync, [org])
+    action = create_and_plan_action(InventorySync::Async::InventoryHostsSync, [org])
+    run_action(action)
 
     assert_equal 3, InsightsMissingHost.count
   end
@@ -370,7 +374,8 @@ class InventoryHostsSyncTest < ActiveSupport::TestCase
       InventorySync::Async::InventoryHostsSync.any_instance.stubs(:candlepin_id_cert)
     end
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryHostsSync, [org])
+    action = create_and_plan_action(InventorySync::Async::InventoryHostsSync, [org])
+    run_action(action)
 
     assert_equal 3, InsightsMissingHost.count
   end

data/test/jobs/inventory_scheduled_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InventoryScheduledSyncTest < ActiveSupport::TestCase
-  include ForemanTasks::TestHelpers::WithInThreadExecutor
+  include Dynflow::Testing::Factories
 
   test 'Schedules an execution if auto upload is enabled' do
     Setting[:allow_auto_inventory_upload] = true
@@ -11,7 +11,8 @@ class InventoryScheduledSyncTest < ActiveSupport::TestCase
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_org_sync).times(Organization.unscoped.count)
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_remove_insights_hosts).times(Organization.unscoped.count)
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryScheduledSync)
+    action = create_and_plan_action(InventorySync::Async::InventoryScheduledSync)
+    run_action(action)
   end
 
   test 'Skips execution if with_iop_smart_proxy? is true' do
@@ -19,8 +20,9 @@ class InventoryScheduledSyncTest < ActiveSupport::TestCase
 
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_org_sync).never
 
-    task = ForemanTasks.sync_task(InventorySync::Async::InventoryScheduledSync)
-    status = task.output[:status].to_s
+    action = create_and_plan_action(InventorySync::Async::InventoryScheduledSync)
+    action = run_action(action)
+    status = action.output[:status].to_s
     assert_match(/Foreman is configured with a local IoP Smart Proxy/, status)
   end
 
@@ -29,7 +31,8 @@ class InventoryScheduledSyncTest < ActiveSupport::TestCase
 
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_org_sync).never
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryScheduledSync)
+    action = create_and_plan_action(InventorySync::Async::InventoryScheduledSync)
+    run_action(action)
   end
 
   test 'Skips mismatch deletion if the setting is disabled' do
@@ -39,6 +42,7 @@ class InventoryScheduledSyncTest < ActiveSupport::TestCase
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_org_sync).times(Organization.unscoped.count)
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_remove_insights_hosts).never
 
-    ForemanTasks.sync_task(InventorySync::Async::InventoryScheduledSync)
+    action = create_and_plan_action(InventorySync::Async::InventoryScheduledSync)
+    run_action(action)
   end
 end

data/test/jobs/inventory_self_host_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InventorySelfHostSyncTest < ActiveSupport::TestCase
-  include ForemanTasks::TestHelpers::WithInThreadExecutor
+  include Dynflow::Testing::Factories
   include MockCerts
 
   setup do

data/test/jobs/queue_for_upload_job_test.rb

@@ -2,24 +2,15 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class QueueForUploadJobTest < ActiveSupport::TestCase
-  include ForemanTasks::TestHelpers::WithInThreadExecutor
-  include FolderIsolation
+  include Dynflow::Testing::Factories
 
   let(:organization) { FactoryBot.create(:organization) }
-  let(:base_folder) { @tmpdir }
+  let(:base_folder) { Dir.mktmpdir }
   let(:report_file) { 'test_report.tar.xz' }
   let(:report_path) { File.join(base_folder, report_file) }
   let(:uploads_folder) { ForemanInventoryUpload.uploads_folder }
-  subject { ForemanTasks.sync_task(ForemanInventoryUpload::Async::QueueForUploadJob, base_folder, report_file, organization.id) }
 
   setup do
-    # Stub the script template source
-    script_source = File.join(ForemanRhCloud::Engine.root, 'lib/foreman_inventory_upload/scripts/uploader.sh.erb')
-    File.stubs(:read).with(script_source).returns('#!/bin/bash\necho "Test script"')
-
-    # Stub template rendering
-    Foreman::Renderer.stubs(:render).returns('#!/bin/bash\necho "Rendered script"')
-
     # Stub additional settings that are accessed
     Setting.stubs(:[]).with(:content_default_http_proxy).returns(nil)
     Setting.stubs(:[]).with(:http_proxy).returns(nil)
@@ -29,35 +20,35 @@ class QueueForUploadJobTest < ActiveSupport::TestCase
 
   teardown do
     FileUtils.rm_rf(uploads_folder) if Dir.exist?(uploads_folder)
+    FileUtils.remove_entry base_folder if Dir.exist?(base_folder)
   end
 
   test 'plan method sets up the job correctly and calls plan_upload_report' do
     # Mock plan_upload_report to verify it's called
     ForemanInventoryUpload::Async::QueueForUploadJob.any_instance.expects(:plan_upload_report).once
 
-    assert_equal 'success', subject.result
+    action = create_and_plan_action(ForemanInventoryUpload::Async::QueueForUploadJob, base_folder, report_file, organization.id)
+    run_action(action)
   end
 
   test 'run method processes file and moves it to uploads folder' do
     ForemanInventoryUpload::Async::QueueForUploadJob.any_instance.stubs(:plan_upload_report)
 
-    assert_equal 'success', subject.result
+    action = create_and_plan_action(ForemanInventoryUpload::Async::QueueForUploadJob, base_folder, report_file, organization.id)
+    run_action(action)
 
     # Verify the file was moved
     refute File.exist?(report_path), "Original file should be moved"
     assert File.exist?(File.join(uploads_folder, report_file)), "File should exist in uploads folder"
   end
 
-  test 'creates necessary folders and scripts' do
+  test 'creates necessary folders' do
     ForemanInventoryUpload::Async::QueueForUploadJob.any_instance.stubs(:plan_upload_report)
 
-    assert_equal 'success', subject.result
+    action = create_and_plan_action(ForemanInventoryUpload::Async::QueueForUploadJob, base_folder, report_file, organization.id)
+    run_action(action)
 
     # Verify the uploads folder was created
     assert Dir.exist?(uploads_folder), "Uploads folder should be created"
-
-    # Verify the script file was created
-    script_path = File.join(uploads_folder, ForemanInventoryUpload.upload_script_file)
-    assert File.exist?(script_path), "Upload script should be created"
   end
 end

data/test/jobs/remove_insights_hosts_job_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class RemoveInsightsHostJobTest < ActiveSupport::TestCase
-  include ForemanTasks::TestHelpers::WithInThreadExecutor
+  include Dynflow::Testing::Factories
 
   setup do
     User.current = User.find_by(login: 'secret_admin')
@@ -18,10 +18,10 @@ class RemoveInsightsHostJobTest < ActiveSupport::TestCase
 
     FactoryBot.create(:insights_missing_host, organization: @org)
 
-    task = ForemanTasks.sync_task(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
+    action = create_and_plan_action(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
+    run_action(action)
 
     assert_equal 0, InsightsMissingHost.count
-    assert_equal 'success', task.result
   end
 
   test 'Does not delete hosts on cloud failure' do
@@ -31,14 +31,13 @@ class RemoveInsightsHostJobTest < ActiveSupport::TestCase
 
     FactoryBot.create(:insights_missing_host, organization: @org)
 
-    begin
-      ForemanTasks.sync_task(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
-    rescue ForemanTasks::TaskError => ex
-      task = ex.task
+    action = create_and_plan_action(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
+
+    assert_raises(StandardError) do
+      run_action(action)
     end
 
     assert_equal 1, InsightsMissingHost.count
-    assert_equal 'error', task.result
   end
 
   test 'Paginates the hosts list' do
@@ -54,12 +53,12 @@ class RemoveInsightsHostJobTest < ActiveSupport::TestCase
     FactoryBot.create(:insights_missing_host, organization: @org)
     FactoryBot.create(:insights_missing_host, organization: @org)
 
-    task = ForemanTasks.sync_task(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
+    action = create_and_plan_action(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
+    action = run_action(action)
 
     assert_equal 0, InsightsMissingHost.count
-    assert_equal 'success', task.result
-    assert_equal 'response1', task.output[:response_page1]
-    assert_equal 'response2', task.output[:response_page2]
+    assert_equal 'response1', action.output[:response_page1]
+    assert_equal 'response2', action.output[:response_page2]
   end
 
   test 'Uses scoped_search to select hosts' do
@@ -73,12 +72,12 @@ class RemoveInsightsHostJobTest < ActiveSupport::TestCase
     FactoryBot.create(:insights_missing_host, name: 'test a', organization: @org)
     FactoryBot.create(:insights_missing_host, name: 'test b', organization: @org)
 
-    task = ForemanTasks.sync_task(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, 'name ~ b', @org.id)
+    action = create_and_plan_action(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, 'name ~ b', @org.id)
+    action = run_action(action)
 
     assert_equal 1, InsightsMissingHost.count
     assert_equal 'test a', InsightsMissingHost.first.name
-    assert_equal 'success', task.result
-    assert_equal 'response1', task.output[:response_page1]
+    assert_equal 'response1', action.output[:response_page1]
   end
 
   def mock_response(code: 200, body: '')

data/test/jobs/single_host_report_job_test.rb

@@ -0,0 +1,155 @@
+require 'test_plugin_helper'
+require 'foreman_tasks/test_helpers'
+
+class SingleHostReportJobTest < ActiveSupport::TestCase
+  include Dynflow::Testing::Factories
+  include Dynflow::Testing::Assertions
+  include KatelloCVEHelper
+
+  let(:base_folder) { Dir.mktmpdir }
+
+  setup do
+    User.current = User.find_by(login: 'secret_admin')
+    cve = make_cve
+    env = cve.lifecycle_environment
+
+    @host = FactoryBot.create(
+      :host,
+      :with_subscription,
+      :with_content,
+      content_view: cve.content_view,
+      lifecycle_environment: env,
+      organization: env.organization
+    )
+  end
+
+  teardown do
+    FileUtils.remove_entry base_folder if Dir.exist?(base_folder)
+  end
+
+  test 'plan sets host_id in input' do
+    action = create_and_plan_action(
+      ForemanInventoryUpload::Async::SingleHostReportJob,
+      base_folder,
+      @host.organization_id,
+      @host.id
+    )
+
+    assert_equal @host.id, action.input[:host_id]
+  end
+
+  test 'plan calls super with id filter' do
+    expected_filter = "id=#{@host.id}"
+
+    action = create_and_plan_action(
+      ForemanInventoryUpload::Async::SingleHostReportJob,
+      base_folder,
+      @host.organization_id,
+      @host.id
+    )
+
+    assert_action_planned_with(
+      action,
+      ForemanInventoryUpload::Async::GenerateHostReport,
+      base_folder,
+      @host.organization_id,
+      expected_filter
+    )
+  end
+
+  test 'inherits behavior from HostInventoryReportJob' do
+    action = create_and_plan_action(
+      ForemanInventoryUpload::Async::SingleHostReportJob,
+      base_folder,
+      @host.organization_id,
+      @host.id
+    )
+
+    # Should schedule all parent actions
+    assert_action_planned(action, ForemanInventoryUpload::Async::GenerateHostReport)
+    assert_action_planned(action, ForemanInventoryUpload::Async::QueueForUploadJob)
+  end
+
+  test 'humanized_name includes hostname' do
+    action = create_and_plan_action(
+      ForemanInventoryUpload::Async::SingleHostReportJob,
+      base_folder,
+      @host.organization_id,
+      @host.id
+    )
+
+    assert_equal "Single-host report job for host #{@host.name}", action.humanized_name
+  end
+
+  test 'humanized_name handles missing host' do
+    non_existent_host_id = 999_999
+
+    action = create_and_plan_action(
+      ForemanInventoryUpload::Async::SingleHostReportJob,
+      base_folder,
+      @host.organization_id,
+      non_existent_host_id
+    )
+
+    assert_equal 'Single-host report job', action.humanized_name
+  end
+
+  test 'hostname method returns correct name' do
+    action = create_and_plan_action(
+      ForemanInventoryUpload::Async::SingleHostReportJob,
+      base_folder,
+      @host.organization_id,
+      @host.id
+    )
+
+    assert_equal @host.name, action.hostname(@host.id)
+  end
+
+  test 'hostname method handles nil gracefully' do
+    action = create_and_plan_action(
+      ForemanInventoryUpload::Async::SingleHostReportJob,
+      base_folder,
+      @host.organization_id,
+      @host.id
+    )
+
+    assert_nil action.hostname(999_999)
+  end
+
+  test 'generates report with correct archive name for single host' do
+    expected_filter = "id=#{@host.id}"
+    expected_archive_name = ForemanInventoryUpload.facts_archive_name(@host.organization_id, expected_filter)
+
+    action = create_and_plan_action(
+      ForemanInventoryUpload::Async::SingleHostReportJob,
+      base_folder,
+      @host.organization_id,
+      @host.id
+    )
+
+    assert_action_planned_with(
+      action,
+      ForemanInventoryUpload::Async::QueueForUploadJob,
+      base_folder,
+      expected_archive_name,
+      @host.organization_id
+    )
+  end
+
+  test 'respects IoP mode for facet creation' do
+    ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
+
+    action = create_and_plan_action(
+      ForemanInventoryUpload::Async::SingleHostReportJob,
+      base_folder,
+      @host.organization_id,
+      @host.id
+    )
+
+    assert_action_planned_with(
+      action,
+      ForemanInventoryUpload::Async::CreateMissingInsightsFacets,
+      @host.organization_id
+    )
+  end
+end