diff --git a/packages/cli/src/config/config.integration.test.ts b/packages/cli/src/config/config.integration.test.ts index e939a753b..c33bc6b33 100644 --- a/packages/cli/src/config/config.integration.test.ts +++ b/packages/cli/src/config/config.integration.test.ts @@ -15,8 +15,6 @@ import type { import { Config } from '@qwen-code/qwen-code-core'; import { http, HttpResponse } from 'msw'; import { setupServer } from 'msw/node'; -import type { Settings } from './settings.js'; - export const server = setupServer(); // TODO(richieforeman): Consider moving this to test setup globally. @@ -271,7 +269,7 @@ describe('Configuration Integration Tests', () => { 'test', ]; - const argv = await parseArguments({} as Settings); + const argv = await parseArguments(); // Verify that the argument was parsed correctly expect(argv.approvalMode).toBe('auto-edit'); @@ -295,7 +293,7 @@ describe('Configuration Integration Tests', () => { 'test', ]; - const argv = await parseArguments({} as Settings); + const argv = await parseArguments(); expect(argv.approvalMode).toBe('plan'); expect(argv.prompt).toBe('test'); @@ -318,7 +316,7 @@ describe('Configuration Integration Tests', () => { 'test', ]; - const argv = await parseArguments({} as Settings); + const argv = await parseArguments(); expect(argv.approvalMode).toBe('yolo'); expect(argv.prompt).toBe('test'); @@ -341,7 +339,7 @@ describe('Configuration Integration Tests', () => { 'test', ]; - const argv = await parseArguments({} as Settings); + const argv = await parseArguments(); expect(argv.approvalMode).toBe('default'); expect(argv.prompt).toBe('test'); @@ -357,7 +355,7 @@ describe('Configuration Integration Tests', () => { try { process.argv = ['node', 'script.js', '--yolo', '-p', 'test']; - const argv = await parseArguments({} as Settings); + const argv = await parseArguments(); expect(argv.yolo).toBe(true); expect(argv.approvalMode).toBeUndefined(); // Should NOT be set when using --yolo @@ -374,7 +372,7 @@ describe('Configuration Integration Tests', () => { process.argv = ['node', 'script.js', '--approval-mode', 'invalid_mode']; // Should throw during argument parsing due to yargs validation - await expect(parseArguments({} as Settings)).rejects.toThrow(); + await expect(parseArguments()).rejects.toThrow(); } finally { process.argv = originalArgv; } @@ -393,7 +391,7 @@ describe('Configuration Integration Tests', () => { ]; // Should throw during argument parsing due to conflict validation - await expect(parseArguments({} as Settings)).rejects.toThrow(); + await expect(parseArguments()).rejects.toThrow(); } finally { process.argv = originalArgv; } @@ -406,7 +404,7 @@ describe('Configuration Integration Tests', () => { // Test that no approval mode arguments defaults to no flags set process.argv = ['node', 'script.js', '-p', 'test']; - const argv = await parseArguments({} as Settings); + const argv = await parseArguments(); expect(argv.approvalMode).toBeUndefined(); expect(argv.yolo).toBe(false); diff --git a/packages/cli/src/config/config.test.ts b/packages/cli/src/config/config.test.ts index d1d5a1af7..2f34cac4c 100644 --- a/packages/cli/src/config/config.test.ts +++ b/packages/cli/src/config/config.test.ts @@ -2061,7 +2061,7 @@ describe('loadCliConfig fileFiltering', () => { fileFiltering: { [property]: value }, }, }; - const argv = await parseArguments(settings); + const argv = await parseArguments(); const config = await loadCliConfig(settings, argv, undefined, []); expect(getter(config)).toBe(value); },