Skip to content

Commit 9c2dff6

Browse files
committed
Update plan review action label tests to match new labels
1 parent 5ff5e9b commit 9c2dff6

1 file changed

Lines changed: 8 additions & 8 deletions

File tree

extensions/copilot/src/extension/chatSessions/copilotcli/node/test/exitPlanModeHandler.spec.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -197,43 +197,43 @@ describe('handleExitPlanMode', () => {
197197
});
198198

199199
it('returns approved with selected action mapped from label', async () => {
200-
toolService.setResult({ rejected: false, action: 'Autopilot' });
200+
toolService.setResult({ rejected: false, action: 'Implement with Autopilot' });
201201
const event = makeEvent();
202202
const result = await handleExitPlanMode(event, session as unknown as Session, 'interactive', FAKE_TOKEN, workspaceService, logService, toolService, CANCEL_TOKEN);
203203
expect(result).toEqual<ExitPlanModeResponse>({ approved: true, selectedAction: 'autopilot', autoApproveEdits: undefined });
204204
});
205205

206-
it('maps "Approve and exit" label to exit_only', async () => {
207-
toolService.setResult({ rejected: false, action: 'Approve' });
206+
it('maps "Approve Plan Only" label to exit_only', async () => {
207+
toolService.setResult({ rejected: false, action: 'Approve Plan Only' });
208208
const event = makeEvent();
209209
const result = await handleExitPlanMode(event, session as unknown as Session, 'interactive', FAKE_TOKEN, workspaceService, logService, toolService, CANCEL_TOKEN);
210210
expect(result.selectedAction).toBe('exit_only');
211211
});
212212

213213
it('sets autoApproveEdits when permissionLevel is autoApprove', async () => {
214-
toolService.setResult({ rejected: false, action: 'Interactive' });
214+
toolService.setResult({ rejected: false, action: 'Implement Plan' });
215215
const event = makeEvent();
216216
const result = await handleExitPlanMode(event, session as unknown as Session, 'autoApprove', FAKE_TOKEN, workspaceService, logService, toolService, CANCEL_TOKEN);
217217
expect(result.autoApproveEdits).toBe(true);
218218
});
219219

220220
it('does not set autoApproveEdits when permissionLevel is interactive', async () => {
221-
toolService.setResult({ rejected: false, action: 'Interactive' });
221+
toolService.setResult({ rejected: false, action: 'Implement Plan' });
222222
const event = makeEvent();
223223
const result = await handleExitPlanMode(event, session as unknown as Session, 'interactive', FAKE_TOKEN, workspaceService, logService, toolService, CANCEL_TOKEN);
224224
expect(result.autoApproveEdits).toBeUndefined();
225225
});
226226

227227
it('passes actions with labels and recommended flag to tool', async () => {
228-
toolService.setResult({ rejected: false, action: 'Interactive' });
228+
toolService.setResult({ rejected: false, action: 'Implement Plan' });
229229
const event = makeEvent({ actions: ['autopilot', 'exit_only'], recommendedAction: 'exit_only' });
230230
await handleExitPlanMode(event, session as unknown as Session, 'interactive', FAKE_TOKEN, workspaceService, logService, toolService, CANCEL_TOKEN);
231231
const call = toolService.invokeToolCalls[0];
232232
expect(call.name).toBe('vscode_reviewPlan');
233233
const input = call.input as any;
234234
expect(input.actions).toHaveLength(2);
235-
expect(input.actions[0]).toEqual(expect.objectContaining({ label: 'Autopilot', default: false }));
236-
expect(input.actions[1]).toEqual(expect.objectContaining({ label: 'Approve', default: true }));
235+
expect(input.actions[0]).toEqual(expect.objectContaining({ label: 'Implement with Autopilot', default: false }));
236+
expect(input.actions[1]).toEqual(expect.objectContaining({ label: 'Approve Plan Only', default: true }));
237237
});
238238

239239
it('includes plan path in tool input when plan path exists', async () => {

0 commit comments

Comments
 (0)