lint

Author: ben-c-at-moz

l10n_CM/Unified/test_demo_ad_7a_capture_doorhanger_stored_data_name_org_fields.py

@@ -64,6 +64,8 @@ def test_demo_ad_name_org_captured_in_doorhanger_and_stored(
             if field_value not in saved_address_profile:
                 missing_fields.append(f"{field_name}: {field_value}")
 
-        assert not missing_fields, f"The following fields were not found in the saved address: {', '.join(missing_fields)}"
+        assert not missing_fields, (
+            f"The following fields were not found in the saved address: {', '.join(missing_fields)}"
+        )
     else:
         pytest.skip("Doorhanger not tested for Live Sites.")

l10n_CM/Unified/test_demo_ad_7b_capture_doorhanger_stored_data_address_fields.py

@@ -76,6 +76,8 @@ def test_demo_ad_address_data_captured_in_doorhanger_and_stored(
             if field_value not in saved_address_profile:
                 missing_fields.append(f"{field_name}: {field_value}")
 
-        assert not missing_fields, f"The following fields were not found in the saved address: {', '.join(missing_fields)}"
+        assert not missing_fields, (
+            f"The following fields were not found in the saved address: {', '.join(missing_fields)}"
+        )
     else:
         pytest.skip("Doorhanger not tested for Live Sites.")

l10n_CM/Unified/test_demo_ad_7c_capture_doorhanger_stored_data_phone_email.py

@@ -65,6 +65,8 @@ def test_demo_ad_email_phone_captured_in_doorhanger_and_stored(
             if field_value not in saved_address_profile:
                 missing_fields.append(f"{field_name}: {field_value}")
 
-        assert not missing_fields, f"The following fields were not found in the saved address: {', '.join(missing_fields)}"
+        assert not missing_fields, (
+            f"The following fields were not found in the saved address: {', '.join(missing_fields)}"
+        )
     else:
         pytest.skip("Doorhanger not tested for Live Sites.")

l10n_CM/Unified/test_demo_cc_1_dropdown.py

@@ -2,7 +2,7 @@
 from selenium.webdriver import Firefox
 
 from modules.classes.credit_card import CreditCardBase
-from modules.page_object import AboutPrefs, CreditCardFill
+from modules.page_object import CreditCardFill
 
 
 @pytest.fixture()

l10n_CM/Unified/test_demo_cc_2_preview.py

@@ -3,7 +3,7 @@
 
 from modules.browser_object_autofill_popup import AutofillPopup
 from modules.classes.credit_card import CreditCardBase
-from modules.page_object import AboutPrefs, CreditCardFill
+from modules.page_object import CreditCardFill
 from modules.util import Utilities
 
 

l10n_CM/Unified/test_demo_cc_3_autofill.py

@@ -4,7 +4,6 @@
 from modules.browser_object_autofill_popup import AutofillPopup
 from modules.classes.credit_card import CreditCardBase
 from modules.page_object import CreditCardFill
-from modules.util import Utilities
 
 
 @pytest.fixture()

l10n_CM/Unified/test_demo_cc_5_clear.py

@@ -1,10 +1,8 @@
 import pytest
 from selenium.webdriver import Firefox
 
-from modules.browser_object_autofill_popup import AutofillPopup
 from modules.classes.credit_card import CreditCardBase
-from modules.page_object import AboutPrefs, CreditCardFill
-from modules.util import Utilities
+from modules.page_object import CreditCardFill
 
 
 @pytest.fixture()

l10n_CM/Unified/test_demo_cc_6_capture_doorhanger_appearance.py

@@ -3,7 +3,6 @@
 
 from modules.browser_object_autofill_popup import AutofillPopup
 from modules.page_object_autofill import CreditCardFill
-from modules.util import Utilities
 
 
 @pytest.fixture()

l10n_CM/Unified/test_demo_cc_7_capture_doorhanger_capture_and_stored_data.py

@@ -33,21 +33,21 @@ def test_demo_cc_data_captured_in_doorhanger_and_stored(
         credit_card_sample_data = credit_card_autofill.fill_and_save(door_hanger=False)
 
         # The "Save credit card?" doorhanger is displayed
-        assert autofill_popup.element_visible(
-            "doorhanger-save-button"
-        ), "Credit card save doorhanger is not visible"
+        assert autofill_popup.element_visible("doorhanger-save-button"), (
+            "Credit card save doorhanger is not visible"
+        )
 
         # Verify Credit Card Doorhanger Data
         doorhanger_text = autofill_popup.get_cc_doorhanger_data("cc-doorhanger-data")
-        assert (
-            credit_card_sample_data.card_number[-4:] in doorhanger_text
-        ), f"Expected last 4 digits '{credit_card_sample_data.card_number[-4:]}' but not found."
-        assert (
-            credit_card_sample_data.name in doorhanger_text
-        ), f"Expected name '{credit_card_sample_data.name}' but not found."
-        assert (
-            credit_card_sample_data.cvv not in doorhanger_text
-        ), f"CVV '{credit_card_sample_data.cvv}' should not be saved, but found in doorhanger."
+        assert credit_card_sample_data.card_number[-4:] in doorhanger_text, (
+            f"Expected last 4 digits '{credit_card_sample_data.card_number[-4:]}' but not found."
+        )
+        assert credit_card_sample_data.name in doorhanger_text, (
+            f"Expected name '{credit_card_sample_data.name}' but not found."
+        )
+        assert credit_card_sample_data.cvv not in doorhanger_text, (
+            f"CVV '{credit_card_sample_data.cvv}' should not be saved, but found in doorhanger."
+        )
 
         # Click the "Save" button using click_doorhanger_button
         autofill_popup.click_doorhanger_button("save")
@@ -63,16 +63,14 @@ def test_demo_cc_data_captured_in_doorhanger_and_stored(
         saved_cc_profile = [x.strip() for x in saved_cc_profiles[0].text.split(",")]
 
         # Validate stored values match expected values
-        assert saved_cc_profile[
-            0
-        ].endswith(
-            credit_card_sample_data.card_number[-4:]
-        ), f"Expected last 4 digits '{credit_card_sample_data.card_number[-4:]}' but got '{saved_cc_profile[0]}'"
-        assert (
-            saved_cc_profile[1] == credit_card_sample_data.name
-        ), f"Expected name '{credit_card_sample_data.name}' but got '{saved_cc_profile[1]}'"
-        assert (
-            credit_card_sample_data.cvv not in saved_cc_profile
-        ), f"CVV '{credit_card_sample_data.cvv}' should not be saved, but found in stored values."
+        assert saved_cc_profile[0].endswith(credit_card_sample_data.card_number[-4:]), (
+            f"Expected last 4 digits '{credit_card_sample_data.card_number[-4:]}' but got '{saved_cc_profile[0]}'"
+        )
+        assert saved_cc_profile[1] == credit_card_sample_data.name, (
+            f"Expected name '{credit_card_sample_data.name}' but got '{saved_cc_profile[1]}'"
+        )
+        assert credit_card_sample_data.cvv not in saved_cc_profile, (
+            f"CVV '{credit_card_sample_data.cvv}' should not be saved, but found in stored values."
+        )
     else:
         pytest.skip("Doorhanger not tested for Live Sites.")

modules/browser_object_tracker_panel.py

@@ -145,7 +145,9 @@ def verify_tracker_shield_indicator(self, nav: Navigation) -> BasePage:
             assert (
                 shield_icon.get_attribute("data-l10n-id")
                 == "tracking-protection-icon-active-container"
-            ), "The label detected did not correspond to the expected one: tracking-protection-icon-active-container"
+            ), (
+                "The label detected did not correspond to the expected one: tracking-protection-icon-active-container"
+            )
         return self
 
     def open_and_return_cross_site_trackers(self) -> List[WebElement]: