Fix tests

Author: cte

webview-ui/src/components/chat/IndexingStatusBadge.tsx

@@ -89,6 +89,7 @@ export const IndexingStatusBadge: React.FC<IndexingStatusBadgeProps> = ({ classN
 					<Button
 						variant="ghost"
 						size="sm"
+						aria-label={tooltipText}
 						className={cn(
 							"relative h-7 w-7 p-0",
 							"text-vscode-foreground opacity-85",

webview-ui/src/components/chat/__tests__/ApiConfigSelector.spec.tsx

@@ -76,8 +76,8 @@ describe("ApiConfigSelector", () => {
 		render(<ApiConfigSelector {...defaultProps} />)
 
 		const trigger = screen.getByTestId("dropdown-trigger")
-		// Check for the icon by looking for the codicon span element
-		const icon = trigger.querySelector(".codicon-chevron-up")
+		// Check for the icon by looking for the svg element (ChevronUp from lucide-react renders as svg)
+		const icon = trigger.querySelector("svg")
 		expect(icon).toBeInTheDocument()
 	})
 

webview-ui/src/components/chat/__tests__/ChatView.keyboard-fix.spec.tsx

@@ -73,16 +73,19 @@ vi.mock("react-i18next", () => ({
 }))
 
 vi.mock("../ChatTextArea", () => {
+	const ChatTextAreaComponent = React.forwardRef(function MockChatTextArea(
+		_props: any,
+		ref: React.ForwardedRef<{ focus: () => void }>,
+	) {
+		React.useImperativeHandle(ref, () => ({
+			focus: vi.fn(),
+		}))
+		return <div data-testid="chat-textarea" />
+	})
+
 	return {
-		default: React.forwardRef(function MockChatTextArea(
-			_props: any,
-			ref: React.ForwardedRef<{ focus: () => void }>,
-		) {
-			React.useImperativeHandle(ref, () => ({
-				focus: vi.fn(),
-			}))
-			return <div data-testid="chat-textarea" />
-		}),
+		default: ChatTextAreaComponent,
+		ChatTextArea: ChatTextAreaComponent, // Export as named export too
 	}
 })
 

webview-ui/src/components/chat/__tests__/ChatView.spec.tsx

@@ -181,30 +181,33 @@ vi.mock("../ChatTextArea", () => {
 	// eslint-disable-next-line @typescript-eslint/no-require-imports
 	const mockReact = require("react")
 
+	const ChatTextAreaComponent = mockReact.forwardRef(function MockChatTextArea(
+		props: ChatTextAreaProps,
+		ref: React.ForwardedRef<{ focus: () => void }>,
+	) {
+		// Use useImperativeHandle to expose the mock focus method
+		mockReact.useImperativeHandle(ref, () => ({
+			focus: mockFocus,
+		}))
+
+		return (
+			<div data-testid="chat-textarea">
+				<input
+					ref={mockInputRef}
+					type="text"
+					onChange={(e) => {
+						// With message queueing, onSend is always called (it handles queueing internally)
+						props.onSend(e.target.value)
+					}}
+					data-sending-disabled={props.sendingDisabled}
+				/>
+			</div>
+		)
+	})
+
 	return {
-		default: mockReact.forwardRef(function MockChatTextArea(
-			props: ChatTextAreaProps,
-			ref: React.ForwardedRef<{ focus: () => void }>,
-		) {
-			// Use useImperativeHandle to expose the mock focus method
-			React.useImperativeHandle(ref, () => ({
-				focus: mockFocus,
-			}))
-
-			return (
-				<div data-testid="chat-textarea">
-					<input
-						ref={mockInputRef}
-						type="text"
-						onChange={(e) => {
-							// With message queueing, onSend is always called (it handles queueing internally)
-							props.onSend(e.target.value)
-						}}
-						data-sending-disabled={props.sendingDisabled}
-					/>
-				</div>
-			)
-		}),
+		default: ChatTextAreaComponent,
+		ChatTextArea: ChatTextAreaComponent, // Export as named export too
 	}
 })