dearwolf/aider
1
0
Fork 0
mirror of https://github.com/Aider-AI/aider.git synced 2025-05-29 16:54:59 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

refactor: Remove redundant test case for request timeout in send_completion

Browse source
This commit is contained in:
Paul Gauthier 2025-02-06 11:48:55 -08:00 • committed by Paul Gauthier (aider)
parent 36ea166c20
commit 53ce96b48f
1 changed files with 0 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

15
tests/basic/test_models.py
View file

@ -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