fix: resolve preflight test timeouts and warnings

Mock shouldPromptForTerminalSetup in test environments to prevent action-required UI interruptions. Remove unnecessary dependency in useGeminiStream hook.
This commit is contained in:
Akhilesh Kumar
2026-03-05 18:37:16 +00:00
parent 058c93b9e3
commit 2c11a9263a
3 changed files with 9 additions and 1 deletions

View File

@@ -131,6 +131,11 @@ vi.mock('../ui/components/GeminiRespondingSpinner.js', async () => {
};
});
vi.mock('../ui/utils/terminalSetup.js', () => ({
shouldPromptForTerminalSetup: vi.fn().mockResolvedValue(false),
useTerminalSetupPrompt: vi.fn(),
}));
export interface AppRigOptions {
fakeResponsesPath?: string;
terminalWidth?: number;

View File

@@ -202,6 +202,10 @@ vi.mock('../utils/events.js');
vi.mock('../utils/handleAutoUpdate.js');
vi.mock('./utils/ConsolePatcher.js');
vi.mock('../utils/cleanup.js');
vi.mock('./utils/terminalSetup.js', () => ({
shouldPromptForTerminalSetup: vi.fn().mockResolvedValue(false),
useTerminalSetupPrompt: vi.fn(),
}));
import { useHistory } from './hooks/useHistoryManager.js';
import { useThemeCommand } from './hooks/useThemeCommand.js';

View File

@@ -1799,7 +1799,6 @@ export const useGeminiStream = (
addItem,
registerBackgroundShell,
consumeUserHint,
config,
isLowErrorVerbosity,
maybeAddSuppressedToolErrorNote,
maybeAddLowVerbosityFailureNote,