From 2943df96ba5676f6c65b0ae3c59c8bc1825c84e5 Mon Sep 17 00:00:00 2001 From: Julio Quinteros Pro Date: Tue, 3 Mar 2026 19:46:20 -0300 Subject: [PATCH] fix: add video_tokens to expected completion_tokens_details in test The CompletionTokensDetailsWrapper type now includes video_tokens field, but this test's expected dict was not updated to include it. Co-Authored-By: Claude Opus 4.6 --- tests/test_litellm/types/test_types_utils.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/test_litellm/types/test_types_utils.py b/tests/test_litellm/types/test_types_utils.py index 70344950d3e..8c20ace98a0 100644 --- a/tests/test_litellm/types/test_types_utils.py +++ b/tests/test_litellm/types/test_types_utils.py @@ -120,7 +120,8 @@ def test_usage_completion_tokens_details_text_tokens(): 'reasoning_tokens': 65, 'rejected_prediction_tokens': None, 'text_tokens': 12, - 'image_tokens': None + 'image_tokens': None, + 'video_tokens': None } assert dump_result['completion_tokens_details'] == expected_completion_details