vrr/qwen-code
2
0
Fork 0
mirror of https://github.com/QwenLM/qwen-code.git synced 2026-05-05 23:42:03 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 11

test: add explicit assertions for setApprovalMode argument

Browse source 
Verify the exact mode value passed to config.setApprovalMode to catch
potential regressions in settings merge/update mechanism.
This commit is contained in:
Tu Shaokun 2026-01-01 20:29:38 +08:00
parent 8fcdd86b91
commit 2f2937aafe
1 changed files with 9 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

11
packages/cli/src/ui/commands/approvalModeCommand.test.ts
View file

@ -30,7 +30,9 @@ describe('approvalModeCommand', () => {
setApprovalMode: mockSetApprovalMode,
},
settings: {
merged: { tools: { approvalMode: 'default' } },
// Use empty merged so ?? fallback triggers, allowing us to verify
// the exact mode passed to setApprovalMode
merged: {},
setValue: mockSetValue,
forScope: () => ({}),
} as unknown as LoadedSettings,
@ -81,7 +83,7 @@ describe('approvalModeCommand', () => {
'tools.approvalMode',
'plan',
);
expect(mockSetApprovalMode).toHaveBeenCalled();
expect(mockSetApprovalMode).toHaveBeenCalledWith('plan');
});
it('should set approval mode to "yolo" when argument is "yolo"', async () => {
@ -98,6 +100,7 @@ describe('approvalModeCommand', () => {
'tools.approvalMode',
'yolo',
);
expect(mockSetApprovalMode).toHaveBeenCalledWith('yolo');
});
it('should set approval mode to "auto-edit" when argument is "auto-edit"', async () => {
@ -114,6 +117,7 @@ describe('approvalModeCommand', () => {
'tools.approvalMode',
'auto-edit',
);
expect(mockSetApprovalMode).toHaveBeenCalledWith('auto-edit');
});
it('should set approval mode to "default" when argument is "default"', async () => {
@ -130,6 +134,7 @@ describe('approvalModeCommand', () => {
'tools.approvalMode',
'default',
);
expect(mockSetApprovalMode).toHaveBeenCalledWith('default');
});
it('should be case-insensitive for mode argument', async () => {
@ -145,6 +150,7 @@ describe('approvalModeCommand', () => {
'tools.approvalMode',
'yolo',
);
expect(mockSetApprovalMode).toHaveBeenCalledWith('yolo');
});
it('should handle argument with leading/trailing whitespace', async () => {
@ -160,6 +166,7 @@ describe('approvalModeCommand', () => {
'tools.approvalMode',
'plan',
);
expect(mockSetApprovalMode).toHaveBeenCalledWith('plan');
});
});