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}`.
82 lines
3.2 KiB
TypeScript
82 lines
3.2 KiB
TypeScript
// @vitest-environment jsdom
|
|
/** Chat-store actions, scoped persistence, and instance isolation. */
|
|
import { beforeEach, describe, expect, it } from 'vitest'
|
|
import { createChatStore } from '../src/client/stores.ts'
|
|
|
|
const KEY = 'dsh.conversation.chat'
|
|
|
|
beforeEach(() => {
|
|
localStorage.clear()
|
|
})
|
|
|
|
describe('createChatStore', () => {
|
|
it('init shape: empty selection/draft/view', () => {
|
|
const store = createChatStore().create()
|
|
expect(store.store.getSnapshot()).toEqual({ selection: null, draft: '', view: null, inspect: null })
|
|
})
|
|
|
|
it('actions cover the declared write set', () => {
|
|
const store = createChatStore().create()
|
|
|
|
store.actions.select({ turnSeq: 3, callId: 'c1', toolName: 'bash' })
|
|
expect(store.store.getSnapshot().selection).toEqual({ turnSeq: 3, callId: 'c1', toolName: 'bash' })
|
|
store.actions.select(null)
|
|
expect(store.store.getSnapshot().selection).toBeNull()
|
|
|
|
store.actions.setDraft('hello')
|
|
expect(store.store.getSnapshot().draft).toBe('hello')
|
|
|
|
store.actions.setView('chat')
|
|
expect(store.store.getSnapshot().view).toBe('chat')
|
|
|
|
store.actions.setInspect({ callId: 'c1' })
|
|
expect(store.store.getSnapshot().inspect).toEqual({ callId: 'c1' })
|
|
store.actions.setInspect(null)
|
|
expect(store.store.getSnapshot().inspect).toBeNull()
|
|
})
|
|
|
|
it('persists per scope key and rehydrates a fresh instance', () => {
|
|
const handle = createChatStore()
|
|
const s1 = handle.create('sess-1')
|
|
s1.actions.setDraft('draft for one')
|
|
s1.actions.select({ turnSeq: 1 })
|
|
|
|
// Scope-suffixed key: each session persists separately.
|
|
expect(localStorage.getItem(`${KEY}.sess-1`)).not.toBeNull()
|
|
expect(localStorage.getItem(`${KEY}.sess-2`)).toBeNull()
|
|
|
|
// A rebuilt instance under the same scope key rehydrates the state.
|
|
const again = createChatStore().create('sess-1')
|
|
expect(again.store.getSnapshot().draft).toBe('draft for one')
|
|
expect(again.store.getSnapshot().selection).toEqual({ turnSeq: 1 })
|
|
|
|
// A sibling scope starts clean.
|
|
const other = createChatStore().create('sess-2')
|
|
expect(other.store.getSnapshot().draft).toBe('')
|
|
})
|
|
|
|
it('clearPersisted removes the scope entry (session-death cleanup hook)', () => {
|
|
const store = createChatStore().create('sess-9')
|
|
store.actions.setDraft('doomed')
|
|
expect(localStorage.getItem(`${KEY}.sess-9`)).not.toBeNull()
|
|
store.clearPersisted()
|
|
expect(localStorage.getItem(`${KEY}.sess-9`)).toBeNull()
|
|
})
|
|
|
|
it('every create() is an independent instance; the factory holds no singleton', () => {
|
|
const handle = createChatStore()
|
|
const a = handle.create()
|
|
const b = handle.create()
|
|
a.actions.setDraft('only in a')
|
|
expect(b.store.getSnapshot().draft).toBe('')
|
|
// Two factory calls likewise share no LIVE state (identity is per handle
|
|
// VALUE, not per module — the sharing contract lives in the framework's
|
|
// handle x scope-key resolution, not in module state). Persistence is the
|
|
// one sanctioned cross-instance channel: clear it so this assertion sees
|
|
// memory identity, not rehydration (covered by the persist case above).
|
|
localStorage.clear()
|
|
const c = createChatStore().create()
|
|
expect(c.store.getSnapshot().draft).toBe('')
|
|
})
|
|
})
|