Skip to content
This repository was archived by the owner on May 15, 2026. It is now read-only.

Commit ed8b991

Browse files
committed
fix: add missing log mock to provider test fixtures for reopenParentFromDelegation
1 parent d74971a commit ed8b991

2 files changed

Lines changed: 11 additions & 0 deletions

File tree

src/__tests__/history-resume-delegation.spec.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,7 @@ describe("History resume delegation - parent metadata transitions", () => {
7373

7474
const provider = {
7575
contextProxy: { globalStorageUri: { fsPath: "/tmp" } },
76+
log: vi.fn(),
7677
getTaskWithId,
7778
emit: providerEmit,
7879
getCurrentTask: vi.fn(() => ({ taskId: "child-1" })),
@@ -122,6 +123,7 @@ describe("History resume delegation - parent metadata transitions", () => {
122123
it("reopenParentFromDelegation injects subtask_result into both UI and API histories", async () => {
123124
const provider = {
124125
contextProxy: { globalStorageUri: { fsPath: "/storage" } },
126+
log: vi.fn(),
125127
getTaskWithId: vi.fn().mockResolvedValue({
126128
historyItem: {
127129
id: "p1",
@@ -205,6 +207,7 @@ describe("History resume delegation - parent metadata transitions", () => {
205207
it("reopenParentFromDelegation injects tool_result when new_task tool_use exists in API history", async () => {
206208
const provider = {
207209
contextProxy: { globalStorageUri: { fsPath: "/storage" } },
210+
log: vi.fn(),
208211
getTaskWithId: vi.fn().mockResolvedValue({
209212
historyItem: {
210213
id: "p-tool",
@@ -291,6 +294,7 @@ describe("History resume delegation - parent metadata transitions", () => {
291294
it("reopenParentFromDelegation injects plain text when no new_task tool_use exists in API history", async () => {
292295
const provider = {
293296
contextProxy: { globalStorageUri: { fsPath: "/storage" } },
297+
log: vi.fn(),
294298
getTaskWithId: vi.fn().mockResolvedValue({
295299
historyItem: {
296300
id: "p-no-tool",
@@ -351,6 +355,7 @@ describe("History resume delegation - parent metadata transitions", () => {
351355

352356
const provider = {
353357
contextProxy: { globalStorageUri: { fsPath: "/tmp" } },
358+
log: vi.fn(),
354359
getTaskWithId: vi.fn().mockResolvedValue({
355360
historyItem: {
356361
id: "parent-2",
@@ -391,6 +396,7 @@ describe("History resume delegation - parent metadata transitions", () => {
391396

392397
const provider = {
393398
contextProxy: { globalStorageUri: { fsPath: "/tmp" } },
399+
log: vi.fn(),
394400
getTaskWithId: vi.fn().mockResolvedValue({
395401
historyItem: {
396402
id: "p3",
@@ -457,6 +463,7 @@ describe("History resume delegation - parent metadata transitions", () => {
457463

458464
const provider = {
459465
contextProxy: { globalStorageUri: { fsPath: "/tmp" } },
466+
log: vi.fn(),
460467
getTaskWithId: vi.fn().mockImplementation(async (id: string) => {
461468
if (id === "parent-rpd06") {
462469
return {
@@ -527,6 +534,7 @@ describe("History resume delegation - parent metadata transitions", () => {
527534

528535
const provider = {
529536
contextProxy: { globalStorageUri: { fsPath: "/tmp" } },
537+
log: vi.fn(),
530538
getTaskWithId: vi.fn().mockResolvedValue({
531539
historyItem: {
532540
id: "p4",
@@ -580,6 +588,7 @@ describe("History resume delegation - parent metadata transitions", () => {
580588

581589
const provider = {
582590
contextProxy: { globalStorageUri: { fsPath: "/tmp" } },
591+
log: vi.fn(),
583592
getTaskWithId: vi.fn().mockImplementation(async (id: string) => {
584593
if (id === "parent-rpd02") {
585594
return {
@@ -726,6 +735,7 @@ describe("History resume delegation - parent metadata transitions", () => {
726735
it("handles empty history gracefully when injecting synthetic messages", async () => {
727736
const provider = {
728737
contextProxy: { globalStorageUri: { fsPath: "/tmp" } },
738+
log: vi.fn(),
729739
getTaskWithId: vi.fn().mockResolvedValue({
730740
historyItem: {
731741
id: "p5",

src/__tests__/nested-delegation-resume.spec.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -138,6 +138,7 @@ describe("Nested delegation resume (A → B → C)", () => {
138138

139139
const provider = {
140140
contextProxy: { globalStorageUri: { fsPath: "/tmp" } },
141+
log: vi.fn(),
141142
getTaskWithId,
142143
emit: emitSpy,
143144
getCurrentTask: vi.fn(() => (currentActiveId ? ({ taskId: currentActiveId } as any) : undefined)),

0 commit comments

Comments
 (0)