DeepSeek/deepseek-harness
1
0
Fork 0
mirror of https://github.com/deepseek-ai/deepseek-harness synced 2026-08-15 21:04:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(snapshot): enforce portable fixture invariants

Browse Source
This commit is contained in:
Tianyi Cui
2026-07-28 21:45:01 +08:00
parent 84d0d92cfb
commit 4cf7603be4
5 changed files with 31 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
examples/jsonrpc-agent/tests/snapshots/bash-tool/session.jsonl
View File

File diff suppressed because one or more lines are too long

2
examples/jsonrpc-agent/tests/snapshots/subagent-spawn/session.1.jsonl
View File

File diff suppressed because one or more lines are too long

2
examples/jsonrpc-agent/tests/snapshots/subagent-spawn/session.jsonl
View File

File diff suppressed because one or more lines are too long

2
examples/jsonrpc-agent/tests/snapshots/text-turn/session.jsonl
View File

File diff suppressed because one or more lines are too long

34
packages/support/acp-snapshot/tests/normalize.spec.ts
View File

@@ -390,6 +390,14 @@ describe('tokenizeSessionFixtureTmpdir', () => {
},
reportedCwd: '/tmp/acp-snap-cwd-abc123',
},
{
name: 'Windows',
context: {
sessionIds: [],
cwd: String.raw`C:\Users\runner\AppData\Local\Temp\acp-snap-cwd-abc123`,
},
reportedCwd: String.raw`C:\Users\runner\AppData\Local\Temp\acp-snap-cwd-abc123`,
},
])('stores $name temporary workspaces with one portable root token', ({ context, reportedCwd }) => {
const raw = [
JSON.stringify({ type: 'session', id: 's', createdAt: 1, cwd: context.cwd }),
@@ -400,7 +408,7 @@ describe('tokenizeSessionFixtureTmpdir', () => {
data: {
content: [{
type: 'text',
text: `wrote ${reportedCwd}/proof.txt. cwd ${context.cwd}. Next; kept ${context.cwd}-backup, ${context.cwd}.backup, and /tmp/authored.txt`,
text: `wrote ${reportedCwd}/proof.txt. alias /different/root/acp-snap-cwd-abc123/alias.txt. cwd ${context.cwd}. Next; kept ${context.cwd}-backup, ${context.cwd}.backup, and /tmp/authored.txt`,
}],
},
}),
@@ -408,14 +416,26 @@ describe('tokenizeSessionFixtureTmpdir', () => {
].join('\n')
const out = tokenizeSessionFixtureTmpdir(raw)
const result = JSON.parse(out.split('\n')[1] as string) as {
data: { content: { text: string }[] }
}
const resultText = (result.data.content[0] as { text: string }).text
expect(out).toContain('"cwd":"{{tmpdir}}/acp-snap-cwd-abc123"')
expect(out).toContain('wrote {{tmpdir}}/acp-snap-cwd-abc123/proof.txt')
expect(out).toContain('cwd {{tmpdir}}/acp-snap-cwd-abc123. Next')
expect(out).toContain(`${context.cwd}-backup`)
expect(out).toContain(`${context.cwd}.backup`)
expect(out).toContain('/tmp/authored.txt')
expect(out).not.toContain(`${reportedCwd}/proof.txt`)
expect(resultText).toContain('wrote {{tmpdir}}/acp-snap-cwd-abc123/proof.txt')
expect(resultText).toContain('alias {{tmpdir}}/acp-snap-cwd-abc123/alias.txt')
expect(resultText).toContain('cwd {{tmpdir}}/acp-snap-cwd-abc123. Next')
expect(resultText).toContain(`${context.cwd}-backup`)
expect(resultText).toContain(`${context.cwd}.backup`)
expect(resultText).toContain('/tmp/authored.txt')
expect(resultText).not.toContain(`${reportedCwd}/proof.txt`)
expect(tokenizeSessionFixtureTmpdir(out)).toBe(out)
})
it('rejects a log without a session cwd', () => {
expect(() => tokenizeSessionFixtureTmpdir('')).toThrow(
'acp-snapshot: cannot tokenize a cwd without a basename',
)
})
})