Merge pull request #743 from mozilla/anca/add_at-region

Anca/ L10n  - Add AT region

Author: soncuteanca

l10n_CM/region/AT.json

@@ -0,0 +1,9 @@
+{
+  "region": "AT",
+  "sites": [
+    "demo",
+    "torfs"
+  ],
+  "tests": [
+  ]
+}

l10n_CM/run_l10n.py

@@ -15,7 +15,7 @@
 current_dir = os.path.dirname(__file__)
 valid_flags = {"--run-headless", "-n", "--reruns", "--fx-executable", "--ci"}
 flag_with_parameter = {"-n", "--reruns"}
-valid_region = {"US", "CA", "DE", "FR", "IT", "GB", "PL", "ES", "BE"}
+valid_region = {"US", "CA", "DE", "FR", "IT", "GB", "PL", "ES", "BE", "AT"}
 valid_sites = {
     "aldoshoes",
     "amazon",

modules/browser_object_navigation.py

@@ -269,10 +269,11 @@ def open_tracker_panel(self) -> BasePage:
 
     @BasePage.context_chrome
     def click_file_download_warning_panel(self) -> BasePage:
-        """ exit file download warning panel if present """
+        """exit file download warning panel if present"""
         self.element_clickable("file-download-warning-button")
         self.click_on("file-download-warning-button")
         return self
+
     def wait_for_item_to_download(self, filename: str) -> BasePage:
         """
         Check the downloads tool in the toolbar to wait for a given file to download

modules/testrail_integration.py

@@ -202,7 +202,9 @@ def reportable(platform_to_test=None):
                             run_region in distributed_mappings[site]
                             and platform in run_platform
                         ):
-                            logging.warning(f"Already reported: {site} {run_region},{run_platform}")
+                            logging.warning(
+                                f"Already reported: {site} {run_region},{run_platform}"
+                            )
                             covered_mappings += 1
         logging.warning(
             f"Potentially matching run found for {platform}, may be reportable. (Found {covered_mappings} site/region mappings reported, expected {expected_mappings}.)"

modules/util.py

@@ -110,6 +110,7 @@ def __init__(self):
             "Spain": "España",
             "Poland": "Polska",
             "Belgium": "België",
+            "Austria": "Österreich",
         }
 
         self.fake = None
@@ -567,8 +568,23 @@ def normalize_regional_phone_numbers(self, phone: str, region: str) -> str:
             "PL": "48",
             "ES": "34",
             "BE": "32",
+            "AT": "43",
         }
 
+        # If phone is already normalized, return as it is
+        expected_country_code = country_codes.get(region)
+        if (
+            expected_country_code
+            and phone.isdigit()
+            and phone.startswith(expected_country_code)
+            and len(phone) >= len(expected_country_code) + 6
+        ):
+            return phone
+
+        # Fix Austrian duplicated country code before processing
+        if region == "AT" and phone.startswith("+4343"):
+            phone = "+43" + phone[5:]  # Remove duplicated 43
+
         # Sub out anything that matches this regex statement with an empty string to get rid of extensions in generated phone numbers
         phone = re.sub(r"\s*(?:x|ext)\s*\d*$", "", phone, flags=re.IGNORECASE)
         # Sub out anything that is not a digit with the empty string to ensure the phone number is formatted with no spaces or special characters