7ad54e7791
A test file under packages/client now says which face it covers:
`*.client.spec.{ts,tsx}` and its `*.client.{ts,tsx}` helpers belong to the
Client aggregate, `*.host.spec.ts` to the host aggregate. The carrier's four
node-half specs take the Host suffix.
The two suffixes are mutually exclusive, so each aggregate excludes the
other's and both keep one broad test glob: `exclude` wins over `include`, and
`packages/client/**` no longer has to be excluded wholesale from the host
program with per-file `files` entries carved back out of it. A Host-face spec
that reaches only Host source therefore needs no cross-face project
reference, which the split-project rule rejects.
vitest still discovers every file through `**/*.spec.{ts,tsx}`.
67 lines
3.0 KiB
TypeScript
67 lines
3.0 KiB
TypeScript
// @vitest-environment jsdom
|
|
import { describe, expect, it, vi } from 'vitest'
|
|
import { stubSettingsScope } from '@deepseek-ai/dsh-client-test-runtime'
|
|
import {
|
|
ComposerSubmissionPolicy, DEFAULT_BUSY_ENTER_BEHAVIOR,
|
|
} from '../src/client/input/submission-policy.ts'
|
|
import type { ConversationSettings } from '../src/submission-settings.ts'
|
|
|
|
describe('ComposerSubmissionPolicy', () => {
|
|
it('defaults to Queue and only applies the preference while running', () => {
|
|
const policy = new ComposerSubmissionPolicy()
|
|
expect(policy.busyEnter.getSnapshot()).toBe(DEFAULT_BUSY_ENTER_BEHAVIOR)
|
|
expect(policy.resolve(false, 'enter', true)).toBe('queue')
|
|
expect(policy.resolve(false, 'accelerated', true)).toBe('queue')
|
|
expect(policy.resolve(true, 'enter', true)).toBe('queue')
|
|
expect(policy.resolve(true, 'accelerated', true)).toBe('steer')
|
|
expect(policy.resolve(true, 'enter', false)).toBe('queue')
|
|
expect(policy.resolve(true, 'accelerated', false)).toBe('queue')
|
|
|
|
const changed = vi.fn()
|
|
policy.busyEnter.subscribe(changed)
|
|
policy.setBusyEnter('steer')
|
|
expect(changed).toHaveBeenCalledTimes(1)
|
|
expect(policy.resolve(true, 'enter', true)).toBe('steer')
|
|
expect(policy.resolve(true, 'accelerated', true)).toBe('queue')
|
|
expect(policy.resolve(false, 'enter', true)).toBe('queue')
|
|
expect(policy.resolve(false, 'accelerated', true)).toBe('queue')
|
|
})
|
|
|
|
it('writes an explicit change through the scope after publishing it locally', () => {
|
|
const host = stubSettingsScope<ConversationSettings>()
|
|
const observed: string[] = []
|
|
let liveBehavior = (): string => 'unconstructed'
|
|
const scope: typeof host.scope = {
|
|
...host.scope,
|
|
set: (field, value) => {
|
|
observed.push(`${field}=${String(value)}:${liveBehavior()}`)
|
|
return host.scope.set(field, value)
|
|
},
|
|
}
|
|
const policy = new ComposerSubmissionPolicy(scope)
|
|
liveBehavior = () => policy.busyEnter.getSnapshot()
|
|
policy.setBusyEnter('steer')
|
|
expect(observed).toEqual(['busyEnter=steer:steer'])
|
|
expect(host.set).toHaveBeenCalledWith('busyEnter', 'steer')
|
|
expect(host.set).toHaveBeenCalledOnce()
|
|
})
|
|
|
|
it('adopts a Host preference without writing it back and leaves an identical write untouched', () => {
|
|
const host = stubSettingsScope<ConversationSettings>()
|
|
const policy = new ComposerSubmissionPolicy(host.scope)
|
|
host.publish({ status: 'ready', value: { busyEnter: 'steer' }, revision: 1, writable: true })
|
|
expect(policy.busyEnter.getSnapshot()).toBe('steer')
|
|
policy.setBusyEnter('steer')
|
|
expect(host.set).not.toHaveBeenCalled()
|
|
host.publish({ value: { busyEnter: 'steer' }, revision: 2 })
|
|
expect(policy.busyEnter.getSnapshot()).toBe('steer')
|
|
})
|
|
|
|
it('adopts a section already standing at construction', () => {
|
|
const host = stubSettingsScope<ConversationSettings>()
|
|
host.publish({ status: 'ready', value: { busyEnter: 'steer' }, revision: 1, writable: true })
|
|
const policy = new ComposerSubmissionPolicy(host.scope)
|
|
expect(policy.busyEnter.getSnapshot()).toBe('steer')
|
|
})
|
|
})
|