test(compression): trim batch clarify coverage to two invariants
Drop the sentinel-only batch test: a batch sentinel is already rejected by the shared _is_clarify_non_response_sentinel list check that the existing sentinel tests pin, so the case adds no new contract. Also add the contributor email mapping for the cherry-picked commit so release CI can attribute it.
This commit is contained in:
@@ -0,0 +1,2 @@
|
||||
gaoanze888
|
||||
# PR #106089 salvage
|
||||
@@ -266,18 +266,6 @@ class TestSummarizeToolResultClarify:
|
||||
assert "Choice A" in summary
|
||||
assert "Choice B" in summary
|
||||
|
||||
def test_batch_with_only_sentinel_falls_back_to_generic(self):
|
||||
"""A batch where every response is a timeout sentinel must not be quoted as a user answer."""
|
||||
content = json.dumps({
|
||||
"responses": [
|
||||
{"question": "Q?", "user_response": "[user did not respond within 15m]"},
|
||||
]
|
||||
})
|
||||
|
||||
summary = _summarize_tool_result("clarify", "{}", content)
|
||||
|
||||
assert summary == "[clarify] asked user a question"
|
||||
|
||||
|
||||
class TestShouldCompress:
|
||||
def test_below_threshold(self, compressor):
|
||||
|
||||
Reference in New Issue
Block a user