Debugging

Author: soncuteanca

modules/browser_object_navigation.py

@@ -250,13 +250,24 @@ def set_always_open_similar_files(self) -> BasePage:
         """
         From the downloads panel, right-click the most recent download and set 'Always Open Similar Files'.
         """
+        print("\n[DEBUG] Starting set_always_open_similar_files")
+
         downloads_button = self.get_download_button()
+        print("[DEBUG] Got downloads button")
         downloads_button.click()
+        print("[DEBUG] Clicked downloads button")
 
-        # 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")
+        print(
+            f"[DEBUG] Got download item: {download_item.get_attribute('outerHTML')[:200]}"
+        )
+
         self.context_click(download_item)
+        print("[DEBUG] Context clicked download item")
+
         self.context_menu.get_element("context-menu-always-open-similar-files").click()
+        print("[DEBUG] Clicked 'Always Open Similar Files'")
+
         return self
 
     @BasePage.context_chrome

modules/page_object_prefs.py

@@ -645,17 +645,41 @@ def get_app_name_for_mime_type(self, mime_type: str) -> str:
         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])
+        print(f"\n[DEBUG] Looking for MIME type: {mime_type}")
 
-        # Find the description element that contains application info
-        action_description = self.get_element(
-            "mime-type-item-description", parent_element=mime_type_item
-        )
+        try:
+            mime_type_item = self.get_element("mime-type-item", labels=[mime_type])
+            print(
+                f"[DEBUG] Found MIME type item: {mime_type_item.get_attribute('outerHTML')[:300]}"
+            )
+        except Exception as e:
+            print(f"[DEBUG] Failed to find MIME type item: {e}")
+            raise
 
-        # 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"]
+        try:
+            action_description = self.get_element(
+                "mime-type-item-description", parent_element=mime_type_item
+            )
+            print(
+                f"[DEBUG] Found action description: {action_description.get_attribute('outerHTML')[:300]}"
+            )
+        except Exception as e:
+            print(f"[DEBUG] Failed to find action description: {e}")
+            raise
+
+        try:
+            data_attr = action_description.get_attribute("data-l10n-args")
+            print(f"[DEBUG] data-l10n-args: {data_attr}")
+
+            mime_type_data = json.loads(data_attr)
+            print(f"[DEBUG] Parsed JSON: {mime_type_data}")
+
+            app_name = mime_type_data["app-name"]
+            print(f"[DEBUG] Extracted app-name: {app_name}")
+            return app_name
+        except Exception as e:
+            print(f"[DEBUG] Failed to parse app name: {e}")
+            raise
 
 
 class AboutAddons(BasePage):

tests/downloads/test_add_mime_type_doc.py

@@ -4,7 +4,6 @@
 from selenium.webdriver import Firefox
 
 from modules.browser_object import Navigation
-from modules.browser_object_tabbar import TabBar
 from modules.page_object import AboutPrefs, GenericPage
 
 
@@ -13,9 +12,7 @@ def test_case():
     return "1756748"
 
 
-# Constants
 DOC_LINK = "https://sapphire-hendrika-5.tiiny.site/"
-# WIN_GHA = environ.get("GITHUB_ACTIONS") == "true" and sys.platform.startswith("win")
 
 
 @pytest.fixture()
@@ -24,44 +21,68 @@ def delete_files_regex_string():
 
 
 def expected_app_name(sys_platform: str, opt_ci: bool) -> str:
-    """
-    Decide which default application should be used to open .doc files, based on OS and CI environment
-    """
     if sys_platform == "Darwin":
         return "TextEdit" if opt_ci else "Pages"
-    # Linux/Windows
     return "LibreOffice Writer"
 
 
 @pytest.mark.noxvfb
-# @pytest.mark.skipif(WIN_GHA, reason="Test unstable in Windows Github Actions")
 def test_mime_type_doc(driver: Firefox, sys_platform: str, opt_ci: bool, delete_files):
     """
     C1756748 - Verify that downloading a .doc file adds a new MIME type entry
     and the correct default application is assigned.
     """
-    # Instantiate objects
+    print(f"\n{'=' * 60}")
+    print(f"Platform: {sys_platform}, CI: {opt_ci}")
+    print(f"{'=' * 60}")
+
+    if sys_platform == "Windows":
+        print("\n--- Checking for LibreOffice ---")
+        result = subprocess.run(
+            ["where", "soffice.exe"], capture_output=True, text=True
+        )
+        print(f"LibreOffice installed: {result.returncode == 0}")
+        if result.returncode == 0:
+            print(f"Location: {result.stdout.strip()}")
+
     page = GenericPage(driver, url=DOC_LINK)
     nav = Navigation(driver)
     about_prefs = AboutPrefs(driver, category="general")
-    tabs = TabBar(driver)
 
-    # Open the test page with the .doc download link
+    print("\n--- Opening page and downloading ---")
     page.open()
     page.click_on("sample-doc-download")
 
-    # 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
-    tabs.new_tab_by_button()
-    tabs.switch_to_new_tab()
+    print("\n--- Opening about:preferences ---")
     about_prefs.open()
+
+    print("\n--- Checking if MIME type exists ---")
+    try:
+        exists = about_prefs.element_exists(
+            "mime-type-item", labels=["application/msword"]
+        )
+        print(f"MIME type exists: {exists}")
+    except Exception as e:
+        print(f"Error checking existence: {e}")
+        exists = False
+
+    if not exists:
+        print("\n!!! MIME TYPE NOT FOUND !!!")
+        driver.save_screenshot("artifacts/debug_no_mime.png")
+        pytest.fail("MIME type entry was not created")
+
+    print("\n--- Getting app name ---")
     app_name = about_prefs.get_app_name_for_mime_type("application/msword")
-    assert app_name == expected_app_name(sys_platform, opt_ci)
+    expected = expected_app_name(sys_platform, opt_ci)
+
+    print(f"\nResult: '{app_name}' vs Expected: '{expected}'")
+    assert app_name == expected, f"Mismatch: got '{app_name}', expected '{expected}'"
+    print("✓ TEST PASSED")
 
-    # Kill LibreOffice before cleanup to prevent file lock
     if sys_platform == "Windows":
+        print("\n--- Cleaning up ---")
         subprocess.run(
             ["taskkill", "/F", "/IM", "soffice.bin"], capture_output=True, check=False
         )