mirror of
https://github.com/deepseek-ai/deepseek-harness.git
synced 2026-08-29 04:26:38 +00:00
287 lines
10 KiB
TypeScript
287 lines
10 KiB
TypeScript
/** Connection generation readiness, loss, retry, and sink isolation. */
|
|
|
|
import { describe, expect, it, vi } from 'vitest'
|
|
import type { ConnectionGenerationSource, ConnectionState } from '../src/client/connection.ts'
|
|
import { ConnectionController } from '../src/client/connection.ts'
|
|
import { FakeGenerationSource } from './fake-generation.client.ts'
|
|
|
|
const FAST = { backoffBaseMs: 10, backoffFactor: 1, backoffMaxMs: 10, generationReadyTimeoutMs: 500 }
|
|
|
|
describe('connection lifecycle', () => {
|
|
it('announces connected with the Host facts from generation readiness', async () => {
|
|
const source = new FakeGenerationSource()
|
|
const homes: string[] = []
|
|
const controller = new ConnectionController(source.source, {
|
|
onConnected: (host) => { homes.push(host.home) },
|
|
}, FAST)
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(homes).toEqual(['/h']) })
|
|
} finally {
|
|
controller.stop()
|
|
}
|
|
})
|
|
|
|
it('reconnects with a fresh generation when its source fails, and stop() ends the loop', async () => {
|
|
const source = new FakeGenerationSource()
|
|
let connected = 0
|
|
const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined)
|
|
const controller = new ConnectionController(source.source, { onConnected: () => { connected++ } }, FAST)
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(connected).toBe(1) })
|
|
source.fail(new Error('stream torn'))
|
|
await vi.waitFor(() => { expect(connected).toBe(2) })
|
|
expect(source.activeCount).toBe(1)
|
|
} finally {
|
|
controller.stop()
|
|
warnSpy.mockRestore()
|
|
}
|
|
await vi.waitFor(() => { expect(source.activeCount).toBe(0) })
|
|
await new Promise(resolve => setTimeout(resolve, 40))
|
|
expect(source.activeCount).toBe(0)
|
|
})
|
|
|
|
it('isolates a connected sink exception from the generation', async () => {
|
|
const source = new FakeGenerationSource()
|
|
let connected = 0
|
|
const errorSpy = vi.spyOn(console, 'error').mockImplementation(() => undefined)
|
|
const controller = new ConnectionController(source.source, {
|
|
onConnected: () => {
|
|
connected++
|
|
throw new Error('business layer bug')
|
|
},
|
|
}, FAST)
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(connected).toBe(1) })
|
|
expect(source.activeCount).toBe(1)
|
|
expect(errorSpy).toHaveBeenCalledWith('[connection] connection sink threw:', expect.any(Error))
|
|
} finally {
|
|
controller.stop()
|
|
errorSpy.mockRestore()
|
|
}
|
|
})
|
|
|
|
it('holds onConnected until the incremental source reports ready', async () => {
|
|
const source = new FakeGenerationSource()
|
|
source.holdReady = true
|
|
let connected = 0
|
|
const controller = new ConnectionController(source.source, { onConnected: () => { connected++ } }, FAST)
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(source.activeCount).toBe(1) })
|
|
await new Promise(resolve => setTimeout(resolve, 30))
|
|
expect(connected).toBe(0)
|
|
source.releaseReady()
|
|
await vi.waitFor(() => { expect(connected).toBe(1) })
|
|
} finally {
|
|
controller.stop()
|
|
}
|
|
})
|
|
|
|
it('accepts only the first readiness report from one generation', async () => {
|
|
const homes: string[] = []
|
|
const source: ConnectionGenerationSource = (signal, ready) => {
|
|
ready({ home: '/first' })
|
|
ready({ home: '/duplicate' })
|
|
return new Promise<void>((resolve) => {
|
|
signal.addEventListener('abort', () => { resolve() }, { once: true })
|
|
})
|
|
}
|
|
const controller = new ConnectionController(source, {
|
|
onConnected: (host) => { homes.push(host.home) },
|
|
}, FAST)
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(homes).toEqual(['/first']) })
|
|
} finally {
|
|
controller.stop()
|
|
}
|
|
})
|
|
|
|
it('does not announce readiness after a stop queued from the ready callback', async () => {
|
|
const owner: { controller?: ConnectionController } = {}
|
|
let sourceCalls = 0
|
|
const connected = vi.fn()
|
|
const source: ConnectionGenerationSource = (signal, ready) => new Promise<void>((resolve) => {
|
|
sourceCalls++
|
|
ready({ home: '/h' })
|
|
queueMicrotask(() => { owner.controller?.stop() })
|
|
signal.addEventListener('abort', () => { resolve() }, { once: true })
|
|
})
|
|
const controller = new ConnectionController(source, { onConnected: connected }, FAST)
|
|
owner.controller = controller
|
|
controller.start()
|
|
await vi.waitFor(() => { expect(sourceCalls).toBe(1) })
|
|
expect(connected).not.toHaveBeenCalled()
|
|
})
|
|
|
|
it('rejects a generation whose source ends during readiness and retries', async () => {
|
|
const source = new FakeGenerationSource()
|
|
source.holdReady = true
|
|
const states: ConnectionState[] = []
|
|
let connected = 0
|
|
const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined)
|
|
const controller = new ConnectionController(source.source, {
|
|
onConnected: () => { connected++ },
|
|
onStateChange: state => states.push(state),
|
|
}, FAST)
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(source.activeCount).toBe(1) })
|
|
source.holdReady = false
|
|
source.end()
|
|
await vi.waitFor(() => { expect(connected).toBe(1) })
|
|
expect(states).toEqual(['reconnecting', 'connected'])
|
|
} finally {
|
|
controller.stop()
|
|
warnSpy.mockRestore()
|
|
}
|
|
})
|
|
|
|
it.each([
|
|
{ label: 'ends normally', fail: () => Promise.resolve() },
|
|
{
|
|
label: 'rejects with a non-Error reason',
|
|
// oxlint-disable-next-line typescript/prefer-promise-reject-errors -- the non-Error rejection is the scenario under test
|
|
fail: () => Promise.reject('fixture offline'),
|
|
},
|
|
])('retries when the generation source $label before reporting ready', async ({ fail }) => {
|
|
let sourceCalls = 0
|
|
let connected = 0
|
|
const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined)
|
|
const source: ConnectionGenerationSource = (signal, ready) => {
|
|
sourceCalls++
|
|
if (sourceCalls === 1) return fail()
|
|
ready({ home: '/h' })
|
|
return new Promise<void>((resolve) => {
|
|
signal.addEventListener('abort', () => { resolve() }, { once: true })
|
|
})
|
|
}
|
|
const controller = new ConnectionController(source, { onConnected: () => { connected++ } }, FAST)
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(sourceCalls).toBe(2) })
|
|
await vi.waitFor(() => { expect(connected).toBe(1) })
|
|
} finally {
|
|
controller.stop()
|
|
warnSpy.mockRestore()
|
|
}
|
|
})
|
|
|
|
it('rejects and retries a generation whose source never reports ready', async () => {
|
|
const source = new FakeGenerationSource()
|
|
source.suppressReady = true
|
|
let connected = 0
|
|
const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined)
|
|
const controller = new ConnectionController(
|
|
source.source,
|
|
{ onConnected: () => { connected++ } },
|
|
{ ...FAST, generationReadyTimeoutMs: 20 },
|
|
)
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(source.activeCount).toBeGreaterThan(0) })
|
|
await new Promise(resolve => setTimeout(resolve, 45))
|
|
expect(connected).toBe(0)
|
|
} finally {
|
|
controller.stop()
|
|
warnSpy.mockRestore()
|
|
}
|
|
})
|
|
|
|
it('emits deduplicated connected/reconnecting state transitions', async () => {
|
|
const source = new FakeGenerationSource()
|
|
const states: ConnectionState[] = []
|
|
let connected = 0
|
|
const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined)
|
|
const controller = new ConnectionController(source.source, {
|
|
onConnected: () => { connected++ },
|
|
onStateChange: state => states.push(state),
|
|
}, FAST)
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(connected).toBe(1) })
|
|
expect(states).toEqual(['connected'])
|
|
source.fail(new Error('torn'))
|
|
await vi.waitFor(() => { expect(connected).toBe(2) })
|
|
expect(states).toEqual(['connected', 'reconnecting', 'connected'])
|
|
} finally {
|
|
controller.stop()
|
|
warnSpy.mockRestore()
|
|
}
|
|
})
|
|
|
|
it('does not announce a generation stopped synchronously by its connected state sink', async () => {
|
|
const source = new FakeGenerationSource()
|
|
const states: ConnectionState[] = []
|
|
let connected = 0
|
|
const controller = new ConnectionController(source.source, {
|
|
onConnected: () => { connected++ },
|
|
onStateChange: (state) => {
|
|
states.push(state)
|
|
if (state === 'connected') controller.stop()
|
|
},
|
|
}, FAST)
|
|
|
|
controller.start()
|
|
await vi.waitFor(() => { expect(states).toEqual(['connected']) })
|
|
await vi.waitFor(() => { expect(source.activeCount).toBe(0) })
|
|
expect(connected).toBe(0)
|
|
})
|
|
|
|
it('deduplicates consecutive reconnecting emissions across two straight failures', async () => {
|
|
let sourceCalls = 0
|
|
const states: ConnectionState[] = []
|
|
let connected = 0
|
|
const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined)
|
|
const source: ConnectionGenerationSource = (signal, ready) => {
|
|
sourceCalls++
|
|
if (sourceCalls <= 2) return Promise.reject(new Error('down'))
|
|
ready({ home: '/h' })
|
|
return new Promise<void>((resolve) => {
|
|
signal.addEventListener('abort', () => { resolve() }, { once: true })
|
|
})
|
|
}
|
|
const controller = new ConnectionController(source, {
|
|
onConnected: () => { connected++ },
|
|
onStateChange: state => states.push(state),
|
|
}, FAST)
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(sourceCalls).toBe(3) })
|
|
await vi.waitFor(() => { expect(connected).toBe(1) })
|
|
expect(states).toEqual(['reconnecting', 'connected'])
|
|
} finally {
|
|
controller.stop()
|
|
warnSpy.mockRestore()
|
|
}
|
|
})
|
|
|
|
it('runs with no sinks at all', async () => {
|
|
const source = new FakeGenerationSource()
|
|
const controller = new ConnectionController(source.source, {}, FAST)
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(source.activeCount).toBe(1) })
|
|
} finally {
|
|
controller.stop()
|
|
}
|
|
})
|
|
|
|
it('start() is idempotent', async () => {
|
|
const source = new FakeGenerationSource()
|
|
let connected = 0
|
|
const controller = new ConnectionController(source.source, { onConnected: () => { connected++ } }, FAST)
|
|
controller.start()
|
|
controller.start()
|
|
try {
|
|
await vi.waitFor(() => { expect(connected).toBe(1) })
|
|
expect(source.activeCount).toBe(1)
|
|
} finally {
|
|
controller.stop()
|
|
}
|
|
})
|
|
})
|