always defer traces for convo and so fix tests to reflect that

This commit is contained in:
lorenzejay
2026-06-09 14:41:58 -07:00
parent 4366587b35
commit 4def6078ad
2 changed files with 7 additions and 0 deletions

View File

@@ -1333,6 +1333,12 @@ class TestFlowTracingWhenSuppressed:
class TestDeferTraceFinalization:
def test_bare_conversational_flow_defers_by_default(self) -> None:
class BareChat(ConversationalFlow):
pass
assert BareChat()._should_defer_trace_finalization() is True
def test_conversation_config_drives_defer_flag(self) -> None:
"""``ConversationConfig(defer_trace_finalization=...)`` controls whether
a conversational subclass defers per-turn trace finalization."""

View File

@@ -221,6 +221,7 @@ def test_flow_definition_includes_conversational_builtins_when_enabled():
assert definition.conversational is not None
assert definition.conversational.enabled is True
assert definition.conversational.defer_trace_finalization is True
assert definition.conversational.builtin_routes == ["converse", "end"]
assert "conversation_start" in methods
assert "route_conversation" in methods