From fd713090e9c140037d0aaaa6b40c3063d9353a2c Mon Sep 17 00:00:00 2001 From: TrueAlpha-spiral <199723968+TrueAlpha-spiral@users.noreply.github.com> Date: Tue, 17 Mar 2026 20:53:48 +0000 Subject: [PATCH] test(core): fix typescript definition for mockChildProcess stdout/stderr Corrects an invalid direct typecast from EventEmitter to Readable by first casting to unknown, addressing a TypeScript strictness issue in `packages/core/src/services/shellExecutionService.test.ts`. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> --- packages/core/src/services/shellExecutionService.test.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/core/src/services/shellExecutionService.test.ts b/packages/core/src/services/shellExecutionService.test.ts index 2fe51a5e984..21855e45c51 100644 --- a/packages/core/src/services/shellExecutionService.test.ts +++ b/packages/core/src/services/shellExecutionService.test.ts @@ -50,8 +50,8 @@ describe('ShellExecutionService', () => { mockChildProcess = new EventEmitter() as EventEmitter & Partial; // FIX: Cast simple EventEmitters to the expected stream type. - mockChildProcess.stdout = new EventEmitter() as Readable; - mockChildProcess.stderr = new EventEmitter() as Readable; + mockChildProcess.stdout = new EventEmitter() as unknown as Readable; + mockChildProcess.stderr = new EventEmitter() as unknown as Readable; mockChildProcess.kill = vi.fn(); // FIX: Use Object.defineProperty to set the readonly 'pid' property.