pylint-fixed

Author: Ritesh-Microsoft

content-gen/tests/e2e-test/pages/HomePage.py

@@ -1,6 +1,7 @@
 """Home page object module for Fabric SQL automation tests."""
 import logging
 import json
+import os
 import re
 import math
 from io import BytesIO
@@ -440,7 +441,7 @@ def delete_chat_history(self):
             initial_count = chat_history.count()
             logger.info(f"Initial chat history count: {initial_count}")
 
-            if initial_count == 0:
+            if not initial_count:
                 error_msg = "No chat history items available to delete"
                 logger.error(f"❌ {error_msg}")
                 raise AssertionError(error_msg)
@@ -574,7 +575,7 @@ def send_prompt_from_quick_link(self, quick_link=None):
 
         try:
             # Step 1: Click on the quick link
-            logger.info(f"Step 1: Clicking on quick link...")
+            logger.info("Step 1: Clicking on quick link...")
             user_message = self.page.locator(quick_link)
             expect(user_message).to_be_visible(timeout=10000)
             user_message.click()
@@ -902,7 +903,6 @@ def download_image(self):
             # Step 3: Validate the downloaded file is not empty
             logger.info("Step 3: Validating downloaded file is not empty...")
             download_path = download.path()
-            import os
             file_size = os.path.getsize(download_path)
             logger.info(f"  Downloaded file size: {file_size} bytes")
             assert file_size > 0, "Downloaded file is empty (0 bytes)"
@@ -1226,7 +1226,7 @@ def validate_brief_confirmed_accuracy(self, expected_keywords=None, extra_fields
             logger.error(f"❌ {error_msg}")
             raise AssertionError(error_msg) from e
 
-    def validate_generated_copy_accuracy(self, product_name, generated_content_locator=None,
+    def validate_generated_copy_accuracy(self, product_name, generated_content_locator=None,  # noqa: ARG002
                                            min_length=30, expected_copy_keywords=None):
         """
         Validate that the generated marketing copy is accurate and relevant.
@@ -1391,7 +1391,6 @@ def validate_products_selected_section(self, expected_product_name,
                     logger.info(f"✓ [Soft] Price found: {expected_price_pattern}")
                 else:
                     # Also try a generic price regex as fallback
-                    import re
                     price_match = re.search(r'\$\d+\.\d{2}\s*usd', section_text_lower)
                     if price_match:
                         logger.info(

content-gen/tests/e2e-test/tests/test_content_gen_gp_tc.py

@@ -1,7 +1,6 @@
 """Test module for Content Generation golden path test cases."""
 import logging
 import time
-import pytest
 
 from pages.HomePage import HomePage
 from config.constants import URL
@@ -691,6 +690,3 @@ def test_validate_clear_all_chat_history(login_logout, request):
         log_test_failure(start_time, e)
         raise
 
-
-
-