Small corrections

Author: soncuteanca

l10n_CM/Unified/test_demo_ad_yellow_highlight.py

@@ -8,13 +8,16 @@
 
 @pytest.fixture()
 def test_case():
-    return "2888569"
+    return "2888559"
 
 
-def test_address_yellow_highlight_on_name_org_fields(driver: Firefox, region: str):
+def test_address_yellow_highlight_on_name_organization_fields(
+    driver: Firefox, region: str
+):
     """
-    C2888569 - Verify Autofill functionality when selecting an entry from the dropdown for tele/email fields
+    C2888559 - Verify the yellow highlight appears on autofilled fields for name and organization.
     """
+
     # Instantiate objects
     address_autofill = AddressFill(driver)
     address_autofill_popup = AutofillPopup(driver)
@@ -28,15 +31,15 @@ def test_address_yellow_highlight_on_name_org_fields(driver: Firefox, region: st
     # Click the "Save" button
     address_autofill_popup.click_doorhanger_button("save")
 
-    # Double inside phone field and select a saved address entry from the dropdown
+    # Double click inside phone field and select a saved address entry from the dropdown
     address_autofill.double_click("form-field", labels=["name"])
 
     # Click on the first element from the autocomplete dropdown
     first_item = address_autofill_popup.get_nth_element(1)
     address_autofill_popup.click_on(first_item)
 
-    # Verify highlighted fields
+    # Verify the name and organization fields are highlighted
     address_autofill.verify_field_yellow_highlights(
-        fields_to_test=["name", "organization"],  # Only test name & org
+        fields_to_test=["name", "organization"],
         expected_highlighted_fields=["name", "organization"],
     )

modules/page_object_autofill.py

@@ -39,18 +39,19 @@ def click_form_button(self, field_name):
         """Clicks submit on the form"""
         self.click_on("submit-button", labels=[field_name])
 
-    def verify_field_highlights_in_list(
+    def verify_field_highlight(
         self,
         fields_to_test: List[str],
         expected_highlighted_fields: Optional[List[str]] = None,
         extra_fields: Optional[List[str]] = None,
     ):
         """
-        A common method to check which fields have the "yellow highlight."
+        A common method to check which fields have the "yellow highlight". This is used in both CC and Address pages.
         - fields_to_test: The primary list of fields for this page (cc fields, address fields).
         - expected_highlighted_fields: Which ones are expected to be highlighted. Defaults to all in `fields_to_test`.
         - extra_fields: If some pages have extra fields to test (e.g. 'cc-csc'), pass them here.
         """
+
         if expected_highlighted_fields is None:
             # By default, everything in fields_to_test is expected to be highlighted
             expected_highlighted_fields = fields_to_test[:]
@@ -388,7 +389,7 @@ def verify_field_yellow_highlights(self, expected_highlighted_fields=None):
         We also want to include the "cc-csc" field in the test, so we
         pass it via 'extra_fields'.
         """
-        return self.verify_field_highlights_in_list(
+        return self.verify_field_highlight(
             fields_to_test=self.fields,
             expected_highlighted_fields=expected_highlighted_fields,
             extra_fields=["cc-csc"],
@@ -563,7 +564,7 @@ def verify_field_yellow_highlights(
         if fields_to_test is None:
             fields_to_test = self.fields  # By default, test all address fields
 
-        return self.verify_field_highlights_in_list(
+        return self.verify_field_highlight(
             fields_to_test=fields_to_test,
             expected_highlighted_fields=expected_highlighted_fields,
         )