Skip to content

Commit a7a6381

Browse files
fix: flaky passthrough tests (#14692)
* fix: flaky passthrough tests * Revert "fix: flaky passthrough tests" This reverts commit ffe692e. * fix: serialize prisma objects
1 parent c6e37bb commit a7a6381

File tree

3 files changed

+46
-8
lines changed

3 files changed

+46
-8
lines changed

litellm/litellm_core_utils/exception_mapping_utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -556,7 +556,7 @@ def exception_type( # type: ignore # noqa: PLR0915
556556
model=model,
557557
llm_provider="anthropic",
558558
)
559-
elif "overloaded_error" in error_str:
559+
elif "overloaded_error" in error_str or "Overloaded" in error_str:
560560
exception_mapping_worked = True
561561
raise InternalServerError(
562562
message="AnthropicError - {}".format(error_str),

tests/otel_tests/test_prometheus.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -252,8 +252,8 @@ async def create_test_team(
252252

253253
async def create_test_user(
254254
session: aiohttp.ClientSession, user_data: Dict[str, Any]
255-
) -> str:
256-
"""Create a new user and return the user_id"""
255+
) -> Dict[str, Any]:
256+
"""Create a new user and return the user info"""
257257
url = "http://0.0.0.0:4000/user/new"
258258
headers = {
259259
"Authorization": "Bearer sk-1234",
@@ -579,7 +579,7 @@ async def test_user_email_in_all_required_metrics():
579579
- litellm_input_tokens_metric_total
580580
- litellm_output_tokens_metric_total
581581
- litellm_requests_metric_total
582-
- litellm_spend_metric_total
582+
- litellm_spend_metric
583583
"""
584584
async with aiohttp.ClientSession() as session:
585585
# Create a user with user_email
@@ -611,12 +611,12 @@ async def test_user_email_in_all_required_metrics():
611611
"litellm_input_tokens_metric_total",
612612
"litellm_output_tokens_metric_total",
613613
"litellm_requests_metric_total",
614-
"litellm_spend_metric_total"
614+
"litellm_spend_metric"
615615
]
616616

617+
import re
617618
for metric_name in required_metrics_with_user_email:
618619
# Check that the metric exists and contains user_email label
619-
import re
620620
# Look for the metric with user_email in its labels
621621
pattern = rf'{metric_name}{{[^}}]*user_email="{re.escape(user_email)}"[^}}]*}}'
622622
matches = re.findall(pattern, metrics_text)

tests/proxy_unit_tests/test_key_generate_prisma.py

Lines changed: 40 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3428,6 +3428,16 @@ async def test_list_keys(prisma_client):
34283428
),
34293429
page=1,
34303430
size=10,
3431+
user_id=None,
3432+
team_id=None,
3433+
organization_id=None,
3434+
key_hash=None,
3435+
key_alias=None,
3436+
return_full_object=False,
3437+
include_team_keys=False,
3438+
include_created_by_keys=False,
3439+
sort_by=None,
3440+
sort_order="desc",
34313441
)
34323442
print("response=", response)
34333443
assert "keys" in response
@@ -3442,6 +3452,16 @@ async def test_list_keys(prisma_client):
34423452
UserAPIKeyAuth(user_role=LitellmUserRoles.PROXY_ADMIN.value),
34433453
page=1,
34443454
size=2,
3455+
user_id=None,
3456+
team_id=None,
3457+
organization_id=None,
3458+
key_hash=None,
3459+
key_alias=None,
3460+
return_full_object=False,
3461+
include_team_keys=False,
3462+
include_created_by_keys=False,
3463+
sort_by=None,
3464+
sort_order="desc",
34453465
)
34463466
print("pagination response=", response)
34473467
assert len(response["keys"]) == 2
@@ -3470,9 +3490,18 @@ async def test_list_keys(prisma_client):
34703490
response = await list_keys(
34713491
request,
34723492
UserAPIKeyAuth(user_role=LitellmUserRoles.PROXY_ADMIN.value),
3473-
user_id=user_id,
34743493
page=1,
34753494
size=10,
3495+
user_id=user_id,
3496+
team_id=None,
3497+
organization_id=None,
3498+
key_hash=None,
3499+
key_alias=None,
3500+
return_full_object=False,
3501+
include_team_keys=False,
3502+
include_created_by_keys=False,
3503+
sort_by=None,
3504+
sort_order="desc",
34763505
)
34773506
print("filtered user_id response=", response)
34783507
assert len(response["keys"]) == 1
@@ -3482,9 +3511,18 @@ async def test_list_keys(prisma_client):
34823511
response = await list_keys(
34833512
request,
34843513
UserAPIKeyAuth(user_role=LitellmUserRoles.PROXY_ADMIN.value),
3485-
key_alias=key_alias,
34863514
page=1,
34873515
size=10,
3516+
user_id=None,
3517+
team_id=None,
3518+
organization_id=None,
3519+
key_hash=None,
3520+
key_alias=key_alias,
3521+
return_full_object=False,
3522+
include_team_keys=False,
3523+
include_created_by_keys=False,
3524+
sort_by=None,
3525+
sort_order="desc",
34883526
)
34893527
assert len(response["keys"]) == 1
34903528
assert _key in response["keys"]

0 commit comments

Comments
 (0)