Fix linting errors: remove escaped quotes, prefix unused app variables, fix indentation, remove extra blank line

Author: Prachig-Microsoft

src/tests/ContentProcessorAPI/libs/test_cosmos_db_helper.py

@@ -167,7 +167,7 @@ def test_delete_document(mock_certifi, mock_mongo_client):
     mock_result = MagicMock()
     mock_container.delete_one.return_value = mock_result
 
-    helper.delete_document(\"test_id\")
+    helper.delete_document("test_id")
     mock_container.delete_one.assert_called_once_with({"Id": "test_id"})
 
 

src/tests/ContentProcessorWorkflow/libs/test_application_base_extended.py

@@ -81,7 +81,7 @@ def test_initialization_with_app_config_endpoint(self, tmp_path, monkeypatch):
             mock_app_config_instance = Mock()
             mock_app_config.return_value = mock_app_config_instance
 
-            app = ConcreteApplication(env_file_path=str(env_file))
+            _app = ConcreteApplication(env_file_path=str(env_file))
 
             mock_app_config.assert_called_once()
             mock_app_config_instance.read_and_set_environmental_variables.assert_called_once()
@@ -101,7 +101,7 @@ def test_initialization_with_logging_enabled(self, tmp_path, monkeypatch):
 
             mock_cred.return_value = Mock()
 
-            app = ConcreteApplication(env_file_path=str(env_file))
+            _app = ConcreteApplication(env_file_path=str(env_file))
 
             # Verify logging was configured
             mock_logging.assert_called_once()
@@ -122,7 +122,7 @@ def test_initialization_without_logging(self, tmp_path, monkeypatch):
 
             mock_cred.return_value = Mock()
 
-            app = ConcreteApplication(env_file_path=str(env_file))
+            _app = ConcreteApplication(env_file_path=str(env_file))
 
             # Verify logging was NOT configured
             mock_logging.assert_not_called()
@@ -158,7 +158,7 @@ def test_load_env_with_explicit_path(self, tmp_path):
              patch('libs.base.application_base.AgentFrameworkSettings'), \
              patch('libs.base.application_base.load_dotenv') as mock_load_dotenv:
 
-            app = ConcreteApplication(env_file_path=str(env_file))
+            _app = ConcreteApplication(env_file_path=str(env_file))
 
             # Verify load_dotenv was called at least once
             assert mock_load_dotenv.call_count >= 1
@@ -257,7 +257,7 @@ def test_empty_app_config_endpoint_skipped(self, tmp_path, monkeypatch):
              patch('libs.base.application_base.AppConfigurationHelper') as mock_app_config, \
              patch('libs.base.application_base.AgentFrameworkSettings'):
 
-            app = ConcreteApplication(env_file_path=str(env_file))
+            _app = ConcreteApplication(env_file_path=str(env_file))
 
             # AppConfigurationHelper should not be called with empty endpoint
             mock_app_config.assert_not_called()
@@ -274,7 +274,7 @@ def test_none_app_config_endpoint_skipped(self, tmp_path, monkeypatch):
              patch('libs.base.application_base.AppConfigurationHelper') as mock_app_config, \
              patch('libs.base.application_base.AgentFrameworkSettings'):
 
-            app = ConcreteApplication(env_file_path=str(env_file))
+            _app = ConcreteApplication(env_file_path=str(env_file))
 
             # AppConfigurationHelper should not be called
             mock_app_config.assert_not_called()

src/tests/ContentProcessorWorkflow/libs/test_final_80_percent_push.py

@@ -221,7 +221,7 @@ def test_get_azure_credential_with_all_env_vars(self):
             'AZURE_TENANT_ID': 'test-tenant-id',
             'AZURE_CLIENT_SECRET': 'test-secret'
         }), \
-            patch('utils.credential_util.DefaultAzureCredential') as mock_cred:
+             patch('utils.credential_util.DefaultAzureCredential') as mock_cred:
 
             mock_cred.return_value = Mock()
 

src/tests/ContentProcessorWorkflow/libs/test_push_to_80_percent.py

@@ -26,7 +26,7 @@ def run(self):
             mock_env_config.return_value.app_config_endpoint = ""
 
             # Test with explicit path
-            app = TestApp(env_file_path="/custom/path/.env")
+            _app = TestApp(env_file_path="/custom/path/.env")
 
             # Should have loaded from explicit path
             mock_load_dotenv.assert_called_with(dotenv_path="/custom/path/.env")
@@ -53,7 +53,7 @@ def run(self):
             mock_env_config.return_value.app_config_endpoint = "https://myconfig.azconfig.io"
             mock_config.return_value.app_logging_enable = False
 
-            app = TestApp()
+            _app = TestApp()
 
             # Should have created AppConfigurationHelper
             assert mock_app_config.called
@@ -85,7 +85,7 @@ def run(self):
             config_instance.app_logging_level = "DEBUG"
             mock_config.return_value = config_instance
 
-            app = TestApp()
+            _app = TestApp()
 
             # Should have configured logging
             mock_logging.assert_called_once()

src/tests/ContentProcessorWorkflow/libs/test_ultra_focused_80.py

@@ -84,7 +84,6 @@ def test_load_env_returns_path(self):
         """Test that _load_env returns the loaded path"""
         from libs.base.application_base import ApplicationBase
 
-
         class TestApp(ApplicationBase):
             def initialize(self):
                 pass
@@ -102,7 +101,7 @@ def run(self):
             mock_config.return_value.app_logging_enable = False
 
             # Create app with no explicit env path
-            app = TestApp()
+            _app = TestApp()
 
             # Should have called load_dotenv
             assert mock_load.called
@@ -119,7 +118,7 @@ def test_validate_authentication_with_kubernetes(self):
             'KUBERNETES_SERVICE_HOST': 'kubernetes.default.svc',
             'IDENTITY_ENDPOINT': 'http://169.254.169.254/metadata/identity'
         }), \
-            patch('utils.credential_util.get_azure_credential') as mock_cred:
+             patch('utils.credential_util.get_azure_credential') as mock_cred:
 
             mock_cred.return_value = Mock()