foreman_rh_cloud 13.0.9 → 13.0.11

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_rh_cloud
 version: !ruby/object:Gem::Version
-  version: 13.0.9
+  version: 13.0.11
 platform: ruby
 authors:
 - Foreman Red Hat Cloud team
@@ -106,6 +106,7 @@ files:
 - app/controllers/concerns/inventory_upload/report_actions.rb
 - app/controllers/concerns/inventory_upload/task_actions.rb
 - app/controllers/foreman_inventory_upload/accounts_controller.rb
+- app/controllers/foreman_inventory_upload/api/tasks_controller.rb
 - app/controllers/foreman_inventory_upload/cloud_status_controller.rb
 - app/controllers/foreman_inventory_upload/missing_hosts_controller.rb
 - app/controllers/foreman_inventory_upload/reports_controller.rb
@@ -130,8 +131,6 @@ files:
 - app/models/insights_resolution.rb
 - app/models/insights_rule.rb
 - app/models/inventory_sync/inventory_status.rb
-- app/models/task_output_line.rb
-- app/models/task_output_status.rb
 - app/overrides/layouts/base/styles.html.erb.deface
 - app/services/foreman_rh_cloud/branch_info.rb
 - app/services/foreman_rh_cloud/cert_auth.rb
@@ -179,6 +178,7 @@ files:
 - db/migrate/20230515140211_add_missing_hosts_table.foreman_rh_cloud.rb
 - db/migrate/20241217190624_add_unique_index_to_rule_id_and_host_id_in_insights_hits.rb
 - db/migrate/20241220184900_change_sync_insights_recommendations_to_true.rb
+- db/migrate/20251209163012_drop_task_output_tables.foreman_rh_cloud.rb
 - db/seeds.d/179_ui_notifications.rb
 - db/seeds.d/189_add_host_inventory_param.rb
 - db/seeds.d/200_features.rb
@@ -189,12 +189,9 @@ files:
 - 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/generators/archived_report.rb
@@ -255,8 +252,6 @@ files:
 - test/controllers/insights_sync/settings_controller_test.rb
 - test/controllers/inventory_upload/api/inventory_controller_test.rb
 - test/controllers/inventory_upload/cloud_status_controller_test.rb
-- test/controllers/reports_controller_test.rb
-- test/controllers/uploads_controller_test.rb
 - test/controllers/uploads_settings_controller_test.rb
 - test/factories/insights_factories.rb
 - test/factories/inventory_upload_factories.rb
@@ -265,7 +260,6 @@ files:
 - 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
@@ -302,7 +296,6 @@ files:
 - test/unit/services/foreman_rh_cloud/tags_auth_test.rb
 - test/unit/services/foreman_rh_cloud/template_renderer_helper_test.rb
 - test/unit/services/foreman_rh_cloud/url_remediations_retriever_test.rb
-- test/unit/shell_process_job_test.rb
 - test/unit/slice_generator_test.rb
 - test/unit/tags_generator_test.rb
 - webpack/CVEsHostDetailsTab/CVEsHostDetailsTab.js
@@ -367,20 +360,8 @@ files:
 - webpack/ForemanInventoryUpload/Components/Dashboard/Dashboard.fixtures.js
 - webpack/ForemanInventoryUpload/Components/Dashboard/Dashboard.js
 - webpack/ForemanInventoryUpload/Components/Dashboard/Dashboard.stories.js
-- webpack/ForemanInventoryUpload/Components/Dashboard/DashboardActions.js
-- webpack/ForemanInventoryUpload/Components/Dashboard/DashboardConstants.js
 - webpack/ForemanInventoryUpload/Components/Dashboard/DashboardHelper.js
-- webpack/ForemanInventoryUpload/Components/Dashboard/DashboardReducer.js
-- webpack/ForemanInventoryUpload/Components/Dashboard/DashboardSelectors.js
 - webpack/ForemanInventoryUpload/Components/Dashboard/__tests__/Dashboard.test.js
-- webpack/ForemanInventoryUpload/Components/Dashboard/__tests__/DashboardActions.test.js
-- webpack/ForemanInventoryUpload/Components/Dashboard/__tests__/DashboardIntegration.test.js
-- webpack/ForemanInventoryUpload/Components/Dashboard/__tests__/DashboardReducer.test.js
-- webpack/ForemanInventoryUpload/Components/Dashboard/__tests__/DashboardSelectors.test.js
-- webpack/ForemanInventoryUpload/Components/Dashboard/__tests__/__snapshots__/Dashboard.test.js.snap
-- webpack/ForemanInventoryUpload/Components/Dashboard/__tests__/__snapshots__/DashboardActions.test.js.snap
-- webpack/ForemanInventoryUpload/Components/Dashboard/__tests__/__snapshots__/DashboardReducer.test.js.snap
-- webpack/ForemanInventoryUpload/Components/Dashboard/__tests__/__snapshots__/DashboardSelectors.test.js.snap
 - webpack/ForemanInventoryUpload/Components/Dashboard/dashboard.scss
 - webpack/ForemanInventoryUpload/Components/Dashboard/index.js
 - webpack/ForemanInventoryUpload/Components/FileDownload/FileDownload.fixtures.js
@@ -390,13 +371,6 @@ files:
 - webpack/ForemanInventoryUpload/Components/FileDownload/__tests__/__snapshots__/FileDownload.test.js.snap
 - webpack/ForemanInventoryUpload/Components/FileDownload/fileDownload.scss
 - webpack/ForemanInventoryUpload/Components/FileDownload/index.js
-- webpack/ForemanInventoryUpload/Components/FullScreenModal/FullScreenModal.fixtures.js
-- webpack/ForemanInventoryUpload/Components/FullScreenModal/FullScreenModal.js
-- webpack/ForemanInventoryUpload/Components/FullScreenModal/FullScreenModalHelper.js
-- webpack/ForemanInventoryUpload/Components/FullScreenModal/__tests__/FullScreenModal.test.js
-- webpack/ForemanInventoryUpload/Components/FullScreenModal/__tests__/__snapshots__/FullScreenModal.test.js.snap
-- webpack/ForemanInventoryUpload/Components/FullScreenModal/fullScreenModal.scss
-- webpack/ForemanInventoryUpload/Components/FullScreenModal/index.js
 - webpack/ForemanInventoryUpload/Components/InventoryFilter/Components/ClearButton/ClearButton.js
 - webpack/ForemanInventoryUpload/Components/InventoryFilter/Components/ClearButton/index.js
 - webpack/ForemanInventoryUpload/Components/InventoryFilter/InventoryFilter.fixtures.js
@@ -478,20 +452,6 @@ files:
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/ToolbarButtons/index.js
 - webpack/ForemanInventoryUpload/Components/PageHeader/components/ToolbarButtons/toolbarButtons.scss
 - webpack/ForemanInventoryUpload/Components/PageHeader/index.js
-- webpack/ForemanInventoryUpload/Components/ReportGenerate/ReportGenerate.fixtures.js
-- webpack/ForemanInventoryUpload/Components/ReportGenerate/ReportGenerate.js
-- webpack/ForemanInventoryUpload/Components/ReportGenerate/ReportGenerateHelper.js
-- webpack/ForemanInventoryUpload/Components/ReportGenerate/__tests__/ReportGenerate.test.js
-- webpack/ForemanInventoryUpload/Components/ReportGenerate/__tests__/__snapshots__/ReportGenerate.test.js.snap
-- webpack/ForemanInventoryUpload/Components/ReportGenerate/index.js
-- webpack/ForemanInventoryUpload/Components/ReportGenerate/reportGenerate.scss
-- webpack/ForemanInventoryUpload/Components/ReportUpload/ReportUpload.fixtures.js
-- webpack/ForemanInventoryUpload/Components/ReportUpload/ReportUpload.js
-- webpack/ForemanInventoryUpload/Components/ReportUpload/ReportUploadHelper.js
-- webpack/ForemanInventoryUpload/Components/ReportUpload/__tests__/ReportUpload.test.js
-- webpack/ForemanInventoryUpload/Components/ReportUpload/__tests__/__snapshots__/ReportUpload.test.js.snap
-- webpack/ForemanInventoryUpload/Components/ReportUpload/index.js
-- webpack/ForemanInventoryUpload/Components/ReportUpload/reportUpload.scss
 - webpack/ForemanInventoryUpload/Components/ScheduledRun/ScheduledRun.fixtures.js
 - webpack/ForemanInventoryUpload/Components/ScheduledRun/ScheduledRun.js
 - webpack/ForemanInventoryUpload/Components/ScheduledRun/ScheduledRunHelper.js
@@ -506,13 +466,6 @@ files:
 - webpack/ForemanInventoryUpload/Components/StatusChart/__tests__/__snapshots__/StatusChart.test.js.snap
 - webpack/ForemanInventoryUpload/Components/StatusChart/index.js
 - webpack/ForemanInventoryUpload/Components/StatusChart/statusChart.scss
-- webpack/ForemanInventoryUpload/Components/TabBody/TabBody.fixtures.js
-- webpack/ForemanInventoryUpload/Components/TabBody/TabBody.js
-- webpack/ForemanInventoryUpload/Components/TabBody/TabBodyHelper.js
-- webpack/ForemanInventoryUpload/Components/TabBody/__tests__/TabBody.test.js
-- webpack/ForemanInventoryUpload/Components/TabBody/__tests__/__snapshots__/TabBody.test.js.snap
-- webpack/ForemanInventoryUpload/Components/TabBody/index.js
-- webpack/ForemanInventoryUpload/Components/TabBody/tabBody.scss
 - webpack/ForemanInventoryUpload/Components/TabContainer/TabContainer.fixtures.js
 - webpack/ForemanInventoryUpload/Components/TabContainer/TabContainer.js
 - webpack/ForemanInventoryUpload/Components/TabContainer/TabContainerHelper.js
@@ -533,13 +486,12 @@ files:
 - webpack/ForemanInventoryUpload/Components/TabHeader/__tests__/TabHeader.test.js
 - webpack/ForemanInventoryUpload/Components/TabHeader/index.js
 - webpack/ForemanInventoryUpload/Components/TabHeader/tabHeader.scss
-- webpack/ForemanInventoryUpload/Components/Terminal/Terminal.fixtures.js
-- webpack/ForemanInventoryUpload/Components/Terminal/Terminal.js
-- webpack/ForemanInventoryUpload/Components/Terminal/TerminalHelper.js
-- webpack/ForemanInventoryUpload/Components/Terminal/__tests__/Terminal.test.js
-- webpack/ForemanInventoryUpload/Components/Terminal/__tests__/__snapshots__/Terminal.test.js.snap
-- webpack/ForemanInventoryUpload/Components/Terminal/index.js
-- webpack/ForemanInventoryUpload/Components/Terminal/terminal.scss
+- webpack/ForemanInventoryUpload/Components/TaskHistory/TaskHistory.js
+- webpack/ForemanInventoryUpload/Components/TaskHistory/index.js
+- webpack/ForemanInventoryUpload/Components/TaskHistory/taskHistory.scss
+- webpack/ForemanInventoryUpload/Components/TaskProgress/TaskProgress.js
+- webpack/ForemanInventoryUpload/Components/TaskProgress/index.js
+- webpack/ForemanInventoryUpload/Components/TaskProgress/taskProgress.scss
 - webpack/ForemanInventoryUpload/ForemanInventoryConstants.js
 - webpack/ForemanInventoryUpload/ForemanInventoryHelpers.js
 - webpack/ForemanInventoryUpload/ForemanInventoryUpload.js
@@ -649,6 +601,7 @@ files:
 - webpack/__mocks__/foremanReact/components/Head.js
 - webpack/__mocks__/foremanReact/components/Layout/LayoutConstants.js
 - webpack/__mocks__/foremanReact/components/ToastsList/index.js
+- webpack/__mocks__/foremanReact/components/common/dates/RelativeDateTime.js
 - webpack/__mocks__/foremanReact/constants.js
 - webpack/__mocks__/foremanReact/redux/API/APISelectors.js
 - webpack/__mocks__/foremanReact/redux/API/index.js
@@ -701,7 +654,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.3
 specification_version: 4
 summary: Summary of ForemanRhCloud.
 test_files:
@@ -713,8 +666,6 @@ test_files:
 - test/controllers/insights_sync/settings_controller_test.rb
 - test/controllers/inventory_upload/api/inventory_controller_test.rb
 - test/controllers/inventory_upload/cloud_status_controller_test.rb
-- test/controllers/reports_controller_test.rb
-- test/controllers/uploads_controller_test.rb
 - test/controllers/uploads_settings_controller_test.rb
 - test/factories/insights_factories.rb
 - test/factories/inventory_upload_factories.rb
@@ -723,7 +674,6 @@ test_files:
 - 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
@@ -760,6 +710,5 @@ test_files:
 - test/unit/services/foreman_rh_cloud/tags_auth_test.rb
 - test/unit/services/foreman_rh_cloud/template_renderer_helper_test.rb
 - test/unit/services/foreman_rh_cloud/url_remediations_retriever_test.rb
-- test/unit/shell_process_job_test.rb
 - test/unit/slice_generator_test.rb
 - test/unit/tags_generator_test.rb

data/app/models/task_output_line.rb

@@ -1,2 +0,0 @@
-class TaskOutputLine < ApplicationRecord
-end

data/app/models/task_output_status.rb

@@ -1,2 +0,0 @@
-class TaskOutputStatus < ApplicationRecord
-end

data/lib/foreman_inventory_upload/async/generate_report_job.rb

@@ -1,61 +0,0 @@
-module ForemanInventoryUpload
-  module Async
-    class GenerateReportJob < ShellProcess
-      def self.output_label(label)
-        "report_for_#{label}"
-      end
-
-      def plan(base_folder, organization_id, disconnected = false, hosts_filter = nil)
-        sequence do
-          super(
-            GenerateReportJob.output_label("#{organization_id}#{hosts_filter.empty? ? nil : "[#{hosts_filter.to_s.parameterize}]"}"),
-            organization_id: organization_id,
-            base_folder: base_folder,
-            hosts_filter: hosts_filter
-          )
-
-          unless content_disconnected?(disconnected)
-            plan_action(
-              QueueForUploadJob,
-              base_folder,
-              ForemanInventoryUpload.facts_archive_name(organization_id, hosts_filter),
-              organization_id
-            )
-          end
-        end
-      end
-
-      def rake_prefix
-        'foreman-' unless Rails.env.development?
-      end
-
-      def command
-        "#{rake_prefix}rake rh_cloud_inventory:report:generate"
-      end
-
-      def env
-        super.merge(
-          'target' => base_folder,
-          'organization_id' => organization_id,
-          'hosts_filter' => hosts_filter
-        )
-      end
-
-      def content_disconnected?(disconnected)
-        disconnected || !Setting[:subscription_connection_enabled]
-      end
-
-      def base_folder
-        input[:base_folder]
-      end
-
-      def organization_id
-        input[:organization_id]
-      end
-
-      def hosts_filter
-        input[:hosts_filter]
-      end
-    end
-  end
-end

data/lib/foreman_inventory_upload/async/progress_output.rb

@@ -1,38 +0,0 @@
-module ForemanInventoryUpload
-  module Async
-    class ProgressOutput
-      def self.get(label)
-        ProgressOutput.new(label, :reader)
-      end
-
-      def self.register(label)
-        TaskOutputLine.where(label: @label).delete_all
-        ProgressOutput.new(label, :writer)
-      end
-
-      def initialize(label, mode)
-        @label = label
-        @mode = mode
-      end
-
-      def full_output
-        TaskOutputLine.where(label: @label).order(:created_at).pluck(:line).join("\n")
-      end
-
-      def write_line(line)
-        TaskOutputLine.create!(label: @label, line: line)
-      end
-
-      def close
-      end
-
-      def status
-        TaskOutputStatus.where(label: @label).pluck(:status).first || ''
-      end
-
-      def status=(status)
-        TaskOutputStatus.upsert({ label: @label, status: status }, unique_by: :label)
-      end
-    end
-  end
-end

data/lib/foreman_inventory_upload/async/shell_process.rb

@@ -1,77 +0,0 @@
-require 'open3'
-
-module ForemanInventoryUpload
-  module Async
-    class ShellProcess < ::Actions::EntryAction
-      include AsyncHelpers
-      include ::ForemanRhCloud::Async::ExponentialBackoff
-
-      def plan(instance_label, more_inputs = {})
-        clear_task_output(instance_label)
-        inputs = more_inputs.merge(instance_label: instance_label)
-        plan_self(inputs)
-      end
-
-      def try_execute
-        klass_name = self.class.name
-        logger.debug("Starting #{klass_name} with label #{instance_label}")
-        progress_output do |progress_output|
-          Open3.popen2e(hash_to_s(env), *preprocess_command(command)) do |_stdin, stdout_stderr, wait_thread|
-            progress_output.status = "Running in pid #{wait_thread.pid}"
-
-            stdout_stderr.each do |out_line|
-              progress_output.write_line(out_line)
-            end
-
-            progress_output.status = wait_thread.value.to_s
-          end
-        end
-        logger.debug("Finished job #{klass_name} with label #{instance_label}")
-
-        assert_task_status(ProgressOutput.get(instance_label).status)
-        done!
-      end
-
-      def command
-      end
-
-      def progress_output
-        progress_output = ProgressOutput.register(instance_label)
-        yield(progress_output)
-      ensure
-        progress_output.close
-      end
-
-      def env
-        {}
-      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
-
-      private
-
-      def preprocess_command(command)
-        command.kind_of?(Array) ? command : [command]
-      end
-
-      def instance_label
-        input[:instance_label]
-      end
-
-      def assert_task_status(status)
-        raise Foreman::Exception.new('Process exited with an unknown status: %{status}', status: status) unless status.match?(/pid \d+ exit 0/)
-      end
-    end
-  end
-end

data/test/controllers/reports_controller_test.rb

@@ -1,21 +0,0 @@
-require 'test_plugin_helper'
-
-class ReportsControllerTest < ActionController::TestCase
-  tests ForemanInventoryUpload::ReportsController
-
-  test 'Returns latest report generation status' do
-    progress_output = mock('progress_output')
-    test_org = FactoryBot.create(:organization)
-    ForemanInventoryUpload::Async::ProgressOutput
-      .expects(:get)
-      .with(ForemanInventoryUpload::Async::GenerateReportJob.output_label(test_org.id))
-      .returns(progress_output)
-    progress_output.expects(:full_output).returns('test output')
-
-    get :last, params: { organization_id: test_org.id }, session: set_session_user
-
-    assert_response :success
-    actual = JSON.parse(response.body)
-    assert_equal 'test output', actual['output']
-  end
-end

data/test/controllers/uploads_controller_test.rb

@@ -1,21 +0,0 @@
-require 'test_plugin_helper'
-
-class UploadsControllerTest < ActionController::TestCase
-  tests ForemanInventoryUpload::UploadsController
-
-  test 'Returns latest upload status' do
-    progress_output = mock('progress_output')
-    test_org = FactoryBot.create(:organization)
-    ForemanInventoryUpload::Async::ProgressOutput
-      .expects(:get)
-      .with(ForemanInventoryUpload::Async::UploadReportDirectJob.output_label(test_org.id))
-      .returns(progress_output)
-    progress_output.expects(:full_output).returns('test output')
-
-    get :last, params: { organization_id: test_org.id }, session: set_session_user
-
-    assert_response :success
-    actual = JSON.parse(response.body)
-    assert_equal 'test output', actual['output']
-  end
-end

data/test/jobs/generate_report_job_test.rb

@@ -1,146 +0,0 @@
-require 'test_plugin_helper'
-require 'foreman_tasks/test_helpers'
-
-class GenerateReportJobTest < ActiveSupport::TestCase
-  include Dynflow::Testing::Factories
-  include Dynflow::Testing::Assertions
-
-  let(:organization) { FactoryBot.create(:organization) }
-  let(:base_folder) { Dir.mktmpdir }
-
-  setup do
-    # Stub settings
-    Setting.stubs(:[]).with(:subscription_connection_enabled).returns(true)
-    Setting.stubs(:[]).with("foreman_tasks_sync_task_timeout").returns(120)
-    Setting.stubs(:[]).with(:content_default_http_proxy).returns(nil)
-    Setting.stubs(:[]).with(:http_proxy).returns(nil)
-  end
-
-  teardown do
-    FileUtils.remove_entry base_folder if Dir.exist?(base_folder)
-  end
-
-  test 'disconnected parameter defaults to false' do
-    # When disconnected defaults to false and subscription_connection is enabled,
-    # QueueForUploadJob should be scheduled
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::GenerateReportJob,
-      base_folder,
-      organization.id
-    )
-
-    assert_action_planned(action, ForemanInventoryUpload::Async::QueueForUploadJob)
-  end
-
-  test 'disconnected parameter can be set to true explicitly' do
-    # When disconnected is explicitly true, QueueForUploadJob should NOT be scheduled
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::GenerateReportJob,
-      base_folder,
-      organization.id,
-      true
-    )
-
-    refute_action_planned(action, ForemanInventoryUpload::Async::QueueForUploadJob)
-  end
-
-  test 'disconnected parameter can be set to false explicitly' do
-    # When disconnected is explicitly false and subscription_connection is enabled,
-    # QueueForUploadJob should be scheduled
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::GenerateReportJob,
-      base_folder,
-      organization.id,
-      false
-    )
-
-    assert_action_planned(action, ForemanInventoryUpload::Async::QueueForUploadJob)
-  end
-
-  test 'skips upload when subscription_connection_enabled is false' do
-    Setting.stubs(:[]).with(:subscription_connection_enabled).returns(false)
-
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::GenerateReportJob,
-      base_folder,
-      organization.id,
-      false
-    )
-
-    refute_action_planned(action, ForemanInventoryUpload::Async::QueueForUploadJob)
-  end
-
-  test 'schedules upload when disconnected is false and subscription_connection is enabled' do
-    Setting.stubs(:[]).with(:subscription_connection_enabled).returns(true)
-
-    expected_archive_name = ForemanInventoryUpload.facts_archive_name(organization.id, nil)
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::GenerateReportJob,
-      base_folder,
-      organization.id,
-      false
-    )
-
-    assert_action_planned_with(
-      action,
-      ForemanInventoryUpload::Async::QueueForUploadJob,
-      base_folder,
-      expected_archive_name,
-      organization.id
-    )
-  end
-
-  test 'handles hosts_filter parameter' do
-    hosts_filter = 'name~test'
-    expected_archive_name = ForemanInventoryUpload.facts_archive_name(organization.id, hosts_filter)
-
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::GenerateReportJob,
-      base_folder,
-      organization.id,
-      false,
-      hosts_filter
-    )
-
-    assert_action_planned_with(
-      action,
-      ForemanInventoryUpload::Async::QueueForUploadJob,
-      base_folder,
-      expected_archive_name,
-      organization.id
-    )
-  end
-
-  test 'output_label generates correct label' do
-    label = ForemanInventoryUpload::Async::GenerateReportJob.output_label('test_org')
-    assert_equal 'report_for_test_org', label
-  end
-
-  test 'output_label with filter includes parameterized filter' do
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::GenerateReportJob,
-      base_folder,
-      organization.id,
-      false,
-      'name~production'
-    )
-
-    # The output label should include organization id and parameterized filter
-    expected_label = "report_for_#{organization.id}[name-production]"
-    assert_equal expected_label, action.input[:instance_label]
-  end
-
-  test 'output_label without filter includes only organization id' do
-    action = create_and_plan_action(
-      ForemanInventoryUpload::Async::GenerateReportJob,
-      base_folder,
-      organization.id,
-      false,
-      ''
-    )
-
-    # The output label should include only organization id when filter is empty
-    expected_label = "report_for_#{organization.id}"
-    assert_equal expected_label, action.input[:instance_label]
-  end
-end

data/test/unit/shell_process_job_test.rb

@@ -1,29 +0,0 @@
-require 'test_plugin_helper'
-require 'foreman_tasks/test_helpers'
-
-class ShellProcessJobTest < ActiveSupport::TestCase
-  class TestProcessJob < ForemanInventoryUpload::Async::ShellProcess
-    def command
-      'echo testing env: $testenv'
-    end
-
-    def env
-      super.merge(
-        'testenv' => 'test_val'
-      )
-    end
-  end
-
-  include ForemanTasks::TestHelpers::WithInThreadExecutor
-  include FolderIsolation
-
-  test 'Runs a process with environment vars' do
-    label = Foreman.uuid
-    ForemanTasks.sync_task(TestProcessJob, label)
-
-    progress_output = ForemanInventoryUpload::Async::ProgressOutput.get(label)
-
-    assert_match(/test_val/, progress_output.full_output)
-    assert_match(/exit 0/, progress_output.status)
-  end
-end