Fix TaskDetailsView and SatFlashMessages for PF5 UI updates

TaskDetailsView: The task details page no longer uses a "Task" tab.
Metadata (Result, State, etc.) is now in an overview section using
<small> labels. Progress bar disappears on completion. Result text
is now capitalized ("Success" instead of "success").

Changes:
- Change task from PF5Tab to plain View with overview section ROOT
- Update TaskReadOnlyEntry locator for <small> label pattern
- Normalize TaskReadOnlyEntry.read() to lowercase for compatibility
- Remove progressbar.is_displayed check from wait_for_result
- Update state, progressbar, output, dynflow_console locators

SatFlashMessages: The flash container <ul> gained an extra class
(foreman-toast-group), breaking the exact @Class= match.

Changes:
- Use contains() instead of exact class match in ROOT locator

Test commands:
  pytest tests/foreman/ui/test_hostcollection.py::test_negative_hosts_limit
  pytest tests/foreman/ui/test_hostcollection.py::test_positive_change_assigned_content
  pytest tests/foreman/ui/test_product.py::test_positive_bulk_action_advanced_sync
  pytest tests/foreman/ui/test_repository.py::test_positive_sync_yum_repo_and_verify_content_checksum

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: jnagare-redhat

airgun/views/task.py

@@ -1,7 +1,7 @@
 from wait_for import wait_for
 from widgetastic.widget import Table, Text, View
 from widgetastic_patternfly import BreadCrumb, Button
-from widgetastic_patternfly5 import Pagination as PF5Pagination, Tab as PF5Tab
+from widgetastic_patternfly5 import Button as PF5Button, Pagination as PF5Pagination
 
 from airgun.views.common import BaseLoggedInView, SearchableViewMixinPF4
 from airgun.widgets import (
@@ -21,10 +21,12 @@
 
 class TaskReadOnlyEntry(ReadOnlyEntry):
     BASE_LOCATOR = (
-        "//span[contains(., '{}') and contains(@class, 'list-group-item-heading')]//parent::div"
-        '/following-sibling::div/span'
+        "//small[normalize-space(.)='{}']/parent::div/following-sibling::div"
     )
 
+    def read(self):
+        return super().read().lower()
+
 
 class TaskReadOnlyEntryError(ReadOnlyEntry):
     BASE_LOCATOR = "//span[contains(., '{}')]//parent::div/following-sibling::pre"
@@ -90,27 +92,31 @@ def is_displayed(self):
         )
 
     @View.nested
-    class task(PF5Tab):
-        name = TaskReadOnlyEntry(name='Name')
+    class task(View):
+        ROOT = '//section[contains(@class, "task-details-overview-section")]'
         result = TaskReadOnlyEntry(name='Result')
         triggered_by = TaskReadOnlyEntry(name='Triggered by')
         execution_type = TaskReadOnlyEntry(name='Execution type')
         start_at = TaskReadOnlyEntry(name='Start at')
         started_at = TaskReadOnlyEntry(name='Started at')
         ended_at = TaskReadOnlyEntry(name='Ended at')
-        start_before = TaskReadOnlyEntry(name='Start before')
-        state = Text("//div[contains(@class, 'progress-description')]")
-        progressbar = ProgressBar(locator='//div[contains(@class,"progress__bar")]')
-        output = TaskReadOnlyEntry(name='Output')
+        state = Text(
+            "//p[@data-ouia-component-id='task-info-running-state-summary']"
+        )
+        progressbar = ProgressBar(
+            locator='//div[contains(@class,"pf-v5-c-progress__bar")]'
+        )
+        output = ReadOnlyEntry(
+            locator="//strong[normalize-space(.)='Output']/parent::div/following-sibling::div"
+        )
         errors = TaskReadOnlyEntryError(name='Errors')
-        dynflow_console = Button('Dynflow console')
+        dynflow_console = PF5Button('Dynflow console')
 
     def wait_for_result(self, timeout=60, delay=1):
         """Wait for invocation job to finish"""
         wait_for(
             lambda: (
                 self.is_displayed
-                and self.task.progressbar.is_displayed
                 and self.task.result.read() == 'success'
             ),
             timeout=timeout,

airgun/widgets.py

@@ -1130,7 +1130,7 @@ class SatFlashMessages(FlashMessages):
 
     """
 
-    ROOT = '//ul[@class="pf-v5-c-alert-group pf-m-toast"]'
+    ROOT = '//ul[contains(@class, "pf-v5-c-alert-group") and contains(@class, "pf-m-toast")]'
     MSG_LOCATOR = f'{ROOT}//div[contains(@class, "foreman-toast")]'
     msg_class = SatFlashMessage