Skip to content

Commit 3505e46

Browse files
committed
fix: update remaining e2e test timeouts from 10s to 30s
Updated timeout values in 5 test files to prevent timeout failures: - 00-client-request-test.py - 04-cache-test.py - 06-tools-test.py - 07-model-selection-test.py - 09-error-handling-test.py This should resolve the remaining timeout issues seen in local testing.
1 parent 933bbd7 commit 3505e46

File tree

5 files changed

+17
-17
lines changed

5 files changed

+17
-17
lines changed

e2e-tests/00-client-request-test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -176,14 +176,14 @@ def test_malformed_request(self):
176176

177177
try:
178178
response = self._make_request(
179-
payload, timeout=10
179+
payload, timeout=30
180180
) # Reduced timeout for error cases
181181
if not response:
182182
response = requests.post(
183183
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
184184
headers={"Content-Type": "application/json"},
185185
json=payload,
186-
timeout=10,
186+
timeout=30,
187187
)
188188

189189
passed = 400 <= response.status_code < 500

e2e-tests/04-cache-test.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ def test_cache_hit_with_identical_query(self):
136136
# First request
137137
self.print_subtest_header("First Request (Expected Cache Miss)")
138138
response1 = requests.post(
139-
f"{ENVOY_URL}{OPENAI_ENDPOINT}", headers=headers, json=payload, timeout=10
139+
f"{ENVOY_URL}{OPENAI_ENDPOINT}", headers=headers, json=payload, timeout=30
140140
)
141141

142142
response1_json = response1.json()
@@ -157,7 +157,7 @@ def test_cache_hit_with_identical_query(self):
157157
# Second identical request
158158
self.print_subtest_header("Second Request (Expected Cache Hit)")
159159
response2 = requests.post(
160-
f"{ENVOY_URL}{OPENAI_ENDPOINT}", headers=headers, json=payload, timeout=10
160+
f"{ENVOY_URL}{OPENAI_ENDPOINT}", headers=headers, json=payload, timeout=30
161161
)
162162

163163
response2_json = response2.json()
@@ -237,7 +237,7 @@ def test_cache_hit_with_similar_query(self):
237237
headers = {"Content-Type": "application/json", "X-Session-ID": session_id}
238238

239239
response1 = requests.post(
240-
f"{ENVOY_URL}{OPENAI_ENDPOINT}", headers=headers, json=payload1, timeout=10
240+
f"{ENVOY_URL}{OPENAI_ENDPOINT}", headers=headers, json=payload1, timeout=30
241241
)
242242

243243
response1_json = response1.json()
@@ -273,7 +273,7 @@ def test_cache_hit_with_similar_query(self):
273273
)
274274

275275
response2 = requests.post(
276-
f"{ENVOY_URL}{OPENAI_ENDPOINT}", headers=headers, json=payload2, timeout=10
276+
f"{ENVOY_URL}{OPENAI_ENDPOINT}", headers=headers, json=payload2, timeout=30
277277
)
278278

279279
response2_json = response2.json()

e2e-tests/06-tools-test.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,7 @@ def test_specific_tool_selection(self):
183183
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
184184
headers=headers,
185185
json=payload,
186-
timeout=10,
186+
timeout=30,
187187
)
188188

189189
# Tool selection should work regardless of vLLM backend availability
@@ -259,7 +259,7 @@ def test_no_tool_requests(self):
259259
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
260260
headers=headers,
261261
json=payload,
262-
timeout=10,
262+
timeout=30,
263263
)
264264

265265
passed = response.status_code in [200, 503]
@@ -331,7 +331,7 @@ def test_tools_configuration_validation(self):
331331
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
332332
headers=headers,
333333
json=payload,
334-
timeout=10,
334+
timeout=30,
335335
)
336336

337337
passed = response.status_code in [200, 503]
@@ -452,7 +452,7 @@ def test_tools_selection_consistency(self):
452452
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
453453
headers=headers,
454454
json=payload,
455-
timeout=10,
455+
timeout=30,
456456
)
457457

458458
# Record the response status for consistency checking

e2e-tests/07-model-selection-test.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ def test_category_based_model_selection(self):
214214
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
215215
headers=headers,
216216
json=payload,
217-
timeout=10,
217+
timeout=30,
218218
)
219219

220220
passed = response.status_code in [200, 503]
@@ -293,7 +293,7 @@ def test_reasoning_mode_selection(self):
293293
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
294294
headers=headers,
295295
json=payload,
296-
timeout=10,
296+
timeout=30,
297297
)
298298

299299
passed = response.status_code in [200, 503]
@@ -365,7 +365,7 @@ def test_model_fallback_behavior(self):
365365
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
366366
headers=headers,
367367
json=payload,
368-
timeout=10,
368+
timeout=30,
369369
)
370370

371371
# Fallback should work, though may get 503 if no vLLM backend
@@ -490,7 +490,7 @@ def test_model_selection_consistency(self):
490490
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
491491
headers=headers,
492492
json=payload,
493-
timeout=10,
493+
timeout=30,
494494
)
495495

496496
# Record the response status and model for consistency checking

e2e-tests/09-error-handling-test.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -244,7 +244,7 @@ def test_malformed_requests(self):
244244
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
245245
headers=headers,
246246
json=test_case["payload"],
247-
timeout=10,
247+
timeout=30,
248248
)
249249

250250
min_status, max_status = test_case["expected_status_range"]
@@ -467,7 +467,7 @@ def test_invalid_content_types(self):
467467
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
468468
headers=headers,
469469
data=test_case["body"],
470-
timeout=10,
470+
timeout=30,
471471
)
472472

473473
# Invalid content types should typically return 4xx errors
@@ -517,7 +517,7 @@ def test_error_response_format(self):
517517
f"{ENVOY_URL}{OPENAI_ENDPOINT}",
518518
headers=headers,
519519
json=payload,
520-
timeout=10,
520+
timeout=30,
521521
)
522522

523523
passed = response.status_code >= 400

0 commit comments

Comments
 (0)