Skip to content

Commit e55203f

Browse files
committed
Fixing test case, and adding better exception message.
1 parent c516c55 commit e55203f

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

aisuite/providers/azure_provider.py

+6-2
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,12 @@ class AzureProvider(Provider):
1010
def __init__(self, **config):
1111
self.base_url = config.get("base_url") or os.getenv("AZURE_BASE_URL")
1212
self.api_key = config.get("api_key") or os.getenv("AZURE_API_KEY")
13-
if not self.api_key or not self.base_url:
14-
raise ValueError("api_key is required in the config")
13+
if not self.api_key:
14+
raise ValueError("For Azure, api_key is required.")
15+
if not self.base_url:
16+
raise ValueError(
17+
"For Azure, base_url is required. Check your deployment page for a URL like this - https://<model-deployment-name>.<region>.models.ai.azure.com"
18+
)
1519

1620
def chat_completions_create(self, model, messages, **kwargs):
1721
url = f"https://{model}.westus3.models.ai.azure.com/v1/chat/completions"

tests/client/test_client.py

+1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ def test_client_chat_completions(
4646
},
4747
ProviderNames.AZURE: {
4848
"api_key": "azure-api-key",
49+
"base_url": "https://model.ai.azure.com",
4950
},
5051
ProviderNames.GROQ: {
5152
"api_key": "groq-api-key",

0 commit comments

Comments
 (0)