Skip to content

Commit 9326d98

Browse files
Remove redundant test_text_content_new_format_without_enable_truncation_works
The test is redundant as the same functionality is already covered by test_text_content_both_old_and_new_format_in_sequence() which tests both old and new format TextContent in sequence. Co-authored-by: openhands <openhands@all-hands.dev>
1 parent c8da765 commit 9326d98

1 file changed

Lines changed: 0 additions & 22 deletions

File tree

tests/sdk/llm/test_message.py

Lines changed: 0 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -559,28 +559,6 @@ def test_text_content_old_format_with_enable_truncation_loads_successfully():
559559
assert content.cache_prompt is False
560560

561561

562-
def test_text_content_new_format_without_enable_truncation_works():
563-
"""Test that new event format without enable_truncation works correctly.
564-
565-
This verifies that the new format (without the deprecated field) continues
566-
to work as expected.
567-
"""
568-
from openhands.sdk.llm.message import TextContent
569-
570-
# New event format (without deprecated field)
571-
new_event_text_content = {
572-
"type": "text",
573-
"text": "New format content",
574-
"cache_prompt": True,
575-
}
576-
577-
content = TextContent.model_validate(new_event_text_content)
578-
579-
assert content.text == "New format content"
580-
assert content.type == "text"
581-
assert content.cache_prompt is True
582-
583-
584562
def test_text_content_both_old_and_new_format_in_sequence():
585563
"""Test that both old and new format TextContent can be loaded in sequence.
586564

0 commit comments

Comments
 (0)