Skip to content

Commit 175e8d6

Browse files
Fix final linting errors: remove unused _app variable and fix E125 continuation line indentation
1 parent d4f67d9 commit 175e8d6

File tree

4 files changed

+5
-9
lines changed

4 files changed

+5
-9
lines changed

src/tests/ContentProcessorWorkflow/libs/test_application_base_extended.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ def test_none_app_config_endpoint_skipped(self, tmp_path, monkeypatch):
274274
patch('libs.base.application_base.AppConfigurationHelper') as mock_app_config, \
275275
patch('libs.base.application_base.AgentFrameworkSettings'):
276276

277-
_app = ConcreteApplication(env_file_path=str(env_file))
277+
ConcreteApplication(env_file_path=str(env_file))
278278

279279
# AppConfigurationHelper should not be called
280280
mock_app_config.assert_not_called()

src/tests/ContentProcessorWorkflow/libs/test_final_80_percent_push.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -220,8 +220,7 @@ def test_get_azure_credential_with_all_env_vars(self):
220220
'AZURE_CLIENT_ID': 'test-client-id',
221221
'AZURE_TENANT_ID': 'test-tenant-id',
222222
'AZURE_CLIENT_SECRET': 'test-secret'
223-
}), \
224-
patch('utils.credential_util.DefaultAzureCredential') as mock_cred:
223+
}), patch('utils.credential_util.DefaultAzureCredential') as mock_cred:
225224

226225
mock_cred.return_value = Mock()
227226

src/tests/ContentProcessorWorkflow/libs/test_push_to_80_percent.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -119,8 +119,7 @@ def test_validate_azure_authentication_azure_hosted(self):
119119
with patch.dict('os.environ', {
120120
'WEBSITE_SITE_NAME': 'my-webapp',
121121
'MSI_ENDPOINT': 'http://localhost:8081/msi/token'
122-
}), \
123-
patch('utils.credential_util.get_azure_credential') as mock_get_cred:
122+
}), patch('utils.credential_util.get_azure_credential') as mock_get_cred:
124123

125124
mock_get_cred.return_value = Mock()
126125

@@ -138,8 +137,7 @@ def test_validate_azure_authentication_with_client_id(self):
138137
with patch.dict('os.environ', {
139138
'AZURE_CLIENT_ID': 'client-id-123',
140139
'IDENTITY_ENDPOINT': 'http://localhost:8081/token'
141-
}), \
142-
patch('utils.credential_util.get_azure_credential') as mock_get_cred:
140+
}), patch('utils.credential_util.get_azure_credential') as mock_get_cred:
143141

144142
mock_get_cred.return_value = Mock()
145143

src/tests/ContentProcessorWorkflow/libs/test_ultra_focused_80.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,8 +117,7 @@ def test_validate_authentication_with_kubernetes(self):
117117
with patch.dict('os.environ', {
118118
'KUBERNETES_SERVICE_HOST': 'kubernetes.default.svc',
119119
'IDENTITY_ENDPOINT': 'http://169.254.169.254/metadata/identity'
120-
}), \
121-
patch('utils.credential_util.get_azure_credential') as mock_cred:
120+
}), patch('utils.credential_util.get_azure_credential') as mock_cred:
122121

123122
mock_cred.return_value = Mock()
124123

0 commit comments

Comments
 (0)