Skip to content

Commit ad5d65e

Browse files
committed
test: remove misleading third test case from task.statusMessage tests
The third test asserted that a stale intermediate statusMessage is preserved through storeTaskResult — which tests InMemoryTaskStore internals, not the bug. The two remaining cases are sufficient regression coverage.
1 parent 2952fa1 commit ad5d65e

1 file changed

Lines changed: 0 additions & 17 deletions

File tree

tests/unit/task.statusMessage.test.ts

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -52,21 +52,4 @@ describe('Task statusMessage after terminal transition', () => {
5252
expect(task!.status).toBe('failed');
5353
expect(task!.statusMessage).toBe('my-tool: failed');
5454
});
55-
56-
it('should show stale statusMessage when final updateTaskStatus is skipped (documents the bug)', async () => {
57-
const store = new InMemoryTaskStore();
58-
const taskId = await createWorkingTask(store);
59-
60-
await store.updateTaskStatus(taskId, 'working', 'Starting the crawler.');
61-
62-
// No final updateTaskStatus — this is the buggy path
63-
await store.storeTaskResult(taskId, 'completed', {
64-
content: [{ type: 'text', text: 'result data' }],
65-
});
66-
67-
const task = await store.getTask(taskId);
68-
expect(task!.status).toBe('completed');
69-
// statusMessage is stale — still the intermediate progress message
70-
expect(task!.statusMessage).toBe('Starting the crawler.');
71-
});
7255
});

0 commit comments

Comments
 (0)