| |
| |
| |
| |
| |
|
|
| import { vi, describe, it, expect, beforeEach, afterEach } from 'vitest'; |
| import { handleEnable, enableCommand } from './enable.js'; |
| import { |
| loadSettings, |
| SettingScope, |
| type LoadedSettings, |
| } from '../../config/settings.js'; |
|
|
| const { emitConsoleLog, debugLogger } = await vi.hoisted(async () => { |
| const { createMockDebugLogger } = await import( |
| '../../test-utils/mockDebugLogger.js' |
| ); |
| return createMockDebugLogger({ stripAnsi: true }); |
| }); |
|
|
| vi.mock('@google/gemini-cli-core', async (importOriginal) => { |
| const actual = |
| await importOriginal<typeof import('@google/gemini-cli-core')>(); |
| return { |
| ...actual, |
| debugLogger, |
| }; |
| }); |
|
|
| vi.mock('../../config/settings.js', async (importOriginal) => { |
| const actual = |
| await importOriginal<typeof import('../../config/settings.js')>(); |
| return { |
| ...actual, |
| loadSettings: vi.fn(), |
| isLoadableSettingScope: vi.fn((s) => s === 'User' || s === 'Workspace'), |
| }; |
| }); |
|
|
| vi.mock('../utils.js', () => ({ |
| exitCli: vi.fn(), |
| })); |
|
|
| describe('skills enable command', () => { |
| const mockLoadSettings = vi.mocked(loadSettings); |
|
|
| beforeEach(() => { |
| vi.clearAllMocks(); |
| }); |
|
|
| afterEach(() => { |
| vi.restoreAllMocks(); |
| }); |
|
|
| describe('handleEnable', () => { |
| it('should enable a disabled skill in user scope', async () => { |
| const mockSettings = { |
| forScope: vi.fn().mockImplementation((scope) => { |
| if (scope === SettingScope.User) { |
| return { |
| settings: { skills: { disabled: ['skill1'] } }, |
| path: '/user/settings.json', |
| }; |
| } |
| return { settings: {}, path: '/workspace/settings.json' }; |
| }), |
| setValue: vi.fn(), |
| }; |
| mockLoadSettings.mockReturnValue( |
| mockSettings as unknown as LoadedSettings, |
| ); |
|
|
| await handleEnable({ name: 'skill1' }); |
|
|
| expect(mockSettings.setValue).toHaveBeenCalledWith( |
| SettingScope.User, |
| 'skills.disabled', |
| [], |
| ); |
| expect(emitConsoleLog).toHaveBeenCalledWith( |
| 'log', |
| 'Skill "skill1" enabled by removing it from the disabled list in user (/user/settings.json) and workspace (/workspace/settings.json) settings.', |
| ); |
| }); |
|
|
| it('should enable a skill across multiple scopes', async () => { |
| const mockSettings = { |
| forScope: vi.fn().mockImplementation((scope) => { |
| if (scope === SettingScope.User) { |
| return { |
| settings: { skills: { disabled: ['skill1'] } }, |
| path: '/user/settings.json', |
| }; |
| } |
| if (scope === SettingScope.Workspace) { |
| return { |
| settings: { skills: { disabled: ['skill1'] } }, |
| path: '/workspace/settings.json', |
| }; |
| } |
| return { settings: {}, path: '' }; |
| }), |
| setValue: vi.fn(), |
| }; |
| mockLoadSettings.mockReturnValue( |
| mockSettings as unknown as LoadedSettings, |
| ); |
|
|
| await handleEnable({ name: 'skill1' }); |
|
|
| expect(mockSettings.setValue).toHaveBeenCalledWith( |
| SettingScope.User, |
| 'skills.disabled', |
| [], |
| ); |
| expect(mockSettings.setValue).toHaveBeenCalledWith( |
| SettingScope.Workspace, |
| 'skills.disabled', |
| [], |
| ); |
| expect(emitConsoleLog).toHaveBeenCalledWith( |
| 'log', |
| 'Skill "skill1" enabled by removing it from the disabled list in workspace (/workspace/settings.json) and user (/user/settings.json) settings.', |
| ); |
| }); |
|
|
| it('should log a message if the skill is already enabled', async () => { |
| const mockSettings = { |
| forScope: vi.fn().mockReturnValue({ |
| settings: { skills: { disabled: [] } }, |
| path: '/user/settings.json', |
| }), |
| setValue: vi.fn(), |
| }; |
| mockLoadSettings.mockReturnValue( |
| mockSettings as unknown as LoadedSettings, |
| ); |
|
|
| await handleEnable({ name: 'skill1' }); |
|
|
| expect(mockSettings.setValue).not.toHaveBeenCalled(); |
| expect(emitConsoleLog).toHaveBeenCalledWith( |
| 'log', |
| 'Skill "skill1" is already enabled.', |
| ); |
| }); |
| }); |
|
|
| describe('enableCommand', () => { |
| it('should have correct command and describe', () => { |
| expect(enableCommand.command).toBe('enable <name>'); |
| expect(enableCommand.describe).toBe('Enables an agent skill.'); |
| }); |
| }); |
| }); |
|
|
