Errata install, and legacy chost methods

airgun/entities/contenthost.py

@@ -121,7 +121,8 @@ def execute_module_stream_action(
 
     def search_package(self, entity_name, package_name):
         """Search for specific package installed in content host"""
-        view = self.navigate_to(self, 'Edit', entity_name=entity_name)
+        view = self.navigate_to(self, 'LegacyDetails', entity_name=entity_name)
+        view.packages_installed.wait_displayed()
         view.packages_installed.search(package_name)
         return view.packages_installed.table.read()
 
@@ -145,7 +146,8 @@ def install_errata(self, entity_name, errata_id, install_via='rex'):
 
         :return: Returns a dict containing task status details
         """
-        view = self.navigate_to(self, 'Edit', entity_name=entity_name)
+        view = self.navigate_to(self, 'LegacyDetails', entity_name=entity_name)
+        view.errata.wait_displayed()
         if errata_id == "All":
             view.errata.select_all.fill(True)
         else:

airgun/entities/errata.py

@@ -5,9 +5,9 @@
 from airgun.views.errata import (
     ErrataDetailsView,
     ErrataInstallationConfirmationView,
-    ErrataTaskDetailsView,
     ErratumView,
 )
+from airgun.views.job_invocation import JobInvocationStatusView
 
 
 class ErrataEntity(BaseEntity):
@@ -59,11 +59,16 @@ def read(
             view.content_hosts.environment_filter.fill(environment)
         return view.read(widget_names=widget_names)
 
-    def install(self, entity_name, host_name):
+    def install(self, entity_name, host_names, environment=None):
         """Install errata on content host.
 
         :param str entity_name: errata id or title
-        :param str host_name: content host name to apply errata on
+        :param str host_names: content host name to apply errata on.
+            pass a single str hostname,
+            or pass 'All' for any available hosts,
+            or pass a list of str hostnames.
+        :param str environment: name of lifecycle environment to scope errata.
+            default: None
         """
         view = self.navigate_to(
             self,
@@ -73,13 +78,31 @@ def install(self, entity_name, host_name):
             installable=False,
             repo=None,
         )
-        view.content_hosts.search(host_name)
-        view.content_hosts.table.row(name=host_name)[0].fill(True)
+        if environment:
+            view.content_hosts.environment_filter.fill(environment)
+        view.content_hosts.wait_displayed()
+
+        if host_names == 'All':
+            # select_all on table, check if single or multiple hosts
+            view.content_hosts.select_all.fill(True)
+        elif isinstance(host_names, list):
+            # find and select hostnames in table from list
+            for hostname in host_names:
+                view.content_hosts.table.row(name=hostname)[0].fill(True)
+        else:
+            # search and select the single passed hostname
+            view.content_hosts.search(host_names)
+            view.content_hosts.wait_displayed()
+            view.content_hosts.table.row(name=host_names)[0].fill(True)
+        view.content_hosts.wait_displayed()
         view.content_hosts.apply.click()
+        # brought to confirmation page
         view = ErrataInstallationConfirmationView(view.browser)
+        view.wait_displayed()
         view.confirm.click()
-        view = ErrataTaskDetailsView(view.browser)
-        view.progressbar.wait_for_result()
+        # wait for redirect to task details page
+        view = JobInvocationStatusView(view.browser)
+        view.wait_for_result(delay=0.01, timeout=1200)
         return view.read()
 
     def search_content_hosts(self, entity_name, value, environment=None):

airgun/entities/host_new.py

@@ -326,6 +326,7 @@ def get_packages(self, entity_name, search=""):
         """Filter installed packages on host"""
         view = self.navigate_to(self, 'NewDetails', entity_name=entity_name)
         view.content.packages.select()
+        view.content.packages.table.wait_displayed()
         view.content.packages.searchbar.fill(search)
         # wait for filter to apply
         self.browser.plugin.ensure_page_safe()

airgun/views/errata.py

@@ -80,6 +80,7 @@ class content_hosts(SatTab):
         TAB_NAME = 'Content Hosts'
         environment_filter = SatSelect(".//select[@ng-model='environmentFilter']")
         searchbox = Search()
+        select_all = Checkbox(locator=".//input[@type='checkbox'][@ng-change='allSelected()']")
         apply = Text(".//button[@ng-click='goToNextStep()']")
         table = SatTable(
             locator=".//table",
@@ -188,7 +189,7 @@ def is_displayed(self):
 
 class ErrataInstallationConfirmationView(BaseLoggedInView):
     cancel = Text(".//button[@ng-click='transitionBack()']")
-    confirm = Text(".//button[@type='submit']")
+    confirm = Text(".//span[text()='Confirm']")
 
 
 class ErrataTaskDetailsView(TaskDetailsView):

airgun/views/host_new.py

@@ -316,7 +316,9 @@ class packages(Tab):
             ROOT = './/div[@id="packages-tab"]'
 
             select_all = Checkbox(locator='.//div[@id="selection-checkbox"]/div/label')
-            searchbar = SearchInput(locator='.//input[contains(@class, "pf-m-search")]')
+            searchbar = SearchInput(
+                locator='.//input[contains(@class, "pf-c-text-input-group__text-input")]'
+            )
             status_filter = Dropdown(locator='.//div[@aria-label="select Status container"]/div')
             upgrade = Pf4ActionsDropdown(locator='.//div[div/button[normalize-space(.)="Upgrade"]]')
             dropdown = Dropdown(locator='.//div[button[@aria-label="bulk_actions"]]')