Skip to content

Commit d09510c

Browse files
PR Review Changes
1 parent 2099be1 commit d09510c

2 files changed

Lines changed: 7 additions & 2 deletions

File tree

src/ContentProcessor/src/tests/azure_helper/test_storage_blob.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,12 @@
55

66
@pytest.fixture
77
def mock_blob_service_client(mocker):
8-
return mocker.patch("libs.azure_helper.storage_blob.BlobServiceClient")
8+
return mocker.patch("azure.storage.blob.BlobServiceClient")
99

1010

1111
@pytest.fixture
1212
def mock_default_azure_credential(mocker):
13-
return mocker.patch("libs.azure_helper.storage_blob.DefaultAzureCredential")
13+
return mocker.patch("azure.storage.blob.BlobServiceClient")
1414

1515

1616
@pytest.fixture

src/ContentProcessorAPI/tests/helpers/test_azure_credential_utils.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010

1111
# Synchronous tests
1212

13+
1314
@patch("helpers.azure_credential_utils.os.getenv")
1415
@patch("helpers.azure_credential_utils.DefaultAzureCredential")
1516
@patch("helpers.azure_credential_utils.ManagedIdentityCredential")
@@ -26,6 +27,7 @@ def test_get_azure_credential_dev_env(mock_managed_identity_credential, mock_def
2627
mock_managed_identity_credential.assert_not_called()
2728
assert credential == mock_default_credential
2829

30+
2931
@patch("helpers.azure_credential_utils.os.getenv")
3032
@patch("helpers.azure_credential_utils.DefaultAzureCredential")
3133
@patch("helpers.azure_credential_utils.ManagedIdentityCredential")
@@ -41,8 +43,10 @@ def test_get_azure_credential_non_dev_env(mock_managed_identity_credential, mock
4143
mock_default_azure_credential.assert_not_called()
4244
assert credential == mock_managed_credential
4345

46+
4447
# Asynchronous tests
4548

49+
4650
@pytest.mark.asyncio
4751
@patch("helpers.azure_credential_utils.os.getenv")
4852
@patch("helpers.azure_credential_utils.AioDefaultAzureCredential")
@@ -60,6 +64,7 @@ async def test_get_azure_credential_async_dev_env(mock_aio_managed_identity_cred
6064
mock_aio_managed_identity_credential.assert_not_called()
6165
assert credential == mock_aio_default_credential
6266

67+
6368
@pytest.mark.asyncio
6469
@patch("helpers.azure_credential_utils.os.getenv")
6570
@patch("helpers.azure_credential_utils.AioDefaultAzureCredential")

0 commit comments

Comments
 (0)