From 104fe9b9e786d695b6e27b4ad5063912d4fa4786 Mon Sep 17 00:00:00 2001
From: Tianyi Cui <53024+tianyicui@users.noreply.github.com>
Date: Mon, 24 Aug 2026 17:16:04 +0800
Subject: [PATCH] test(sdk-app): cover default profile configuration

The startup test helper always constructed an explicit sdk profile, so coverage never exercised apply's supported default-config path even though runtime behavior depended on it.\n\nPass Config objects through the helper and default them to an empty config. The existing startup and help tests now prove sdk defaulting, while the explicit sdk-minimal case remains covered.
---
 packages/bundle/sdk-app/tests/startup.spec.ts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/packages/bundle/sdk-app/tests/startup.spec.ts b/packages/bundle/sdk-app/tests/startup.spec.ts
index 5f0ec6524a..ec128a1c4a 100644
--- a/packages/bundle/sdk-app/tests/startup.spec.ts
+++ b/packages/bundle/sdk-app/tests/startup.spec.ts
@@ -4,7 +4,7 @@ import { EventEmitter } from 'node:events'
 import { Context } from '@deepseek-ai/cordis'
 import { afterEach, describe, expect, it } from 'vitest'
 import { internals, provideCmdline } from '@deepseek-ai/dsh-cmdline'
-import { apply, SDK_APP_STARTUP_SERVICE } from '../src/index.ts'
+import { apply, type Config, SDK_APP_STARTUP_SERVICE } from '../src/index.ts'
 
 /** Controllable stdin for one startup invocation. */
 class TestStdin extends EventEmitter {
@@ -27,7 +27,7 @@ afterEach(() => {
 })
 
 /** Run the provider with captured command output and exit requests. */
-function start(args: string[], profile = 'sdk'): { ctx: Context; exits: number[]; out: () => string; stdin: TestStdin } {
+function start(args: string[], config: Config = {}): { ctx: Context; exits: number[]; out: () => string; stdin: TestStdin } {
   const ctx = new Context()
   const exits: number[] = []
   const stdin = new TestStdin()
@@ -41,7 +41,7 @@ function start(args: string[], profile = 'sdk'): { ctx: Context; exits: number[]
     exit: code => void exits.push(code),
     ready: { onReady: (listener) => { listener(); return () => {} } },
   })
-  apply(ctx, { profile })
+  apply(ctx, config)
   return { ctx, exits, out: () => out, stdin }
 }
 
@@ -64,7 +64,7 @@ describe('SDK app startup', () => {
   })
 
   it('renders the selected SDK profile name in help', () => {
-    const { out } = start(['--help'], 'sdk-minimal')
+    const { out } = start(['--help'], { profile: 'sdk-minimal' })
     expect(out()).toContain('Usage: dsh --profile sdk-minimal')
     expect(out()).toContain('dsh --profile sdk-minimal')
   })