From e04187cf6c2d055da7802331c3834cecf272ad84 Mon Sep 17 00:00:00 2001 From: Viktor Liu Date: Mon, 19 Oct 2026 21:04:00 +0900 Subject: [PATCH] Check the accepted request's error in TestPromptMetadataInjected --- client/internal/approval/broker_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/client/internal/approval/broker_test.go b/client/internal/approval/broker_test.go index 0310f40e1..65f2f56b9 100644 --- a/client/internal/approval/broker_test.go +++ b/client/internal/approval/broker_test.go @@ -309,7 +309,7 @@ func TestPromptMetadataInjected(t *testing.T) { assert.Equal(t, "peerA", ev.Metadata["peer_name"], "caller metadata must pass through") require.True(t, b.Respond(id, Decision{Accept: true})) - <-done + assert.NoError(t, <-done, "accepted request must return no error") } // TestConcurrentRequests verifies that two concurrent prompts are tracked