test(terminal): fix TerminalRegistry.spec.ts CI failures after getShell() integration (#321)

PR #333 added shellPath and iconPath to Terminal constructor but
TerminalRegistry.spec.ts assertions were not updated, causing 4 test
failures on ubuntu-latest CI.

Author: F915

src/integrations/terminal/__tests__/TerminalRegistry.spec.ts

@@ -3,6 +3,7 @@
 import * as vscode from "vscode"
 import { Terminal } from "../Terminal"
 import { TerminalRegistry } from "../TerminalRegistry"
+import * as shellUtils from "../../../utils/shell"
 
 const PAGER = process.platform === "win32" ? "" : "cat"
 
@@ -34,6 +35,12 @@ describe("TerminalRegistry", () => {
 					},
 				}) as any,
 		)
+
+		vi.spyOn(shellUtils, "getShell").mockReturnValue("/mock/fallback-shell")
+	})
+
+	afterEach(() => {
+		vi.restoreAllMocks()
 	})
 
 	describe("createTerminal", () => {
@@ -43,13 +50,14 @@ describe("TerminalRegistry", () => {
 			expect(mockCreateTerminal).toHaveBeenCalledWith({
 				cwd: "/test/path",
 				name: "Roo Code",
-				iconPath: expect.any(Object),
+				iconPath: expect.objectContaining({ id: expect.any(String) }),
 				env: {
 					PAGER,
 					ROO_ACTIVE: "true",
 					VTE_VERSION: "0",
 					PROMPT_EOL_MARK: "",
 				},
+				shellPath: "/mock/fallback-shell",
 			})
 		})
 
@@ -64,14 +72,15 @@ describe("TerminalRegistry", () => {
 				expect(mockCreateTerminal).toHaveBeenCalledWith({
 					cwd: "/test/path",
 					name: "Roo Code",
-					iconPath: expect.any(Object),
+					iconPath: expect.objectContaining({ id: expect.any(String) }),
 					env: {
 						PAGER,
 						ROO_ACTIVE: "true",
 						PROMPT_COMMAND: "sleep 0.05",
 						VTE_VERSION: "0",
 						PROMPT_EOL_MARK: "",
 					},
+					shellPath: "/mock/fallback-shell",
 				})
 			} finally {
 				// Restore original delay
@@ -87,14 +96,15 @@ describe("TerminalRegistry", () => {
 				expect(mockCreateTerminal).toHaveBeenCalledWith({
 					cwd: "/test/path",
 					name: "Roo Code",
-					iconPath: expect.any(Object),
+					iconPath: expect.objectContaining({ id: expect.any(String) }),
 					env: {
 						PAGER,
 						ROO_ACTIVE: "true",
 						VTE_VERSION: "0",
 						PROMPT_EOL_MARK: "",
 						ITERM_SHELL_INTEGRATION_INSTALLED: "Yes",
 					},
+					shellPath: "/mock/fallback-shell",
 				})
 			} finally {
 				Terminal.setTerminalZshOhMy(false)
@@ -109,14 +119,15 @@ describe("TerminalRegistry", () => {
 				expect(mockCreateTerminal).toHaveBeenCalledWith({
 					cwd: "/test/path",
 					name: "Roo Code",
-					iconPath: expect.any(Object),
+					iconPath: expect.objectContaining({ id: expect.any(String) }),
 					env: {
 						PAGER,
 						ROO_ACTIVE: "true",
 						VTE_VERSION: "0",
 						PROMPT_EOL_MARK: "",
 						POWERLEVEL9K_TERM_SHELL_INTEGRATION: "true",
 					},
+					shellPath: "/mock/fallback-shell",
 				})
 			} finally {
 				Terminal.setTerminalZshP10k(false)