Skip to content

Commit 8ba163e

Browse files
committed
fix tests
1 parent c50518b commit 8ba163e

File tree

2 files changed

+9
-8
lines changed

2 files changed

+9
-8
lines changed

tests/test_cloud_tracing.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,7 @@ def test_create_tracer_pro_tier_success(self, capsys):
250250
# Mock upload response
251251
mock_put.return_value = Mock(status_code=200)
252252

253-
tracer = create_tracer(api_key="sk_pro_test123", run_id="test-run")
253+
tracer = create_tracer(api_key="sk_pro_test123", run_id="test-run", upload_trace=True)
254254

255255
# Verify Pro tier message
256256
captured = capsys.readouterr()
@@ -294,7 +294,7 @@ def test_create_tracer_api_forbidden_fallback(self, capsys):
294294
mock_post.return_value = mock_response
295295

296296
with tempfile.TemporaryDirectory():
297-
tracer = create_tracer(api_key="sk_free_test123", run_id="test-run")
297+
tracer = create_tracer(api_key="sk_free_test123", run_id="test-run", upload_trace=True)
298298

299299
# Verify warning message
300300
captured = capsys.readouterr()
@@ -315,7 +315,7 @@ def test_create_tracer_api_timeout_fallback(self, capsys):
315315
mock_post.side_effect = requests.exceptions.Timeout("Connection timeout")
316316

317317
with tempfile.TemporaryDirectory():
318-
tracer = create_tracer(api_key="sk_test123", run_id="test-run")
318+
tracer = create_tracer(api_key="sk_test123", run_id="test-run", upload_trace=True)
319319

320320
# Verify warning message
321321
captured = capsys.readouterr()
@@ -336,7 +336,7 @@ def test_create_tracer_api_connection_error_fallback(self, capsys):
336336
mock_post.side_effect = requests.exceptions.ConnectionError("Connection refused")
337337

338338
with tempfile.TemporaryDirectory():
339-
tracer = create_tracer(api_key="sk_test123", run_id="test-run")
339+
tracer = create_tracer(api_key="sk_test123", run_id="test-run", upload_trace=True)
340340

341341
# Verify warning message
342342
captured = capsys.readouterr()
@@ -371,7 +371,7 @@ def test_create_tracer_uses_constant_api_url(self):
371371
mock_post.return_value = mock_response
372372
mock_put.return_value = Mock(status_code=200)
373373

374-
tracer = create_tracer(api_key="sk_test123", run_id="test-run")
374+
tracer = create_tracer(api_key="sk_test123", run_id="test-run", upload_trace=True)
375375

376376
# Verify correct API URL was used (constant)
377377
assert mock_post.called
@@ -395,7 +395,7 @@ def test_create_tracer_custom_api_url(self):
395395
mock_put.return_value = Mock(status_code=200)
396396

397397
tracer = create_tracer(
398-
api_key="sk_test123", run_id="test-run", api_url=custom_api_url
398+
api_key="sk_test123", run_id="test-run", api_url=custom_api_url, upload_trace=True
399399
)
400400

401401
# Verify custom API URL was used
@@ -415,7 +415,7 @@ def test_create_tracer_missing_upload_url_in_response(self, capsys):
415415
mock_post.return_value = mock_response
416416

417417
with tempfile.TemporaryDirectory():
418-
tracer = create_tracer(api_key="sk_test123", run_id="test-run")
418+
tracer = create_tracer(api_key="sk_test123", run_id="test-run", upload_trace=True)
419419

420420
# Verify warning message
421421
captured = capsys.readouterr()
@@ -463,7 +463,7 @@ def test_create_tracer_orphaned_trace_recovery(self, capsys):
463463
mock_put.return_value = Mock(status_code=200)
464464

465465
# Create tracer - should trigger orphaned trace recovery
466-
tracer = create_tracer(api_key="sk_test123", run_id="new-run-456")
466+
tracer = create_tracer(api_key="sk_test123", run_id="new-run-456", upload_trace=True)
467467

468468
# Verify recovery messages
469469
captured = capsys.readouterr()

tests/test_file_size_tracking.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -186,6 +186,7 @@ def test_create_tracer_passes_logger_to_cloud_sink(self, mock_requests):
186186
api_key="sk_test_key",
187187
run_id="test-logger",
188188
logger=mock_logger,
189+
upload_trace=True,
189190
)
190191

191192
# Verify tracer was created

0 commit comments

Comments
 (0)