From 225af9734e75d7ea10444b1480fbe5ace236adda Mon Sep 17 00:00:00 2001
From: mkorwel <matt.korwel@gmail.com>
Date: Fri, 17 Apr 2026 22:32:52 +0000
Subject: [PATCH] test: fix ToolGroupMessage tests

---
 .../messages/ToolGroupMessage.test.tsx        | 29 ++++++++++++++++---
 1 file changed, 25 insertions(+), 4 deletions(-)

diff --git a/packages/cli/src/ui/components/messages/ToolGroupMessage.test.tsx b/packages/cli/src/ui/components/messages/ToolGroupMessage.test.tsx
index 94584879f9..98269eeb91 100644
--- a/packages/cli/src/ui/components/messages/ToolGroupMessage.test.tsx
+++ b/packages/cli/src/ui/components/messages/ToolGroupMessage.test.tsx
@@ -115,7 +115,11 @@ describe('<ToolGroupMessage />', () => {
 
       const { lastFrame, unmount } = await renderWithProviders(
         <ToolGroupMessage {...baseProps} item={item} toolCalls={toolCalls} />,
-        { config: baseMockConfig, settings: fullVerbositySettings },
+        {
+          config: baseMockConfig,
+          settings: fullVerbositySettings,
+          allowEmptyFrame: true,
+        },
       );
 
       // Should now hide confirming tools (to avoid duplication with Global Queue)
@@ -458,6 +462,7 @@ describe('<ToolGroupMessage />', () => {
               },
             ],
           },
+          allowEmptyFrame: true,
         },
       );
       expect(lastFrame({ allowEmpty: true })).toMatchSnapshot();
@@ -730,7 +735,11 @@ describe('<ToolGroupMessage />', () => {
 
         const { lastFrame, unmount } = await renderWithProviders(
           <ToolGroupMessage {...baseProps} item={item} toolCalls={toolCalls} />,
-          { config: baseMockConfig, settings: fullVerbositySettings },
+          {
+            config: baseMockConfig,
+            settings: fullVerbositySettings,
+            allowEmptyFrame: shouldHide,
+          },
         );
         if (shouldHide) {
           expect(lastFrame({ allowEmpty: true })).toBe('');
@@ -785,7 +794,11 @@ describe('<ToolGroupMessage />', () => {
           toolCalls={toolCalls}
           borderBottom={false}
         />,
-        { config: baseMockConfig, settings: fullVerbositySettings },
+        {
+          config: baseMockConfig,
+          settings: fullVerbositySettings,
+          allowEmptyFrame: true,
+        },
       );
       // AskUser tools in progress are rendered by AskUserDialog, so we expect nothing.
       expect(lastFrame({ allowEmpty: true })).toBe('');
@@ -815,6 +828,7 @@ describe('<ToolGroupMessage />', () => {
         {
           config: baseMockConfig,
           settings: lowVerbositySettings,
+          allowEmptyFrame: true,
         },
       );
 
@@ -867,6 +881,7 @@ describe('<ToolGroupMessage />', () => {
         {
           config: baseMockConfig,
           settings: fullVerbositySettings,
+          allowEmptyFrame: true,
         },
       );
 
@@ -899,6 +914,7 @@ describe('<ToolGroupMessage />', () => {
         {
           config: baseMockConfig,
           settings: fullVerbositySettings,
+          allowEmptyFrame: true,
         },
       );
 
@@ -956,6 +972,7 @@ describe('<ToolGroupMessage />', () => {
         {
           config: baseMockConfig,
           settings: lowVerbositySettings,
+          allowEmptyFrame: true,
         },
       );
       await secondRender.waitUntilReady();
@@ -1055,7 +1072,11 @@ describe('<ToolGroupMessage />', () => {
 
         const { lastFrame, unmount } = await renderWithProviders(
           <ToolGroupMessage {...baseProps} item={item} toolCalls={toolCalls} />,
-          { config: baseMockConfig, settings: fullVerbositySettings },
+          {
+            config: baseMockConfig,
+            settings: fullVerbositySettings,
+            allowEmptyFrame: !visible,
+          },
         );
 
         if (visible) {