merge main

Author: Hani Yacoub

.github/workflows/check-devedition.yml

@@ -2,7 +2,7 @@ name: Check new devedition version
 
 on:
   schedule:
-    - cron: "38 * * * *"
+    - cron: "38 */12 * * *"
 env:
   LATEST: ""
 permissions:

.github/workflows/smoke.yml

@@ -23,6 +23,7 @@ env:
 
 jobs:
   Smoke-Windows:
+    if: ${{ inputs.job_to_run == 'Smoke-Windows' || github.event_name == 'pull_request' }}
     runs-on: windows-latest
     steps:
       - name: Create app token
@@ -116,6 +117,7 @@ jobs:
           name: artifacts-win
           path: artifacts-win
   Smoke-MacOS:
+    if: ${{ inputs.job_to_run == 'Smoke-MacOS' || github.event_name == 'pull_request' }}
     runs-on: macos-latest
     steps:
       - name: Create app token
@@ -151,11 +153,11 @@ jobs:
           rm ./pyproject.toml;
           mv ./ci_pyproject.toml ./pyproject.toml;
           pipenv install;
-          bash ./collect_executables.sh;
+          echo app_name=$(bash ./collect_executables.sh | xargs -0 ./parse_executables.sh) >> "$GITHUB_OUTPUT"
       - name: Run Smoke Tests in MacOS
         if: steps.setup.conclusion == 'success'
         env:
-          FX_EXECUTABLE: /Volumes/Firefox/Firefox.app/Contents/MacOS/firefox
+          FX_EXECUTABLE: /Volumes/${{ steps.setup.outputs.app_name }}/${{ steps.setup.outputs.app_name }}.app/Contents/MacOS/firefox
         run: |
           /Volumes/Firefox/Firefox.app/Contents/MacOS/firefox --version
           pipenv run python choose_ci_set.py

SELECTOR_INFO.md

@@ -1389,6 +1389,13 @@ Location: Appears as part of the dropdown under the autofill panel, within any e
 Path to .json: modules/data/autofill_popup.components.json
 ```
 ```
+Selector Name: select-form-option-by-index
+Selector Data: ".autocomplete-richlistbox .autocomplete-richlistitem:nth:child({index})"
+Description: Select n individual entry within the autofill dropdown, using the index of the child.
+Location: Appears as part of the dropdown under the autofill panel, within any eligible form field when suggestions are available.
+Path to .json: modules/data/autofill_popup.components.json
+```
+```
 Selector Name: clear-form-option
 Selector Data: ".autocomplete-richlistbox .autocomplete-richlistitem[ac-value='Clear Autofill Form']"
 Description: The "Clear Autofill Form" option in the dropdown that appears when interacting with an autofill-enabled input field
@@ -2976,7 +2983,7 @@ Path to .json: modules/data/navigation.components.json
 ```
 ```
 Selector Name: search-mode-switcher-option
-Selector Data: toolbarbutton[label*='{title}']
+Selector Data: menuitem[label*='{title}']
 Description: Option by label in search mode list
 Location: Search mode of awesomebar
 Path to .json: modules/data/navigation.components.json

collect_executables.py

@@ -103,6 +103,8 @@ def get_gd_platform():
         language = "en-US"
 
     if channel == "-devedition":
+        # Devedition has special requirements as it's testing a release
+
         this_devedition = BACKSTOP
         fx_download_dir_url = (
             "https://archive.mozilla.org/pub/devedition/releases/135.0b5/"
@@ -121,62 +123,60 @@ def get_gd_platform():
             fx_download_dir_url = next_candidate
 
         devedition_version = fx_download_dir_url.split("/")[-2]
-        if NUMBER_ONLY:
-            print(devedition_version)
-        else:
-            print(
-                f"{fx_download_dir_url}{get_fx_platform()}/{language}/Firefox%20{devedition_version}.{get_fx_executable_extension()}"
-            )
-        exit()
-
-    candidate_exists = True
-    this_beta = BACKSTOP
-    while candidate_exists:
-        (major, minor_beta) = this_beta.split(".")
-        (minor, beta) = minor_beta.split("b")
-        major = int(major)
-        minor = int(minor)
-        beta = int(beta)
-
-        next_major = f"{major + 1}.0b1"
-        fx_download_dir_url = f"https://archive.mozilla.org/pub/firefox/candidates/{next_major}-candidates/build1/"
-        rs = requests.get(fx_download_dir_url)
-        if rs.status_code < 300:
-            latest_beta_ver = next_major
-            this_beta = next_major
-            continue
-
-        next_minor = f"{major}.{minor + 1}b1"
-        fx_download_dir_url = f"https://archive.mozilla.org/pub/firefox/candidates/{next_minor}-candidates/build1/"
-        rs = requests.get(fx_download_dir_url)
-        if rs.status_code < 300:
-            latest_beta_ver = next_minor
-            this_beta = next_minor
-            continue
+        fx_download_dir_url = f"{fx_download_dir_url}{get_fx_platform()}/{language}/"
 
-        next_beta = f"{major}.{minor}b{beta + 1}"
-        fx_download_dir_url = f"https://archive.mozilla.org/pub/firefox/candidates/{next_beta}-candidates/build1/"
-        rs = requests.get(fx_download_dir_url)
-        if rs.status_code < 300:
-            latest_beta_ver = next_beta
-            this_beta = next_beta
-            continue
-
-        candidate_exists = False
+    else:
+        # Anything but devedition
 
-    status = 200
-    build = 0
-    while status < 400 and build < 20:
-        build += 1
-        fx_download_dir_url = f"https://archive.mozilla.org/pub/firefox/candidates/{latest_beta_ver}-candidates/build{build}/"
+        candidate_exists = True
+        this_beta = BACKSTOP
+        while candidate_exists:
+            (major, minor_beta) = this_beta.split(".")
+            (minor, beta) = minor_beta.split("b")
+            major = int(major)
+            minor = int(minor)
+            beta = int(beta)
+
+            next_major = f"{major + 1}.0b1"
+            fx_download_dir_url = f"https://archive.mozilla.org/pub/firefox/candidates/{next_major}-candidates/build1/"
+            rs = requests.get(fx_download_dir_url)
+            if rs.status_code < 300:
+                latest_beta_ver = next_major
+                this_beta = next_major
+                continue
+
+            next_minor = f"{major}.{minor + 1}b1"
+            fx_download_dir_url = f"https://archive.mozilla.org/pub/firefox/candidates/{next_minor}-candidates/build1/"
+            rs = requests.get(fx_download_dir_url)
+            if rs.status_code < 300:
+                latest_beta_ver = next_minor
+                this_beta = next_minor
+                continue
+
+            next_beta = f"{major}.{minor}b{beta + 1}"
+            fx_download_dir_url = f"https://archive.mozilla.org/pub/firefox/candidates/{next_beta}-candidates/build1/"
+            rs = requests.get(fx_download_dir_url)
+            if rs.status_code < 300:
+                latest_beta_ver = next_beta
+                this_beta = next_beta
+                continue
+
+            candidate_exists = False
+
+        status = 200
+        build = 0
+        while status < 400 and build < 20:
+            build += 1
+            fx_download_dir_url = f"https://archive.mozilla.org/pub/firefox/candidates/{latest_beta_ver}-candidates/build{build}/"
+
+            # Fetch the page
+            response = requests.get(fx_download_dir_url)
+            status = response.status_code
 
-        # Fetch the page
-        response = requests.get(fx_download_dir_url)
-        status = response.status_code
+        # Correct build is the last one that didn't 404
+        build -= 1
+        fx_download_dir_url = f"https://archive.mozilla.org/pub/firefox/candidates/{latest_beta_ver}-candidates/build{build}/{get_fx_platform()}/{language}/"
 
-    # Correct build is the last one that didn't 404
-    build -= 1
-    fx_download_dir_url = f"https://archive.mozilla.org/pub/firefox/candidates/{latest_beta_ver}-candidates/build{build}/{get_fx_platform()}/{language}/"
     response = requests.get(fx_download_dir_url)
     status = response.status_code
     response_text = None
@@ -206,8 +206,11 @@ def get_gd_platform():
 
     fx_download_executable_url = rf"{fx_download_dir_url}{executable_name}"
     if NUMBER_ONLY:
-        number_cand = fx_download_dir_url.split("/")[6]
-        number = number_cand.split("-")[0]
-        print(f"{number}-build{build}")
+        if channel == "-devedition":
+            print(devedition_version)
+        else:
+            number_cand = fx_download_dir_url.split("/")[6]
+            number = number_cand.split("-")[0]
+            print(f"{number}-build{build}")
     else:
         print(fx_download_executable_url)

collect_executables.sh

@@ -7,6 +7,7 @@ then
     curl -o Firefox.dmg -L "$(pipenv run python collect_executables.py)"
     hdiutil attach Firefox.dmg
 else
+    pipenv run python collect_executables.py
     curl -o firefox.tar.xz -L "$(pipenv run python collect_executables.py)"
     tar xf firefox.tar.xz
 fi

l10n_CM/Unified/test_demo_ad_autofill_address_fields.py

@@ -0,0 +1,45 @@
+import pytest
+from selenium.webdriver import Firefox
+
+from modules.browser_object_autofill_popup import AutofillPopup
+from modules.page_object_autofill import AddressFill
+from modules.util import Utilities
+
+
+@pytest.fixture()
+def test_case():
+    return "2888563"
+
+
+def test_demo_ad_autofill_address_fields(
+    driver: Firefox,
+    region: str,
+    address_autofill: AddressFill,
+    util: Utilities,
+    autofill_popup: AutofillPopup,
+):
+    """
+    C2888563 - Verify Autofill functionality when selecting an entry from the dropdown for address fields
+    """
+    # Create fake data and fill it in
+    address_autofill.open()
+    address_autofill_data = util.fake_autofill_data(region)
+    address_autofill.save_information_basic(address_autofill_data)
+
+    # Click the "Save" button
+    autofill_popup.click_doorhanger_button("save")
+
+    # List of field labels to be autofilled and verified
+    fields_to_test = [
+        "street-address",
+        "address-level2",
+        "address-level1",  # This will be skipped for DE/FR
+        "postal-code",
+        "country",
+    ]
+
+    # Loop through each field and perform the autofill test
+    for field in fields_to_test:
+        address_autofill.autofill_and_verify(
+            autofill_popup, field, address_autofill_data, util
+        )

l10n_CM/Unified/test_demo_ad_autofill_name_org.py

@@ -1,7 +1,6 @@
 import pytest
 from selenium.webdriver import Firefox
 
-from l10n_CM.Unified.conftest import autofill_popup
 from modules.browser_object_autofill_popup import AutofillPopup
 from modules.page_object_autofill import AddressFill
 from modules.util import Utilities

l10n_CM/Unified/test_demo_ad_clear_name_org.py

@@ -0,0 +1,40 @@
+import pytest
+from selenium.webdriver import Firefox
+
+from modules.browser_object_autofill_popup import AutofillPopup
+from modules.page_object_autofill import AddressFill
+from modules.util import Utilities
+
+
+@pytest.fixture()
+def test_case():
+    return "2888560"
+
+
+def test_demo_ad_clear_name_org(
+        driver: Firefox,
+        region: str,
+        address_autofill: AddressFill,
+        util: Utilities,
+        autofill_popup: AutofillPopup
+):
+    """
+    C2888560 - Verify clear functionality after selecting an entry from name/org fields
+    """
+    # Create fake data and fill it in
+    address_autofill.open()
+    address_autofill_data = util.fake_autofill_data(region)
+    address_autofill.save_information_basic(address_autofill_data)
+
+    # Click the "Save" button
+    autofill_popup.click_doorhanger_button("save")
+
+    # List of field labels to be autofilled and verified
+    fields_to_test = [
+        "name",
+        "organization"
+    ]
+
+    # Loop through each field and perform the autofill test
+    for field in fields_to_test:
+        address_autofill.clear_and_verify(autofill_popup, field, address_autofill_data)

l10n_CM/Unified/test_demo_ad_email_phone_captured_in_doorhanger_and_stored.py

@@ -38,12 +38,17 @@ def test_demo_ad_email_phone_captured_in_doorhanger_and_stored(
 
     # containing phone field
     expected_phone = address_autofill_data.telephone
-    actual_phone = autofill_popup.get_cc_doorhanger_data("address-doorhanger-phone")
-    normalize_expected = util.normalize_regional_phone_numbers(expected_phone, region)
-    normalized_actual = util.normalize_regional_phone_numbers(actual_phone, region)
-    assert normalized_actual == normalize_expected, (
-        f"Phone number mismatch for {region} | Expected: {normalize_expected}, Got: {normalized_actual}"
-    )
+    # Skip verification if phone number isn't provided
+    if expected_phone:
+        with driver.context(driver.CONTEXT_CHROME):
+            actual_phone = autofill_popup.get_element("address-doorhanger-phone").text
+        normalize_expected = util.normalize_regional_phone_numbers(
+            expected_phone, region
+        )
+        normalized_actual = util.normalize_regional_phone_numbers(actual_phone, region)
+        assert normalized_actual == normalize_expected, (
+            f"Phone number mismatch for {region} | Expected: {normalize_expected}, Got: {normalized_actual}"
+        )
 
     # Click the "Save" button
     autofill_popup.click_doorhanger_button("save")

l10n_CM/Unified/test_demo_cc_add_new_credit_card.py

@@ -3,7 +3,6 @@
 import pytest
 from selenium.webdriver import Firefox
 
-from l10n_CM.Unified.conftest import region
 from modules.page_object import AboutPrefs
 from modules.util import Utilities