forked from phoenix/litellm-mirror
Merge pull request #4504 from BerriAI/litellm_fix_exception_provider_not_known
fix exception provider not known
This commit is contained in:
commit
4bb418acf3
2 changed files with 31 additions and 0 deletions
|
@ -249,6 +249,25 @@ def test_completion_azure_exception():
|
||||||
# test_completion_azure_exception()
|
# test_completion_azure_exception()
|
||||||
|
|
||||||
|
|
||||||
|
def test_azure_embedding_exceptions():
|
||||||
|
try:
|
||||||
|
|
||||||
|
response = litellm.embedding(
|
||||||
|
model="azure/azure-embedding-model",
|
||||||
|
input="hello",
|
||||||
|
messages="hello",
|
||||||
|
)
|
||||||
|
pytest.fail(f"Bad request this should have failed but got {response}")
|
||||||
|
|
||||||
|
except Exception as e:
|
||||||
|
print(vars(e))
|
||||||
|
# CRUCIAL Test - Ensures our exceptions are readable and not overly complicated. some users have complained exceptions will randomly have another exception raised in our exception mapping
|
||||||
|
assert (
|
||||||
|
e.message
|
||||||
|
== "litellm.APIError: AzureException APIError - Embeddings.create() got an unexpected keyword argument 'messages'"
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
async def asynctest_completion_azure_exception():
|
async def asynctest_completion_azure_exception():
|
||||||
try:
|
try:
|
||||||
import openai
|
import openai
|
||||||
|
|
|
@ -5810,6 +5810,18 @@ def exception_type(
|
||||||
_model_group = _metadata.get("model_group")
|
_model_group = _metadata.get("model_group")
|
||||||
_deployment = _metadata.get("deployment")
|
_deployment = _metadata.get("deployment")
|
||||||
extra_information = f"\nModel: {model}"
|
extra_information = f"\nModel: {model}"
|
||||||
|
|
||||||
|
exception_provider = "Unknown"
|
||||||
|
if (
|
||||||
|
isinstance(custom_llm_provider, str)
|
||||||
|
and len(custom_llm_provider) > 0
|
||||||
|
):
|
||||||
|
exception_provider = (
|
||||||
|
custom_llm_provider[0].upper()
|
||||||
|
+ custom_llm_provider[1:]
|
||||||
|
+ "Exception"
|
||||||
|
)
|
||||||
|
|
||||||
if _api_base:
|
if _api_base:
|
||||||
extra_information += f"\nAPI Base: `{_api_base}`"
|
extra_information += f"\nAPI Base: `{_api_base}`"
|
||||||
if (
|
if (
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue