From 74982efc60bb4b5ca49a2efdf7698f0f98991517 Mon Sep 17 00:00:00 2001 From: Martin Vogel Date: Tue, 27 Oct 2026 23:41:04 +0100 Subject: [PATCH] test(cli): restore Pi abort-kill assertion dropped alongside #2787 test The #2787 change added client_adapter_pi_emits_string_enum_issue2787 directly below client_adapter_pi_wraps_result_and_honors_abort and, in the same hunk, removed that test's check that the generated abort handler kills the cli child (if (!child.killed) child.kill();). The generator still emits the line; restore the assertion byte-for-byte so the abort path keeps its coverage. Signed-off-by: Martin Vogel --- tests/test_agent_clients.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/test_agent_clients.c b/tests/test_agent_clients.c index 6d134a8b8..0d57723a0 100644 --- a/tests/test_agent_clients.c +++ b/tests/test_agent_clients.c @@ -1243,6 +1243,7 @@ TEST(client_adapter_pi_wraps_result_and_honors_abort) { ASSERT_NOT_NULL( strstr(js, "[{ type: 'text', text: JSON.stringify(result ?? null, null, 2) }]")); ASSERT_NOT_NULL(strstr(js, "signal?.addEventListener('abort', onAbort, { once: true })")); + ASSERT_NOT_NULL(strstr(js, "if (!child.killed) child.kill();")); free(js); PASS(); }