Extend flatpak view and entity with remote details page and actions (SatelliteQE#1924)

vsedmik · web-flow · commit 3e9beb5af749 · 2025-08-12T14:34:02.000+05:30
diff --git a/airgun/entities/flatpak.py b/airgun/entities/flatpak.py
@@ -1,6 +1,16 @@
+from time import sleep
+
 from airgun.entities.base import BaseEntity
 from airgun.navigation import NavigateStep, navigator
-from airgun.views.flatpak import FlatpakRemotesView
+from airgun.utils import retry_navigation
+from airgun.views.flatpak import (
+    CreateFlatpakRemoteModal,
+    EditFlatpakRemoteModal,
+    FlatpakRemoteDeleteModal,
+    FlatpakRemoteDetailsView,
+    FlatpakRemotesView,
+    MirrorFlatpakRemoteModal,
+)
 
 
 class FlatpakRemotesEntity(BaseEntity):
@@ -40,17 +50,38 @@ def read_table(self):
         view.wait_displayed()
         return view.table.read()
 
-    def scan(self, entity_name):
-        """Scan a Flatpak remote.
+    def read_remote_details(self, name, repo_search=None):
+        """
+        Read details from the Flatpak remote details page
 
-        :param str entity_name: name of the remote to scan
+        Args:
+            name (str): Name of the flatpak remote to be read
+            repo_search (str, optional): Name of scanned remote repository to be searched
         """
         view = self.navigate_to(self, 'All')
         view.wait_displayed()
-        view.table.row(name=entity_name)[2].click()
-        view.table.row(name=entity_name)[2].widget.item_select('Scan')
-        view.flash.assert_no_error()
-        view.flash.dismiss()
+        view.search(name)
+        view.table.row(name=name)['Name'].widget.click()
+        view = FlatpakRemoteDetailsView(self.browser)
+        view.wait_displayed()
+        if repo_search:
+            view.search(repo_search)
+        return view.read()
+
+    def create(self, values):
+        """Create a Flatpak remote.
+
+        :param dict values: values to create the remote with
+        """
+        view = self.navigate_to(self, 'All')
+        view.wait_displayed()
+        view.create_new_btn.click()
+        create_modal = CreateFlatpakRemoteModal(self.browser)
+        create_modal.wait_displayed()
+        create_modal.fill(values)
+        create_modal.create_btn.click()
+        view = FlatpakRemoteDetailsView(self.browser)
+        view.wait_displayed(delay=3)
 
     def edit(self, entity_name, values):
         """Edit a Flatpak remote.
@@ -62,10 +93,12 @@ def edit(self, entity_name, values):
         view.wait_displayed()
         view.table.row(name=entity_name)[2].click()
         view.table.row(name=entity_name)[2].widget.item_select('Edit')
-        view.fill(values)
-        view.submit.click()
-        view.flash.assert_no_error()
-        view.flash.dismiss()
+        edit_modal = EditFlatpakRemoteModal(self.browser)
+        edit_modal.wait_displayed()
+        edit_modal.fill(values)
+        edit_modal.update_btn.click()
+        view = FlatpakRemoteDetailsView(self.browser)
+        view.wait_displayed(delay=3)
 
     def delete(self, entity_name):
         """Delete a Flatpak remote.
@@ -76,16 +109,55 @@ def delete(self, entity_name):
         view.wait_displayed()
         view.table.row(name=entity_name)[2].click()
         view.table.row(name=entity_name)[2].widget.item_select('Delete')
-        self.browser.handle_alert()
+        delete_modal = FlatpakRemoteDeleteModal(self.browser)
+        delete_modal.wait_displayed()
+        delete_modal.delete_btn.click()
+        sleep(3)
+        self.browser.plugin.ensure_page_safe()
+        view = FlatpakRemotesView(self.browser)
+        view.wait_displayed(delay=3)
+
+    def scan(self, entity_name):
+        """Scan a Flatpak remote.
+
+        :param str entity_name: name of the remote to scan
+        """
+        view = self.navigate_to(self, 'All')
+        view.wait_displayed()
+        view.table.row(name=entity_name)[2].click()
+        view.table.row(name=entity_name)[2].widget.item_select('Scan')
         view.flash.assert_no_error()
         view.flash.dismiss()
 
+    def mirror(self, remote, repo, product):
+        """Mirror a repository from Flatpak remote.
+
+        :param str remote: name of the remote to mirror from
+        :param str repo: name of the repository to be mirrored
+        :param str product: name of the product where the repository should be mirrored
+        """
+        view = self.navigate_to(self, 'All')
+        view.wait_displayed()
+        view.search(remote)
+        view.table.row(name=remote)['Name'].widget.click()
+        view = FlatpakRemoteDetailsView(self.browser)
+        view.wait_displayed()
+        view.search(repo)  # in case of many repos
+        view.table.row(name=repo)['Mirror'].widget.click()
+        mirror_modal = MirrorFlatpakRemoteModal(self.browser)
+        mirror_modal.wait_displayed()
+        mirror_modal.searchbar.fill(product.name)
+        self.browser.plugin.ensure_page_safe()
+        mirror_modal.mirror_btn.click()
+        view.wait_displayed(delay=3)
+
 
 @navigator.register(FlatpakRemotesEntity, 'All')
 class ShowAllFlatpakRemotes(NavigateStep):
     """Navigate to All Flatpak Remotes page."""
 
     VIEW = FlatpakRemotesView
 
+    @retry_navigation
     def step(self, *args, **kwargs):
         self.view.menu.select('Content', 'Flatpak Remotes')
diff --git a/airgun/views/flatpak.py b/airgun/views/flatpak.py
@@ -1,13 +1,19 @@
 from widgetastic.widget import Text
 from widgetastic_patternfly5 import (
+    Button as PF5Button,
     Menu as PF5Menu,
+    Modal as PF5Modal,
     Pagination,
 )
 from widgetastic_patternfly5.ouia import (
     PatternflyTable as PF5OUIATable,
+    Text as PF5OUIAText,
+    TextInput as PF5OUIATextInput,
+    Title as PF5OUIATitle,
 )
 
 from airgun.views.common import BaseLoggedInView, SearchableViewMixinPF4
+from airgun.widgets import SearchInput
 
 
 class FlatpakRemotesView(BaseLoggedInView, SearchableViewMixinPF4):
@@ -18,10 +24,12 @@ class FlatpakRemotesView(BaseLoggedInView, SearchableViewMixinPF4):
     table_loading = Text("//h5[normalize-space(.)='Loading']")
     no_results = Text("//h5[normalize-space(.)='No Results']")
 
+    create_new_btn = PF5Button('Create new')
+
     table = PF5OUIATable(
         component_id='flatpak-remotes-table',
         column_widgets={
-            'Name': Text('./a'),
+            'Name': Text('./a[contains(@href, "flatpak_remotes")]'),
             'URL': Text('./a'),
             2: PF5Menu(locator='.//div[contains(@class, "pf-v5-c-menu")]'),
         },
@@ -30,7 +38,103 @@ class FlatpakRemotesView(BaseLoggedInView, SearchableViewMixinPF4):
 
     @property
     def is_displayed(self):
-        return (
-            self.browser.wait_for_element(self.table_loading, exception=False) is None
-            and self.browser.wait_for_element(self.table, exception=False) is not None
-        )
+        return self.browser.wait_for_element(self.create_new_btn, exception=False) is not None
+
+
+class FlatpakRemoteDetailsView(BaseLoggedInView, SearchableViewMixinPF4):
+    """View for the Flatpak Remote details page"""
+
+    title = PF5OUIATitle('flatpak-remote-title')
+    url = PF5OUIAText('url-text-value')
+    subtitle = PF5OUIATitle('flatpak-remote-subtitle')
+    description = PF5OUIAText('flatpak-remote-description')
+
+    table = PF5OUIATable(
+        component_id='remote-repos-table',
+        column_widgets={
+            'Name': Text('./a'),
+            'ID': Text('./a'),
+            'Application name': Text('./a'),
+            'Last mirrored': Text('./a'),
+            'Mirror': PF5Button('Mirror'),
+        },
+    )
+    pagination = Pagination("//div[@class = 'pf-v5-c-pagination pf-m-bottom tfm-pagination']")
+
+    @property
+    def is_displayed(self):
+        return self.browser.wait_for_element(self.title, exception=False) is not None
+
+
+class CreateFlatpakRemoteModal(PF5Modal):
+    """View for the Create Flatpak Remote modal"""
+
+    ROOT = './/div[@data-ouia-component-id="create-flatpak-modal"]'
+
+    title = Text("//span[normalize-space(.)='Create Flatpak Remote']")
+
+    name = PF5OUIATextInput('input_name')
+    url = PF5OUIATextInput('input_url')
+    username = PF5OUIATextInput('input_username')
+    password = PF5OUIATextInput('input_password')
+
+    create_btn = PF5Button('Create')
+    cancel_btn = PF5Button('Cancel')
+
+    @property
+    def is_displayed(self):
+        return self.browser.wait_for_element(self.title, exception=False) is not None
+
+
+class EditFlatpakRemoteModal(PF5Modal):
+    """View for the Edit Flatpak Remote modal"""
+
+    ROOT = './/div[@data-ouia-component-id="edit-flatpak-modal"]'
+
+    title = Text("//span[normalize-space(.)='Edit Flatpak Remote']")
+
+    name = PF5OUIATextInput('input_name')
+    url = PF5OUIATextInput('input_url')
+    username = PF5OUIATextInput('input_username')
+    password = PF5OUIATextInput('input_password')
+
+    update_btn = PF5Button('Update')
+    cancel_btn = PF5Button('Cancel')
+
+    @property
+    def is_displayed(self):
+        return self.browser.wait_for_element(self.title, exception=False) is not None
+
+
+class MirrorFlatpakRemoteModal(PF5Modal, SearchableViewMixinPF4):
+    """View for the Mirror Flatpak Remote modal"""
+
+    ROOT = './/div[@data-ouia-component-id="mirror-repo-modal"]'
+
+    title = Text("//span[normalize-space(.)='Mirror Repository']")
+
+    searchbar = SearchInput(
+        locator='.//input[contains(@class, "pf-v5-c-text-input-group__text-input")]'
+    )
+
+    mirror_btn = PF5Button('Mirror')
+    cancel_btn = PF5Button('Cancel')
+
+    @property
+    def is_displayed(self):
+        return self.browser.wait_for_element(self.title, exception=False) is not None
+
+
+class FlatpakRemoteDeleteModal(PF5Modal):
+    """Confirmation dialog for deleting Flatpak Remote"""
+
+    ROOT = './/div[@data-ouia-component-id="flatpak-delete-modal"]'
+
+    title = Text("//span[normalize-space(.)='Delete Flatpak remote?']")
+
+    delete_btn = PF5Button('Delete')
+    cancel_btn = PF5Button('Cancel')
+
+    @property
+    def is_displayed(self):
+        return self.browser.wait_for_element(self.title, exception=False) is not None
