Merge pull request #622 from mozilla/as/cm-lowes-us

Anca/ L10n [US] - Lowes (ad+cc)

Author: soncuteanca

l10n_CM/constants/lowes/US/lowes_ad.json

@@ -0,0 +1,19 @@
+{
+ "url": "http://127.0.0.1:8080/lowes_ad.html",
+ "field_mapping": {
+    "given_name":"4b1e90aa-9614-4c10-9a07-e1f5d0d0fa5e",
+    "family_name":"eda01517-1593-4907-bd89-e760ac1b3301",
+    "street_address":"24128044-0f39-4679-93f1-788031618aa1",
+    "address_level_2":"35740687-1404-4d7c-973e-4f4a8d49fd2c",
+    "postal_code":"68980560-b0a3-4c07-b4a9-5bbf800cfc5d"
+
+ },
+ "form_field": "*[data-moz-autofill-inspect-id='{name}']",
+ "fields": [
+    "4b1e90aa-9614-4c10-9a07-e1f5d0d0fa5e",
+    "eda01517-1593-4907-bd89-e760ac1b3301",
+    "24128044-0f39-4679-93f1-788031618aa1",
+    "35740687-1404-4d7c-973e-4f4a8d49fd2c",
+    "68980560-b0a3-4c07-b4a9-5bbf800cfc5d"
+ ]
+}

l10n_CM/constants/lowes/US/lowes_cc.json

@@ -0,0 +1,14 @@
+{
+  "url": "http://127.0.0.1:8080/lowes_cc.html",
+  "field_mapping": {
+    "card_number": "311621a3-d8f9-4dd3-bf9d-1eeb08f7f8a5",
+    "expiration_date": "4ef6a37d-b02f-4fa7-bfc1-0994180238c8",
+    "cvv": "78daaaff-d49c-4a2e-91a7-9cc214224f72"
+},
+  "form_field": "*[data-moz-autofill-inspect-id='{name}']",
+  "fields": [
+    "311621a3-d8f9-4dd3-bf9d-1eeb08f7f8a5",
+    "4ef6a37d-b02f-4fa7-bfc1-0994180238c8",
+    "78daaaff-d49c-4a2e-91a7-9cc214224f72"
+]
+}

l10n_CM/run_l10n.py

@@ -13,7 +13,7 @@
 valid_flags = {"--run-headless", "-n", "--reruns", "--fx-executable", "--ci"}
 flag_with_parameter = {"-n", "--reruns"}
 valid_region = {"US", "CA", "DE", "FR"}
-valid_sites = {"demo", "amazon", "walmart", "mediamarkt"}
+valid_sites = {"demo", "amazon", "walmart", "mediamarkt", "lowes"}
 live_sites = []
 
 LOCALHOST = "127.0.0.1"

l10n_CM/sites/lowes/US/lowes_ad.html

@@ -225,28 +225,26 @@ def verify_form_data(self, sample_data: CreditCardBase | AutofillAddressBase):
         for attr_name, field_name in self.field_mapping.items():
             if non_us_ca_address and field_name == "address-level1":
                 continue
-            if field_name != "cc-csc":
-                expected_value = getattr(sample_data, attr_name, None)
-                autofilled_field = self.get_element("form-field", labels=[field_name])
-                if autofilled_field.tag_name.lower() != "select":
-                    autofilled_field_value = autofilled_field.get_attribute("value")
-                    # self.expect_element_attribute_contains(
-                    #     "form-field", "value", expected_value, labels=[field_name]
-                    # )
-                else:
-                    autofilled_field_value = Select(
-                        autofilled_field
-                    ).first_selected_option.text
-                if (
-                    field_name == "address-level1"
-                    and autofilled_field_value != expected_value
-                ):
-                    expected_value = self.util.get_state_province_abbreviation(
-                        expected_value
-                    )
-                assert expected_value in autofilled_field_value, (
-                    f"{autofilled_field_value} is different from {expected_value}"
+            if attr_name == "cvv":
+                continue
+            expected_value = getattr(sample_data, attr_name, None)
+            autofilled_field = self.get_element("form-field", labels=[field_name])
+            if autofilled_field.tag_name.lower() != "select":
+                autofilled_field_value = autofilled_field.get_attribute("value")
+            else:
+                autofilled_field_value = Select(
+                    autofilled_field
+                ).first_selected_option.text
+            if (
+                field_name == "address-level1"
+                and autofilled_field_value != expected_value
+            ):
+                expected_value = self.util.get_state_province_abbreviation(
+                    expected_value
                 )
+            assert expected_value in autofilled_field_value, (
+                f"{autofilled_field_value} is different from {expected_value}"
+            )
 
     def verify_field_autofill_dropdown(
         self,

l10n_CM/sites/lowes/US/lowes_cc.html

@@ -1,5 +1,7 @@
 import logging
+
 from dotenv import load_dotenv
+
 from modules.testrail_integration import testrail_init
 
 # Load environment variables from .env file
@@ -31,7 +33,9 @@ def get_all_cases_from_suite(tr, project_id, suite_id):
 
     while True:
         # Build the API endpoint with pagination parameters
-        endpoint = f"get_cases/{project_id}&suite_id={suite_id}&limit={limit}&offset={offset}"
+        endpoint = (
+            f"get_cases/{project_id}&suite_id={suite_id}&limit={limit}&offset={offset}"
+        )
         response = tr.client.send_get(endpoint)  # Send API request to TestRail
         cases = response.get("cases", [])  # Extract the list of test cases
 
@@ -86,13 +90,21 @@ def get_all_cases_from_suite(tr, project_id, suite_id):
 
         if not DRY_RUN:
             # If not in dry run mode, perform the update
-            result = tr.update_case_field(case["id"], "custom_sub_test_suites", CUSTOM_SUB_TEST_SUITES)
-            logging.info(f"Updated case {case['id']} to '{CUSTOM_SUB_TEST_SUITES}', Result: {result}")
+            result = tr.update_case_field(
+                case["id"], "custom_sub_test_suites", CUSTOM_SUB_TEST_SUITES
+            )
+            logging.info(
+                f"Updated case {case['id']} to '{CUSTOM_SUB_TEST_SUITES}', Result: {result}"
+            )
         else:
             # In dry run mode, just log the intended change without making it
-            logging.info(f"[DRY RUN] Would update case {case['id']} from '{current_value}' to '{CUSTOM_SUB_TEST_SUITES}'.")
+            logging.info(
+                f"[DRY RUN] Would update case {case['id']} from '{current_value}' to '{CUSTOM_SUB_TEST_SUITES}'."
+            )
 
         updated_count += 1  # Increment the updated count
 
     # Log the total number of cases updated or to be updated
-    logging.info(f"Total cases {'updated' if not DRY_RUN else 'to be updated'}: {updated_count}")
+    logging.info(
+        f"Total cases {'updated' if not DRY_RUN else 'to be updated'}: {updated_count}"
+    )