Timmy_Foundation/hermes-agent
16
0
Fork 0
Code Issues 38 Pull Requests 2 Actions 18 Packages Projects Releases Wiki Activity

fix(test): update streaming test to match PR #3566 behavior change (#3574)

Browse Source 
PR #3566 intentionally routes suppressed content to stream_delta_callback
when tool calls are present, so reasoning tag extraction can fire during
streaming. The test was still asserting the old behavior where content
after tool calls was fully suppressed from the callback.

Updated the assertion to match: content IS delivered to the callback
(for tag extraction), with display-level suppression handled by the
CLI's _stream_delta.
This commit is contained in:
Teknium
2026-03-28 13:41:23 -07:00
committed by GitHub
parent 3273732891
commit c0aa06f300
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
tests/test_streaming.py
View File

@@ -362,9 +362,11 @@ class TestStreamingCallbacks:
# Text before tool call IS fired (we don't know yet it will have tools) # Text before tool call IS fired (we don't know yet it will have tools)
assert "thinking..." in deltas assert "thinking..." in deltas
# Text after tool call is NOT fired # Text after tool call IS still routed to stream_delta_callback so that
assert " more text" not in deltas # reasoning tag extraction can fire (PR #3566). Display-level suppression
# But content is still accumulated in the response # of non-reasoning text happens in the CLI's _stream_delta, not here.
assert " more text" in deltas
# Content is still accumulated in the response
assert response.choices[0].message.content == "thinking... more text" assert response.choices[0].message.content == "thinking... more text"