Fix F841 and E127 linting errors: remove unused _app assignments and fix continuation line indentation

Author: Prachig-Microsoft

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))
+            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))
+            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))
+            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))
+            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))
+            ConcreteApplication(env_file_path=str(env_file))
 
             # AppConfigurationHelper should not be called with empty endpoint
             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")
+            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()
+            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()
+            TestApp()
 
             # Should have configured logging
             mock_logging.assert_called_once()
@@ -120,7 +120,7 @@ def test_validate_azure_authentication_azure_hosted(self):
             'WEBSITE_SITE_NAME': 'my-webapp',
             'MSI_ENDPOINT': 'http://localhost:8081/msi/token'
         }), \
-             patch('utils.credential_util.get_azure_credential') as mock_get_cred:
+            patch('utils.credential_util.get_azure_credential') as mock_get_cred:
 
             mock_get_cred.return_value = Mock()
 
@@ -139,7 +139,7 @@ def test_validate_azure_authentication_with_client_id(self):
             'AZURE_CLIENT_ID': 'client-id-123',
             'IDENTITY_ENDPOINT': 'http://localhost:8081/token'
         }), \
-             patch('utils.credential_util.get_azure_credential') as mock_get_cred:
+            patch('utils.credential_util.get_azure_credential') as mock_get_cred:
 
             mock_get_cred.return_value = Mock()
 

src/tests/ContentProcessorWorkflow/libs/test_ultra_focused_80.py

@@ -101,7 +101,7 @@ def run(self):
             mock_config.return_value.app_logging_enable = False
 
             # Create app with no explicit env path
-            _app = TestApp()
+            TestApp()
 
             # Should have called load_dotenv
             assert mock_load.called
@@ -118,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()