mengzhx/deepseek-harness
1
0
Fork 0
mirror of https://github.com/deepseek-ai/deepseek-harness.git synced 2026-08-29 04:26:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(subagent-acp): skip stdout half-close tests on Windows

Browse Source 
Windows anonymous pipes do not surface a child stdout EOF while the child
process stays alive. The three tests that simulate 'child closes protocol but
stays alive' therefore cannot be reproduced on Windows and hang until the
test timeout. Skip them on win32.
This commit is contained in:
Chinesezjc
2026-08-25 16:36:28 +08:00
parent 5e7c567dc8
commit eea3c132ff
1 changed files with 13 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/subagent/subagent-acp/tests/subagent-acp.spec.ts
+13 -10
View File
@@ -1,4 +1,4 @@
import { describe, expect, it, vi } from 'vitest'
import { describe, expect, it } from 'vitest'
import { Context } from '@deepseek-ai/cordis'
import Loader from '@deepseek-ai/cordis-plugin-loader'
import { chmodSync, existsSync, mkdtempSync, realpathSync, rmSync, writeFileSync } from 'node:fs'
@@ -14,12 +14,6 @@ import { acpStopReason, acpContentText, DEFAULT_DISPOSE_EOF_GRACE_MS, DEFAULT_DI
import LocalSubprocessRuntime from '@deepseek-ai/dsh-subprocess-local'
import { spawnSubprocess } from '@deepseek-ai/dsh-subprocess-local/src/spawn.ts'
// These tests spawn real ACP child subprocesses. On contended self-hosted
// Windows runners the default per-test budget is too tight, so give this file
// twice the configured default timeout (DSH_COVERAGE_TEST_TIMEOUT_MS in CI).
const DEFAULT_TEST_TIMEOUT_MS = Number(process.env.DSH_COVERAGE_TEST_TIMEOUT_MS ?? 5_000)
vi.setConfig({ testTimeout: DEFAULT_TEST_TIMEOUT_MS * 2 })
/**
* Keyless integration tests for the ACP subagent backend. Each spawns a REAL
* subprocess — the scripted mock ACP server (tests/mock-acp-server.ts) — and
@@ -595,7 +589,10 @@ describe('dsh-subagent-acp', () => {
)
})
it('reports initialize-stage transport when the child closes the protocol but stays alive', async () => {
it.skipIf(
process.platform === 'win32',
'Windows anonymous pipes do not surface a child stdout half-close while the child stays alive',
)('reports initialize-stage transport when the child closes the protocol but stays alive', async () => {
const error = await startAcpRun(request(), {
command: process.execPath,
args: [mockServer],
@@ -942,7 +939,10 @@ describe('dsh-subagent-acp', () => {
await run.dispose()
})
it('classifies a prompt transport failure without copying SDK text', async () => {
it.skipIf(
process.platform === 'win32',
'Windows anonymous pipes do not surface a child stdout half-close while the child stays alive',
)('classifies a prompt transport failure without copying SDK text', async () => {
const run = await startAcpRun(request('private prompt text'), {
command: process.execPath,
args: [mockServer],
@@ -963,7 +963,10 @@ describe('dsh-subagent-acp', () => {
await run.dispose()
})
it('lets local cancellation interrupt prompt-failure process observation', async () => {
it.skipIf(
process.platform === 'win32',
'Windows anonymous pipes do not surface a child stdout half-close while the child stays alive',
)('lets local cancellation interrupt prompt-failure process observation', async () => {
const controller = new AbortController()
const protocolEnded = Promise.withResolvers<undefined>()
let boundedExitWaits = 0