From 53ce96b48f919cac512c1d183738448a12725a77 Mon Sep 17 00:00:00 2001 From: Paul Gauthier Date: Thu, 6 Feb 2025 11:48:55 -0800 Subject: [PATCH] refactor: Remove redundant test case for request timeout in send_completion --- tests/basic/test_models.py | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/tests/basic/test_models.py b/tests/basic/test_models.py index 462e9111b..aa99040a1 100644 --- a/tests/basic/test_models.py +++ b/tests/basic/test_models.py @@ -473,21 +473,6 @@ And this text should remain""" timeout=300, # From extra_params ) - @patch("aider.models.litellm.completion") - def test_request_timeout_explicit_in_call(self, mock_completion): - # Test explicit timeout in send_completion overrides both default and extra_params - model = Model("gpt-4") - model.extra_params = {"timeout": 300} # 5 minutes - messages = [{"role": "user", "content": "Hello"}] - model.send_completion(messages, functions=None, stream=False, timeout=120) # 2 minutes - mock_completion.assert_called_with( - model=model.name, - messages=messages, - stream=False, - temperature=0, - timeout=120, # Explicit in call - ) - @patch("aider.models.litellm.completion") def test_use_temperature_in_send_completion(self, mock_completion): # Test use_temperature=True sends temperature=0