From cae9245b3343421c04279eac4a8aa9df2279410e Mon Sep 17 00:00:00 2001 From: Ishaan Jaff Date: Tue, 20 Aug 2024 13:02:24 -0700 Subject: [PATCH] fix router retries tests --- litellm/tests/test_router_retries.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/litellm/tests/test_router_retries.py b/litellm/tests/test_router_retries.py index 4d9abb2d953..f0503cd3f18 100644 --- a/litellm/tests/test_router_retries.py +++ b/litellm/tests/test_router_retries.py @@ -392,7 +392,7 @@ def test_retry_rate_limit_error_with_healthy_deployments(): def test_do_retry_rate_limit_error_with_no_fallbacks_and_no_healthy_deployments(): """ - Test 2. It SHOULD Retry, when healthy_deployments is [] and fallbacks is None + Test 2. It SHOULD NOT Retry, when healthy_deployments is [] and fallbacks is None """ healthy_deployments = [] @@ -415,9 +415,10 @@ def test_do_retry_rate_limit_error_with_no_fallbacks_and_no_healthy_deployments( response = router.should_retry_this_error( error=rate_limit_error, healthy_deployments=healthy_deployments ) - assert response == True + pytest.fail("Should have raised an error") except Exception as e: - pytest.fail("Should not have failed this error - {}".format(str(e))) + print("got an exception", e) + pass def test_raise_context_window_exceeded_error():