pf5 on job_invocation wizard

Author: rmynar

airgun/entities/host_new.py

@@ -3,18 +3,19 @@
 from navmazing import NavigateToSibling
 from wait_for import wait_for
 
+from airgun.entities.all_hosts import AllHostsEntity
 from airgun.entities.host import HostEntity
 from airgun.navigation import NavigateStepWithWait as NavigateStep, navigator
+from airgun.views.all_hosts import AllHostsTableView
 from airgun.views.cloud_insights import RemediateSummary
 from airgun.views.fact import HostFactView
-from airgun.views.host import HostsView as LegacyHostsView
+from airgun.views.host import HostsJobInvocationStatusView, HostsView as LegacyHostsView
 from airgun.views.host_new import (
     AllAssignedRolesView,
     ContainerfileInstallCommandView,
     EditAnsibleRolesView,
     EditSystemPurposeView,
     EnableTracerView,
-    HostsView,
     InstallPackagesView,
     ManageHostCollectionModal,
     ManageHostStatusesView,
@@ -349,6 +350,72 @@ def schedule_job(self, entity_name, values):
         view.fill(values)
         view.submit.click()
 
+    def schedule_remote_job(self, entities_list, values, timeout=60, wait_for_results=True):
+        """Apply Schedule Remote Job action to the hosts names in entities_list using PF5 UI.
+
+        For multiple hosts, uses AllHostsEntity helper to select checkboxes, then clicks
+        "Schedule a job" button which opens the wizard with hosts pre-selected.
+
+        :param entities_list: The host names to apply the remote job.
+        :param values: the values to fill The Job invocation view.
+        :param timeout: The time to wait for the job to finish.
+        :param wait_for_results: Whether to wait for the job to finish execution.
+
+        :returns: The job invocation status view values
+        """
+        if len(entities_list) == 1:
+            # Single host - use the host details page approach
+            entity_name = entities_list[0]
+            details_view = self.navigate_to(self, 'NewDetails', entity_name=entity_name)
+            self.browser.plugin.ensure_page_safe()
+            details_view.wait_displayed()
+            # Click the Schedule a job dropdown
+            self.browser.wait_for_element(details_view.schedule_job, exception=False)
+            details_view.schedule_job.fill('Schedule a job')
+            # The job invocation form appears in the same browser context
+            view = JobInvocationCreateView(self.browser)
+            self.browser.plugin.ensure_page_safe()
+        else:
+            # Multiple hosts - use AllHostsEntity helper to navigate and select hosts
+            # This helper handles: clearing previous selections, searching, selecting checkboxes
+            # and returns AllHostsTableView with selected hosts
+            hosts_view = AllHostsEntity.all_hosts_navigate_and_select_hosts_helper(
+                self, host_names=entities_list
+            )
+
+            # After selecting hosts, click the "Schedule a job" button
+            # This button appears dynamically after hosts are selected
+            schedule_button_locator = ".//button[contains(., 'Schedule a job') or @data-ouia-component-id='schedule-a-job-dropdown']"
+            wait_for(
+                lambda: hosts_view.browser.element(schedule_button_locator, check_visibility=True),
+                timeout=10,
+                delay=0.5,
+                handle_exception=True,
+            )
+            schedule_button = hosts_view.browser.element(schedule_button_locator)
+            hosts_view.browser.click(schedule_button)
+
+            # The job invocation form should now open with hosts pre-selected
+            view = JobInvocationCreateView(self.browser)
+            self.browser.plugin.ensure_page_safe()
+            # Wait for the wizard to be displayed
+            view.wait_displayed()
+
+        # Fill the form with provided values
+        view.fill(values)
+        # Wait for form to be fully filled and ready for submission
+        self.browser.plugin.ensure_page_safe()
+        view.submit.click()
+        view.flash.assert_no_error()
+        view.flash.dismiss()
+        # Wait for status view to appear after job submission
+        status_view = HostsJobInvocationStatusView(self.browser)
+        self.browser.plugin.ensure_page_safe()
+        status_view.wait_displayed()
+        if wait_for_results:
+            status_view.wait_for_result(timeout=timeout)
+        return status_view.read()
+
     def run_job(self, entity_name):
         """Run a job on selected host"""
         view = self.navigate_to(self, 'NewDetails', entity_name=entity_name)
@@ -1231,7 +1298,7 @@ def get_content_view_envs(self, entity_name):
 class ShowAllHosts(NavigateStep):
     """Navigate to new UI All Hosts page"""
 
-    VIEW = HostsView
+    VIEW = AllHostsTableView
 
     def step(self, *args, **kwargs):
         self.view.menu.select('Hosts', 'All Hosts')
@@ -1252,6 +1319,7 @@ def prerequisite(self, *args, **kwargs):
 
     def step(self, *args, **kwargs):
         entity_name = kwargs.get('entity_name')
+
         self.parent.search(entity_name)
         self.parent.table.row(name=entity_name)['Name'].widget.click()
 

airgun/entities/job_invocation.py

@@ -22,9 +22,7 @@ def run(self, values):
         """Run specific job"""
         view = self.navigate_to(self, 'Run')
         view.fill(values)
-        view.submit.expander.click()
-        self.browser.wait_for_element(view.submit.submit, exception=False)
-        view.submit.click()
+        view.submit.click()  # PF5 wizard: submit.click() handles everything
 
     def search(self, value):
         """Search for specific job invocation"""
@@ -40,6 +38,7 @@ def wait_job_invocation_state(self, entity_name, host_name, expected_state='succ
         """Check job invocation state from table view"""
         view = self.navigate_to(self, 'All')
         view.search(f'host = {host_name}')
+
         wait_for(
             lambda: view.table.row(description=entity_name)['Status'].read() == expected_state,
             timeout=300,

airgun/views/all_hosts.py

@@ -106,6 +106,9 @@ class AllHostsTableView(BaseLoggedInView, SearchableViewMixinPF4):
         locator='//li[contains(@class, "pf-v5-c-menu__list-item")]//button[span/span[text()="Change associations"]]/following-sibling::div[contains(@class, "pf-v5-c-menu")]'
     )
 
+    # Additional action buttons
+    schedule_a_job = PF5OUIADropdown('schedule-a-job-dropdown')
+
     table_loading = Text('//h5[normalize-space(.)="Loading"]')
     no_results = Text('//h5[normalize-space(.)="No Results"]')
     manage_columns = PF5Button('Manage columns')

airgun/views/common.py

@@ -772,3 +772,18 @@ def __init__(self, parent, logger=None):
         """Expand the selected wizard step"""
         View.__init__(self, parent, logger=logger)
         self.expander.click()
+
+
+class PF5WizardStepView(View):
+    """PF5 Wizard step - doesn't use expanders, uses wizard navigation instead.
+
+    The PF5 wizard pattern navigates between steps using Next/Back buttons
+    in the footer, not by clicking expander buttons. Steps are already visible
+    when you're on that step.
+    """
+
+    def __init__(self, parent, logger=None):
+        """Initialize PF5 wizard step without clicking expander."""
+        View.__init__(self, parent, logger=logger)
+        # PF5 wizards don't have expanders - steps are navigated via Next/Back buttons
+        # The step is already visible when active

airgun/views/host_new.py

@@ -268,7 +268,7 @@ class HostsView(BaseLoggedInView, SearchableViewMixinPF4):
 
     @property
     def is_displayed(self):
-        return self.title.is_displayed
+        return self.browser.wait_for_element(self.title, exception=False) is not None
 
 
 class BreadcrumbSwitcher(Widget):