Merge main

Author: Hani Yacoub

SELECTOR_INFO.md

@@ -1151,6 +1151,13 @@ Description: The dropdown menu for default zoom selection
 Location: about:preferences - Zoom settings
 Path to .json: modules/data/about_prefs.components.json
 ```
+```
+Selector Name: unknown-content-type-dialog
+Selector Data: unknownContentTypeWindo
+Description: The unknown content type dialog
+Location: about:preferences#general Applications subsection
+Path to .json: modules/data/about_prefs.components.json
+```
 #### about_profiles
 ```
 Selector Name: profile-container
@@ -1788,6 +1795,13 @@ Description: Open all bookmarks from the context menu option from a Toolbar book
 Location: Context menu - Toolbar
 Path to .json: modules/data/context_menu.components.json
 ```
+```
+Selector Name: context-menu-move-tab-to-start
+Selector Data: menuitem[data-l10n-id='move-to-start']
+Description: Context menu option to move a tab to the start of the tab bar.
+Location: Context menu - Tab
+Path to .json: modules/data/context_menu.components.json
+```
 #### credit_card_fill
 ```
 Selector Name: form-field
@@ -2797,6 +2811,27 @@ Location: Download panel
 Path to .json: modules/data/navigation.components.json
 ```
 ```
+Selector name: download-target-element
+Selector Data: downloadTarget
+Description: Downloaded item in the download panel
+Location: Download panel
+Path to .json: modules/data/navigation.components.json
+```
+```
+Selector name: download-progress-element
+Selector Data: downloadProgress
+Description: Download progress bar
+Location: Download panel
+Path to .json: modules/data/navigation.components.json
+```
+```
+Selector name: ddownload-details-element
+Selector Data: downloadDetailsNormal
+Description: Download details element
+Location: Download panel
+Path to .json: modules/data/navigation.components.json
+```
+```
 Selector name: remove-bookmark-button
 Selector Data: editBookmarkPanelRemoveButton
 Description: Remove bookmark button

modules/browser_object_navigation.py

@@ -1,7 +1,8 @@
 import logging
+import re
 from typing import Literal
 
-from selenium.common.exceptions import TimeoutException
+from selenium.common.exceptions import StaleElementReferenceException, TimeoutException
 from selenium.webdriver import ActionChains, Firefox
 from selenium.webdriver.common.by import By
 from selenium.webdriver.common.keys import Keys
@@ -246,13 +247,25 @@ def click_download_button(self) -> BasePage:
         return self
 
     @BasePage.context_chrome
-    def wait_for_download_animation_finish(
-        self, downloads_button: WebElement
-    ) -> BasePage:
+    def set_always_open_similar_files(self) -> BasePage:
         """
-        Waits for the download button to finish playing the animation for downloading to local computer
+        From the downloads panel, right-click the most recent download and set 'Always Open Similar Files'.
         """
+        downloads_button = self.get_download_button()
+        downloads_button.click()
+
+        # Locate the latest downloaded file in the panel, open context menu and choose 'Always Open Similar Files'
+        download_item = self.get_element("download-panel-item")
+        self.context_click(download_item)
+        self.context_menu.get_element("context-menu-always-open-similar-files").click()
+
+        return self
+
+    @BasePage.context_chrome
+    def wait_for_download_animation_finish(self) -> BasePage:
+        """Waits for the download button to finish playing the animation"""
         try:
+            downloads_button = self.get_download_button()
             self.wait.until(
                 lambda _: downloads_button.get_attribute("notification") == "finish"
             )
@@ -298,6 +311,34 @@ def wait_for_item_to_download(self, filename: str) -> BasePage:
             self.driver.implicitly_wait(original_timeout)
         return self
 
+    @BasePage.context_chrome
+    def verify_download_name(self, expected_pattern: str) -> BasePage:
+        """
+        Verify download name matches expected pattern.
+        Argument:
+            expected_pattern: Regex pattern to match against download name
+        """
+        download_name = self.get_element("download-target-element")
+        download_value = download_name.get_attribute("value")
+        assert re.match(expected_pattern, download_value), (
+            f"The download name is incorrect: {download_value}"
+        )
+        return self
+
+    @BasePage.context_chrome
+    def wait_for_download_completion(self) -> BasePage:
+        """Wait until the most recent download reaches 100% progress."""
+
+        def _download_complete(_):
+            try:
+                element = self.get_element("download-progress-element")
+                return element.get_attribute("value") == "100"
+            except StaleElementReferenceException:
+                return False
+
+        self.wait.until(_download_complete)
+        return self
+
     @BasePage.context_chrome
     def refresh_page(self) -> BasePage:
         """

modules/browser_object_tabbar.py

@@ -316,3 +316,13 @@ def open_multiple_tabs_with_pages(self, pages: list) -> "TabBar":
             else:
                 page.open()
         return self
+
+    @BasePage.context_chrome
+    def verify_tab_focus_cycle(self, num_tabs: int):
+        """Go through all the tabs and ensure the focus changes correctly."""
+        for i in range(1, num_tabs + 2):
+            target_tab = self.get_tab(i)
+            self.click_on(target_tab)
+            self.custom_wait(timeout=3).until(
+                lambda d: target_tab.get_attribute("visuallyselected") == ""
+            )

modules/data/about_prefs.components.json

@@ -541,5 +541,13 @@
     "selectorData": "menuitem[data-l10n-id='preferences-default-zoom-value'][value='{.*}']",
     "strategy": "css",
     "groups": []
-  }
+  },
+
+    "unknown-content-type-dialog": {
+        "selectorData": "unknownContentTypeWindow",
+        "strategy": "id",
+        "groups": [
+            "doNotCache"
+        ]
+    }
 }

modules/data/context_menu.components.json

@@ -109,6 +109,12 @@
         "groups": []
     },
 
+    "context-menu-move-tab-to-start": {
+        "selectorData": "menuitem[data-l10n-id='move-to-start']",
+        "strategy": "css",
+        "groups": []
+    },
+
     "context-menu-open-link-in-tab": {
         "selectorData": "context-openlinkintab",
         "strategy": "id",

modules/data/navigation.components.json

@@ -405,6 +405,24 @@
         "groups": []
     },
 
+    "download-target-element": {
+        "selectorData": "downloadTarget",
+        "strategy": "class",
+        "groups": []
+    },
+
+    "download-progress-element": {
+        "selectorData": "downloadProgress",
+        "strategy": "class",
+        "groups": []
+    },
+
+    "download-details-element": {
+        "selectorData": "downloadDetailsNormal",
+        "strategy": "class",
+        "groups": []
+    },
+
     "bookmark-in-bar": {
         "selectorData": "toolbarbutton.bookmark-item",
         "strategy": "css",

modules/page_object_prefs.py

@@ -1,4 +1,5 @@
 import datetime
+import json
 import re
 from time import sleep
 from typing import List, Literal
@@ -638,6 +639,50 @@ def click_popup_panel_button(self, field: str) -> BasePage:
             self.get_element("panel-popup-button", labels=[field]).click()
         return self
 
+    @BasePage.context_content
+    def get_app_name_for_mime_type(self, mime_type: str) -> str:
+        """
+        Return the application name associated with a given MIME type in about:preferences.
+        Argument:
+            mime_type: the MIME type to look up (e.g., "application/msword").
+        """
+        # Locate the row for the given MIME type
+        mime_type_item = self.get_element("mime-type-item", labels=[mime_type])
+
+        # Find the description element that contains application info
+        action_description = self.get_element(
+            "mime-type-item-description", parent_element=mime_type_item
+        )
+
+        # Parse the JSON data-l10n-args attribute and extract app name
+        mime_type_data = json.loads(action_description.get_attribute("data-l10n-args"))
+        return mime_type_data["app-name"]
+
+    def set_pdf_handling_to_always_ask(self) -> BasePage:
+        """
+        Set PDF content type handling to "Always ask" in Applications settings.
+        """
+        self.click_on("pdf-content-type")
+        self.click_on("pdf-actions-menu")
+        menu = self.get_element("pdf-actions-menu")
+        menu.send_keys(Keys.DOWN)
+        menu.send_keys(Keys.ENTER)
+        return self
+
+    @BasePage.context_chrome
+    def handle_unknown_content_dialog(self) -> BasePage:
+        """
+        Wait for the unknown content type dialog to appear and close it with Escape.
+        """
+        self.wait.until(lambda _: len(self.driver.window_handles) > 1)
+        self.driver.switch_to.window(self.driver.window_handles[-1])
+        self.wait.until(lambda _: self.get_element("unknown-content-type-dialog"))
+
+        # Close the dialog with Escape
+        dialog = self.get_element("unknown-content-type-dialog")
+        dialog.send_keys(Keys.ESCAPE)
+        return self
+
 
 class AboutAddons(BasePage):
     """

tests/downloads/test_add_mime_type_doc.py

@@ -1,11 +1,10 @@
-import json
 import sys
 from os import environ
 
 import pytest
 from selenium.webdriver import Firefox
 
-from modules.browser_object import ContextMenu, Navigation
+from modules.browser_object import Navigation
 from modules.page_object import AboutPrefs, GenericPage
 
 
@@ -15,7 +14,6 @@ def test_case():
 
 
 DOC_LINK = "https://sapphire-hendrika-5.tiiny.site/"
-
 WIN_GHA = environ.get("GITHUB_ACTIONS") == "true" and sys.platform.startswith("win")
 
 
@@ -24,41 +22,35 @@ def delete_files_regex_string():
     return r"sample.*\.doc"
 
 
-@pytest.mark.skipif(WIN_GHA, reason="Test unstable in Windows Github Actions")
+def expected_app_name(sys_platform: str, opt_ci: bool) -> str:
+    if sys_platform == "Darwin":
+        return "TextEdit" if opt_ci else "Pages"
+    return "LibreOffice Writer"
+
+
+@pytest.mark.skipif(
+    WIN_GHA, reason="Most runners don’t have a .doc handler registered on Windows"
+)
 @pytest.mark.noxvfb
 def test_mime_type_doc(driver: Firefox, sys_platform: str, opt_ci: bool, delete_files):
     """
-    C1756748: Verify the user can add the .doc type
+    C1756748 - Verify that downloading a .doc file adds a new MIME type entry
+    and the correct default application is assigned.
     """
-    doc_page = GenericPage(driver, url=DOC_LINK).open()
+
+    # Instantiate objects
+    page = GenericPage(driver, url=DOC_LINK)
     nav = Navigation(driver)
-    context_menu = ContextMenu(driver)
     about_prefs = AboutPrefs(driver, category="general")
-    doc_page.get_element("sample-doc-download").click()
 
-    downloads_button = nav.get_download_button()
+    # Open the test page with the .doc download link
+    page.open()
+    page.click_on("sample-doc-download")
 
-    with driver.context(driver.CONTEXT_CHROME):
-        downloads_button.click()
-        download_item = nav.get_element("download-panel-item")
-        nav.context_click(download_item)
-        context_menu.get_element("context-menu-always-open-similar-files").click()
+    # Download the file and set 'Always Open Similar Files'
+    nav.set_always_open_similar_files()
 
+    # Verify the MIME type entry exists and default app matches expectation
     about_prefs.open()
-    about_prefs.element_exists("mime-type-item", labels=["application/msword"])
-
-    mime_type_item = about_prefs.get_element(
-        "mime-type-item", labels=["application/msword"]
-    )
-    action_description_item = about_prefs.get_element(
-        "mime-type-item-description", parent_element=mime_type_item
-    )
-
-    mime_type_data = json.loads(action_description_item.get_attribute("data-l10n-args"))
-    if sys_platform == "Darwin":
-        if opt_ci:
-            assert mime_type_data["app-name"] == "TextEdit"
-        else:
-            assert mime_type_data["app-name"] == "Pages"
-    else:
-        assert mime_type_data["app-name"] == "LibreOffice Writer"
+    app_name = about_prefs.get_app_name_for_mime_type("application/msword")
+    assert app_name == expected_app_name(sys_platform, opt_ci)

tests/downloads/test_add_zip_type.py

@@ -4,7 +4,6 @@
 import pytest
 from selenium.webdriver import Firefox
 
-from modules.browser_object_context_menu import ContextMenu
 from modules.browser_object_navigation import Navigation
 from modules.page_object_generics import GenericPage
 from modules.page_object_prefs import AboutPrefs
@@ -27,7 +26,8 @@ def delete_files_regex_string():
 def temp_selectors():
     return {
         "github-code-button": {
-            "selectorData": "/html/body/div[1]/div[4]/div/main/turbo-frame/div/div/div/div/div[1]/react-partial/div/div/div[2]/div[2]/button",
+            "selectorData": "/html/body/div[1]/div[4]/div/main/turbo-frame/div/div/div/div/div["
+            "1]/react-partial/div/div/div[2]/div[2]/button",
             "strategy": "xpath",
             "groups": [],
         },
@@ -50,28 +50,26 @@ def test_add_zip_type(
     """
     C1756743: Verify that the user can add the .zip mime type to Firefox
     """
-    # instantiate object
-    web_page = GenericPage(driver, url=ZIP_URL).open()
+    # Instantiate objects
+    web_page = GenericPage(driver, url=ZIP_URL)
     nav = Navigation(driver)
-    context_menu = ContextMenu(driver)
     about_prefs = AboutPrefs(driver, category="general")
 
     web_page.elements |= temp_selectors
 
     # Click on the available zip
+    web_page.open()
     web_page.click_on("github-code-button")
     web_page.click_on("github-download-button")
 
     # In the download panel right-click on the download and click "Always Open Similar Files"
-    with driver.context(driver.CONTEXT_CHROME):
-        nav.context_click(nav.get_element("download-panel-item"))
-        context_menu.get_element("context-menu-always-open-similar-files").click()
+    nav.set_always_open_similar_files()
 
     # Open about:preferences and check that zip mime type is present in the application list
     about_prefs.open()
-    about_prefs.element_exists("mime-type-item", labels=["application/zip"])
+    about_prefs.get_app_name_for_mime_type("application/zip")
 
-    # Remove the directory created as MacOS automatically unzips
+    # Remove the directory created as macOS automatically unzips
     if sys_platform == "Darwin":
         dir_created = os.path.join(home_folder, "Downloads", "api-guidelines-vNext")
         shutil.rmtree(dir_created)