Skip to content

Commit 634b6dc

Browse files
committed
Fix test expectations
1 parent 0ec62e3 commit 634b6dc

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

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

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ suite('RunInTerminalTool', () => {
159159
command: 'rm file.txt',
160160
explanation: 'Remove a file'
161161
});
162-
assertConfirmationRequired(result, 'pwsh');
162+
assertConfirmationRequired(result, 'Run `pwsh` command?');
163163
});
164164

165165
test('should require confirmation for commands in deny list even if in allow list', async () => {
@@ -172,7 +172,7 @@ suite('RunInTerminalTool', () => {
172172
command: 'rm dangerous-file.txt',
173173
explanation: 'Remove a dangerous file'
174174
});
175-
assertConfirmationRequired(result, 'pwsh');
175+
assertConfirmationRequired(result, 'Run `pwsh` command?');
176176
});
177177

178178
test('should handle background commands with confirmation', async () => {
@@ -185,7 +185,7 @@ suite('RunInTerminalTool', () => {
185185
explanation: 'Start watching for file changes',
186186
isBackground: true
187187
});
188-
assertConfirmationRequired(result, 'pwsh (background terminal)');
188+
assertConfirmationRequired(result, 'Run `pwsh` command? (background terminal)');
189189
});
190190

191191
test('should auto-approve background commands in allow list', async () => {
@@ -312,7 +312,7 @@ suite('RunInTerminalTool', () => {
312312
explanation: 'Build the project'
313313
});
314314

315-
assertConfirmationRequired(result, 'pwsh');
315+
assertConfirmationRequired(result, 'Run `pwsh` command?');
316316
ok(result!.confirmationMessages!.terminalCustomActions, 'Expected custom actions to be defined');
317317

318318
const customActions = result!.confirmationMessages!.terminalCustomActions!;
@@ -384,7 +384,7 @@ suite('RunInTerminalTool', () => {
384384
explanation: 'Build the project'
385385
});
386386

387-
assertConfirmationRequired(result, 'pwsh');
387+
assertConfirmationRequired(result, 'Run `pwsh` command?');
388388
ok(result!.confirmationMessages!.terminalCustomActions, 'Expected custom actions to be defined');
389389

390390
const customActions = result!.confirmationMessages!.terminalCustomActions!;
@@ -401,7 +401,7 @@ suite('RunInTerminalTool', () => {
401401
explanation: 'Install dependencies and build'
402402
});
403403

404-
assertConfirmationRequired(result, 'pwsh');
404+
assertConfirmationRequired(result, 'Run `pwsh` command?');
405405
ok(result!.confirmationMessages!.terminalCustomActions, 'Expected custom actions to be defined');
406406

407407
const customActions = result!.confirmationMessages!.terminalCustomActions!;
@@ -432,7 +432,7 @@ suite('RunInTerminalTool', () => {
432432
explanation: 'Run foo command and show first 20 lines'
433433
});
434434

435-
assertConfirmationRequired(result, 'pwsh');
435+
assertConfirmationRequired(result, 'Run `pwsh` command?');
436436
ok(result!.confirmationMessages!.terminalCustomActions, 'Expected custom actions to be defined');
437437

438438
const customActions = result!.confirmationMessages!.terminalCustomActions!;
@@ -478,7 +478,7 @@ suite('RunInTerminalTool', () => {
478478
explanation: 'Run multiple piped commands'
479479
});
480480

481-
assertConfirmationRequired(result, 'pwsh');
481+
assertConfirmationRequired(result, 'Run `pwsh` command?');
482482
ok(result!.confirmationMessages!.terminalCustomActions, 'Expected custom actions to be defined');
483483

484484
const customActions = result!.confirmationMessages!.terminalCustomActions!;

0 commit comments

Comments
 (0)