mirror of
https://github.com/BerriAI/litellm.git
synced 2025-04-24 18:24:20 +00:00
fix(databricks/common_utils.py): fix custom endpoint check (#9925)
* fix(databricks/common_utils.py): fix custom endpoint check Fixes https://github.com/BerriAI/litellm/issues/9915 * fix(common_utils.py): add unit test to ensure custom_endpoint=False is handled correctly Fixes https://github.com/BerriAI/litellm/issues/9915
This commit is contained in:
parent
3ca82c22b6
commit
b9f01c9f5b
2 changed files with 33 additions and 1 deletions
|
@ -68,7 +68,7 @@ class DatabricksBase:
|
|||
headers: Optional[dict],
|
||||
) -> Tuple[str, dict]:
|
||||
if api_key is None and not headers: # handle empty headers
|
||||
if custom_endpoint is not None:
|
||||
if custom_endpoint is True:
|
||||
raise DatabricksException(
|
||||
status_code=400,
|
||||
message="Missing API Key - A call is being made to LLM Provider but no key is set either in the environment variables ({LLM_PROVIDER}_API_KEY) or via params",
|
||||
|
|
|
@ -0,0 +1,32 @@
|
|||
import json
|
||||
import os
|
||||
import sys
|
||||
|
||||
import pytest
|
||||
from fastapi.testclient import TestClient
|
||||
|
||||
sys.path.insert(
|
||||
0, os.path.abspath("../../../..")
|
||||
) # Adds the parent directory to the system path
|
||||
from unittest.mock import MagicMock, patch
|
||||
|
||||
from litellm.llms.databricks.common_utils import DatabricksBase
|
||||
|
||||
|
||||
def test_databricks_validate_environment():
|
||||
databricks_base = DatabricksBase()
|
||||
|
||||
with patch.object(
|
||||
databricks_base, "_get_databricks_credentials"
|
||||
) as mock_get_credentials:
|
||||
try:
|
||||
databricks_base.databricks_validate_environment(
|
||||
api_key=None,
|
||||
api_base="my_api_base",
|
||||
endpoint_type="chat_completions",
|
||||
custom_endpoint=False,
|
||||
headers=None,
|
||||
)
|
||||
except Exception:
|
||||
pass
|
||||
mock_get_credentials.assert_called_once()
|
Loading…
Add table
Add a link
Reference in a new issue