vrr/qwen-code
2
0
Fork 0
mirror of https://github.com/QwenLM/qwen-code.git synced 2026-05-02 21:50:52 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

fix(review): fix misleading test name and comment

Browse source 
Rename "should use unknown when model is not available" to
"when getModel returns undefined" — the mock config does define
getModel, it just returns undefined.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
wenshao 2026-04-06 23:47:59 +08:00
parent 3ac54581a8
commit 71733b5dcb
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
packages/cli/src/services/BundledSkillLoader.test.ts
View file

@ -150,12 +150,12 @@ describe('BundledSkillLoader', () => {
});
});
it('should use "unknown" when model is not available for {{model}}', async () => {
it('should use "unknown" for {{model}} when getModel returns undefined', async () => {
const skill = makeSkill({
body: 'Review by {{model}}',
});
mockSkillManager.listSkills.mockResolvedValue([skill]);
// No getModel on config
// getModel returns undefined (default mock behavior)
const loader = new BundledSkillLoader(mockConfig);
const commands = await loader.loadCommands(signal);