Fix test expectations

Author: Tyriar

src/vs/workbench/contrib/terminalContrib/chatAgentTools/test/browser/runInTerminalTool.test.ts

@@ -21,6 +21,7 @@ import { OperatingSystem } from '../../../../../../base/common/platform.js';
 import { Emitter } from '../../../../../../base/common/event.js';
 import { IChatService } from '../../../../chat/common/chatService.js';
 import { ShellIntegrationQuality } from '../../browser/toolTerminalCreator.js';
+import { ITerminalProfileResolverService } from '../../../../terminal/common/terminal.js';
 
 class TestRunInTerminalTool extends RunInTerminalTool {
 	protected override _osBackend: Promise<OperatingSystem> = Promise.resolve(OperatingSystem.Windows);
@@ -67,6 +68,9 @@ suite('RunInTerminalTool', () => {
 		instantiationService.stub(IChatService, {
 			onDidDisposeSession: chatServiceDisposeEmitter.event
 		});
+		instantiationService.stub(ITerminalProfileResolverService, {
+			getDefaultShell: async () => 'pwsh'
+		});
 		workspaceService = instantiationService.invokeFunction(accessor => accessor.get(IWorkspaceContextService)) as TestContextService;
 
 		runInTerminalTool = store.add(instantiationService.createInstance(TestRunInTerminalTool));
@@ -150,7 +154,7 @@ suite('RunInTerminalTool', () => {
 				command: 'rm file.txt',
 				explanation: 'Remove a file'
 			});
-			assertConfirmationRequired(result, 'Run command in terminal');
+			assertConfirmationRequired(result, 'pwsh');
 		});
 
 		test('should require confirmation for commands in deny list even if in allow list', async () => {
@@ -163,7 +167,7 @@ suite('RunInTerminalTool', () => {
 				command: 'rm dangerous-file.txt',
 				explanation: 'Remove a dangerous file'
 			});
-			assertConfirmationRequired(result, 'Run command in terminal');
+			assertConfirmationRequired(result, 'pwsh');
 		});
 
 		test('should handle background commands with confirmation', async () => {
@@ -176,7 +180,7 @@ suite('RunInTerminalTool', () => {
 				explanation: 'Start watching for file changes',
 				isBackground: true
 			});
-			assertConfirmationRequired(result, 'Run command in background terminal');
+			assertConfirmationRequired(result, 'pwsh (background terminal)');
 		});
 
 		test('should auto-approve background commands in allow list', async () => {
@@ -283,7 +287,7 @@ suite('RunInTerminalTool', () => {
 				explanation: 'Build the project'
 			});
 
-			assertConfirmationRequired(result, 'Run command in terminal');
+			assertConfirmationRequired(result, 'pwsh');
 			ok(result!.confirmationMessages!.terminalCustomActions, 'Expected custom actions to be defined');
 
 			const customActions = result!.confirmationMessages!.terminalCustomActions!;
@@ -345,7 +349,7 @@ suite('RunInTerminalTool', () => {
 				explanation: 'Build the project'
 			});
 
-			assertConfirmationRequired(result, 'Run command in terminal');
+			assertConfirmationRequired(result, 'pwsh');
 			ok(result!.confirmationMessages!.terminalCustomActions, 'Expected custom actions to be defined');
 
 			const customActions = result!.confirmationMessages!.terminalCustomActions!;
@@ -361,7 +365,7 @@ suite('RunInTerminalTool', () => {
 				explanation: 'Install dependencies and build'
 			});
 
-			assertConfirmationRequired(result, 'Run command in terminal');
+			assertConfirmationRequired(result, 'pwsh');
 			ok(result!.confirmationMessages!.terminalCustomActions, 'Expected custom actions to be defined');
 
 			const customActions = result!.confirmationMessages!.terminalCustomActions!;