foreman_openbolt 1.0.0 → 1.1.1

data/test/acceptance/tests/ssh_options_test.rb

@@ -0,0 +1,77 @@
+# frozen_string_literal: true
+
+require_relative '../acceptance_helper'
+
+# Tests that SSH transport options are correctly passed through to the
+# OpenBolt CLI and affect task execution.
+class SshOptionsTest < AcceptanceTestCase
+  def setup
+    super
+    foreman_login
+    visit '/foreman_openbolt/page_launch_task'
+    assert_selector '#smart-proxy-input', wait: 15
+    select_first_proxy
+  end
+
+  def test_default_user_passed_to_bolt
+    select_hosts_via_search('target1')
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+    select 'acceptance::echo', from: 'task-name-input'
+    fill_in 'param_message', with: 'user test'
+    click_button 'Launch Task'
+    assert_selector 'h1', text: 'Task Execution', wait: 15
+
+    assert_task_completed
+    # The default user (from settings) should appear in the bolt command on the Log Output tab
+    assert_log_contains '--user=openbolt'
+  end
+
+  def test_user_override_passed_to_bolt
+    select_hosts_via_search('target1')
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+    select 'acceptance::echo', from: 'task-name-input'
+    fill_in 'param_message', with: 'override test'
+
+    set_openbolt_option('user', 'root')
+    click_button 'Launch Task'
+    assert_selector 'h1', text: 'Task Execution', wait: 15
+
+    # Task may fail because root doesn't have the SSH key, but the
+    # command should show the overridden user
+    assert_selector '.pf-v5-c-label', text: /Success|Failed/i, wait: 120
+    assert_log_contains '--user=root'
+  end
+
+  def test_host_key_check_false_passed_to_bolt
+    select_hosts_via_search('target1')
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+    select 'acceptance::noop_task', from: 'task-name-input'
+
+    # host-key-check is a boolean OpenBolt option and must render as a
+    # checkbox (ParameterField's boolean branch), not a text input.
+    assert_equal 'checkbox', find_by_id('param_host-key-check')['type']
+
+    set_openbolt_option('host-key-check', false)
+    click_button 'Launch Task'
+    assert_selector 'h1', text: 'Task Execution', wait: 15
+
+    assert_task_completed
+    assert_log_contains '--no-host-key-check'
+  end
+
+  def test_verbose_flag_passed_to_bolt
+    select_hosts_via_search('target1')
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+    select 'acceptance::noop_task', from: 'task-name-input'
+
+    # verbose is boolean and must render as a checkbox.
+    assert_equal 'checkbox', find_by_id('param_verbose')['type']
+
+    set_openbolt_option('verbose', true)
+    click_button 'Launch Task'
+    assert_selector 'h1', text: 'Task Execution', wait: 15
+
+    assert_task_completed
+    assert_log_contains '--verbose'
+  end
+end

data/test/acceptance/tests/task_execution_test.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+require_relative '../acceptance_helper'
+
+# Tests for the Task Execution page: URL validation, tab switching, and
+# task metadata display on the Task Details tab.
+class TaskExecutionTest < AcceptanceTestCase
+  def setup
+    super
+    foreman_login
+  end
+
+  def test_execution_page_without_job_id_redirects_to_launch
+    # Visiting the execution URL without a job_id query parameter should
+    # client-side redirect back to the Launch page (TaskExecution/index.js
+    # useEffect with !jobId calls history.push(LAUNCH_TASK)).
+    visit '/foreman_openbolt/page_task_execution'
+    assert_selector 'h1', text: 'Launch OpenBolt Task', wait: 15
+  end
+
+  def test_task_details_tab_shows_task_name_and_submitted_parameters
+    launch_task_via_ui('acceptance::echo',
+      params: { 'message' => 'details tab test' })
+    assert_task_completed
+
+    # Click the "Task Details" tab (second tab in ExecutionDisplay).
+    find('.pf-v5-c-tabs__link', text: 'Task Details').click
+
+    # TaskDetails renders task name in a DescriptionList and the
+    # submitted parameters in a table with aria-label="Task parameters".
+    assert_selector 'dt', text: 'Task Name', wait: 10
+    assert_selector 'dd', text: 'acceptance::echo'
+
+    assert_selector 'table[aria-label="Task parameters"]', wait: 10
+    within 'table[aria-label="Task parameters"]' do
+      assert_selector 'td', text: 'message'
+      assert_selector 'td', text: 'details tab test'
+    end
+  end
+end

data/test/acceptance/tests/task_history_test.rb

@@ -0,0 +1,84 @@
+# frozen_string_literal: true
+
+require_relative '../acceptance_helper'
+
+# Tests the task history page: listing, navigation, and new entry creation.
+class TaskHistoryTest < AcceptanceTestCase
+  def setup
+    super
+    foreman_login
+  end
+
+  def test_history_shows_completed_tasks_and_navigation
+    # Launch a task to seed history
+    launch_task_via_ui('acceptance::echo', params: { 'message' => 'history test' })
+    assert_task_completed
+
+    # Visit history and verify the entry appears
+    visit '/foreman_openbolt/page_task_history'
+    assert_selector 'h1', text: 'Task History', wait: 15
+    assert_no_selector '[aria-label="Loading task history"]', wait: 15
+    assert_selector 'table[aria-label="Task history table"] tbody tr', minimum: 1
+    # Capture the first row so we can confirm a newer entry lands above it.
+    # History paginates (default 20 rows), so counting rows is unreliable
+    # once history fills the first page.
+    first_row_before = first('table[aria-label="Task history table"] tbody tr').text
+
+    # Navigate to execution details from history
+    first('a[aria-label="View Details"]').click
+    assert_selector 'h1', text: 'Task Execution', wait: 15
+
+    # Launch another task and verify it appears at the top of history.
+    launch_task_via_ui('acceptance::noop_task')
+    assert_task_completed
+
+    visit '/foreman_openbolt/page_task_history'
+    assert_selector 'table[aria-label="Task history table"] tbody tr', minimum: 1, wait: 15
+    first_row_after = first('table[aria-label="Task history table"] tbody tr').text
+    assert_not_equal first_row_before, first_row_after,
+      'Expected newly launched task to appear as the first history row'
+  end
+
+  def test_hosts_popover_lists_targets_from_history_row
+    # 'target' matches both target1 and target2, so the popover count
+    # will be 2.
+    launch_task_via_ui('acceptance::noop_task')
+    assert_task_completed
+
+    visit '/foreman_openbolt/page_task_history'
+    assert_no_selector '[aria-label="Loading task history"]', wait: 15
+    assert_selector 'table[aria-label="Task history table"] tbody tr', minimum: 1, wait: 15
+
+    # The hosts count button has aria-label '<count> target hosts'
+    within first('table[aria-label="Task history table"] tbody tr') do
+      find('button[aria-label$=" target hosts"]').click
+    end
+
+    # Popover content is portaled to document body
+    assert_selector 'table[aria-label="Target hosts"]', wait: 10
+    within 'table[aria-label="Target hosts"]' do
+      assert_selector 'td', text: 'target1'
+      assert_selector 'td', text: 'target2'
+    end
+  end
+
+  def test_task_popover_shows_submitted_parameters_on_history_row
+    launch_task_via_ui('acceptance::echo',
+      params: { 'message' => 'popover history param' })
+    assert_task_completed
+
+    visit '/foreman_openbolt/page_task_history'
+    assert_no_selector '[aria-label="Loading task history"]', wait: 15
+    assert_selector 'table[aria-label="Task history table"] tbody tr', minimum: 1, wait: 15
+
+    # The task name is rendered as a popover trigger whose aria-label is
+    # "View details for task <name>"
+    first('button[aria-label="View details for task acceptance::echo"]').click
+
+    assert_selector 'table[aria-label="Task parameters"]', wait: 10
+    within 'table[aria-label="Task parameters"]' do
+      assert_selector 'td', text: 'message'
+      assert_selector 'td', text: 'popover history param'
+    end
+  end
+end

data/test/acceptance/tests/transport_options_test.rb

@@ -0,0 +1,121 @@
+# frozen_string_literal: true
+
+require_relative '../acceptance_helper'
+
+# Tests that the proxy correctly loads and exposes tasks, and that
+# task metadata (parameters) is displayed in the UI.
+class TransportOptionsTest < AcceptanceTestCase
+  def setup
+    super
+    foreman_login
+    visit '/foreman_openbolt/page_launch_task'
+    assert_selector '#smart-proxy-input', wait: 15
+    select_first_proxy
+  end
+
+  def test_proxy_selection_populates_task_list
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+  end
+
+  def test_all_fixture_tasks_discoverable
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+    options = all('#task-name-input option').map(&:text)
+
+    %w[echo complex_params failing_task noop_task slow_task target_conditional].each do |task|
+      assert options.any?("acceptance::#{task}"),
+        "Expected acceptance::#{task} in task list, got: #{options}"
+    end
+  end
+
+  def test_echo_task_shows_message_param_as_empty_text_input
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+    select 'acceptance::echo', from: 'task-name-input'
+    message = find_by_id('param_message', wait: 10)
+    assert_equal 'text', message['type']
+    assert_equal '', message.value
+  end
+
+  def test_complex_params_task_populates_fields_with_correct_defaults
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+    select 'acceptance::complex_params', from: 'task-name-input'
+
+    # No-default fields render empty.
+    assert_equal '', find_by_id('param_required_string', wait: 10).value
+    assert_equal '', find_by_id('param_array_param').value
+    # with_default has a 'default_value' default in its task metadata;
+    # the launch page must populate it when the task is selected
+    # (handleTaskChange in LaunchTask/index.js).
+    assert_equal 'default_value', find_by_id('param_with_default').value
+  end
+
+  def test_slow_task_shows_seconds_param_with_default_value
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+    select 'acceptance::slow_task', from: 'task-name-input'
+    # slow_task.json declares "default": 5
+    assert_equal '5', find_by_id('param_seconds', wait: 10).value
+  end
+
+  def test_target_conditional_shows_succeed_on_param_as_empty_text_input
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+    select 'acceptance::target_conditional', from: 'task-name-input'
+    succeed_on = find_by_id('param_succeed_on', wait: 10)
+    assert_equal 'text', succeed_on['type']
+    assert_equal '', succeed_on.value
+  end
+
+  def test_switching_tasks_swaps_parameter_fields_and_defaults
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+
+    select 'acceptance::echo', from: 'task-name-input'
+    assert_equal '', find_by_id('param_message', wait: 10).value
+
+    select 'acceptance::slow_task', from: 'task-name-input'
+    assert_no_selector '#param_message', wait: 5
+    # Asserting the seconds default of 5 (not just field presence)
+    # proves the new task's metadata loaded rather than stale state.
+    assert_equal '5', find_by_id('param_seconds', wait: 10).value
+  end
+
+  def test_deselecting_proxy_clears_task_selection_and_parameters
+    assert_selector '#task-name-input option', minimum: 2, wait: 15
+    select 'acceptance::echo', from: 'task-name-input'
+    assert_selector '#param_message', wait: 10
+
+    # Deselect by choosing the placeholder option (value="")
+    select 'Select Smart Proxy', from: 'smart-proxy-input'
+
+    # Task selection and its parameters should be cleared, showing the
+    # "Select a task to see parameters" empty state.
+    assert_no_selector '#param_message', wait: 10
+    assert_selector '.pf-v5-c-empty-state',
+      text: 'Select a task to see parameters', wait: 10
+  end
+
+  def test_transport_option_renders_as_select_with_ssh_and_winrm
+    # The transport OpenBolt option has type ["ssh", "winrm"] and
+    # ParameterField renders array-typed values as a <select>.
+    field = find_by_id('param_transport', wait: 10)
+    assert_equal 'select', field.tag_name
+    option_values = all('#param_transport option').map(&:value)
+    assert_equal %w[ssh winrm], option_values
+    assert_equal 'ssh', field.value
+  end
+
+  def test_switching_transport_to_winrm_hides_ssh_only_options
+    # SSH-only options (private-key, host-key-check) are present on load
+    # because the default transport is ssh.
+    assert_selector '#param_host-key-check', wait: 10
+    assert_selector '#param_private-key', wait: 10
+
+    # Switching transport to winrm should hide options whose metadata
+    # tags them as ssh-only (OpenBoltOptionsSection filters by transport).
+    select 'winrm', from: 'param_transport'
+    assert_no_selector '#param_host-key-check', wait: 10
+    assert_no_selector '#param_private-key', wait: 10
+
+    # Switching back restores them
+    select 'ssh', from: 'param_transport'
+    assert_selector '#param_host-key-check', wait: 10
+    assert_selector '#param_private-key', wait: 10
+  end
+end

data/test/test_plugin_helper.rb

@@ -1,8 +1,17 @@
 # frozen_string_literal: true
 
-# This calls the main test_helper in Foreman-core
 require 'test_helper'
+require 'dynflow/testing'
 
-# Add plugin to FactoryBot's paths
-FactoryBot.definition_file_paths << File.join(File.dirname(__FILE__), 'factories')
+# Load plugin factories and foreman-tasks factories
+FactoryBot.definition_file_paths << File.join(File.dirname(__FILE__), 'unit', 'factories')
+FactoryBot.definition_file_paths << File.join(ForemanTasks::Engine.root, 'test', 'factories')
 FactoryBot.reload
+
+module ForemanOpenbolt
+  class PluginTestCase < ActiveSupport::TestCase
+    teardown do
+      WebMock.reset!
+    end
+  end
+end