From 752852800733ee1869f3053391f60cd2f2180fae Mon Sep 17 00:00:00 2001 From: Tianyi Cui <53024+tianyicui@users.noreply.github.com> Date: Sat, 19 Sep 2026 14:38:20 +0800 Subject: [PATCH] test(session): name the wrapper pass-through assertion precisely This case calls only liftToolResult; the native validator has separate coverage. Name the behavior actually asserted so the test title does not claim both paths are covered by one assertion. Assertions and production behavior are unchanged. --- .../session/session-format-v3-to-v4/tests/tool-role.spec.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/session/session-format-v3-to-v4/tests/tool-role.spec.ts b/packages/session/session-format-v3-to-v4/tests/tool-role.spec.ts index 9b3b14cb41..8e753cb3ff 100644 --- a/packages/session/session-format-v3-to-v4/tests/tool-role.spec.ts +++ b/packages/session/session-format-v3-to-v4/tests/tool-role.spec.ts @@ -157,7 +157,7 @@ describe('V4 tool-role view and lift', () => { expect(() => liftToolResult(row)).toThrow(/tool-result isError must be boolean/) }) - it('passes non-tool rows through both transforms', () => { + it('passes non-tool rows through the wrapper conversion', () => { const row = event(0, 'user/message', { content: [], source: { kind: 'user' }, role: 'user', id: 'u' }) expect(liftToolResult(row)).toBe(row) })