MediaMetz/gemini-cli
1
0
Fork 0
mirror of https://github.com/google-gemini/gemini-cli.git synced 2026-03-10 22:21:22 -07:00
Code Issues Packages Projects Releases Wiki Activity

Fix test in windows (#12681)

Browse Source
This commit is contained in:
Tommaso Sciortino
2025-11-06 20:01:30 -08:00
committed by GitHub
parent 054497c7a6
commit 475e92da5d
2 changed files with 10 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
packages/cli/src/config/config.test.ts
View File

@@ -36,7 +36,7 @@ vi.mock('./sandboxConfig.js', () => ({
vi.mock('fs', async (importOriginal) => {
const actualFs = await importOriginal<typeof import('fs')>();
const pathMod = await import('node:path');
const mockHome = '/mock/home/user';
const mockHome = pathMod.resolve(pathMod.sep, 'mock', 'home', 'user');
const MOCK_CWD1 = process.cwd();
const MOCK_CWD2 = pathMod.resolve(pathMod.sep, 'home', 'user', 'project');
@@ -69,7 +69,7 @@ vi.mock('os', async (importOriginal) => {
const actualOs = await importOriginal<typeof os>();
return {
...actualOs,
homedir: vi.fn(() => '/mock/home/user'),
homedir: vi.fn(() => path.resolve(path.sep, 'mock', 'home', 'user')),
};
});
@@ -1480,7 +1480,9 @@ describe('loadCliConfig folderTrust', () => {
describe('loadCliConfig with includeDirectories', () => {
beforeEach(() => {
vi.resetAllMocks();
vi.mocked(os.homedir).mockReturnValue('/mock/home/user');
vi.mocked(os.homedir).mockReturnValue(
path.resolve(path.sep, 'mock', 'home', 'user'),
);
vi.stubEnv('GEMINI_API_KEY', 'test-api-key');
vi.spyOn(process, 'cwd').mockReturnValue(
path.resolve(path.sep, 'home', 'user', 'project'),

10
packages/core/src/config/config.test.ts
View File

@@ -391,7 +391,8 @@ describe('Server Config (config.ts)', () => {
});
it('should initialize WorkspaceContext with includeDirectories', () => {
const includeDirectories = ['/path/to/dir1', '/path/to/dir2'];
const resolved = path.resolve(baseParams.targetDir);
const includeDirectories = ['dir1', 'dir2'];
const paramsWithIncludeDirs: ConfigParameters = {
...baseParams,
includeDirectories,
@@ -399,12 +400,11 @@ describe('Server Config (config.ts)', () => {
const config = new Config(paramsWithIncludeDirs);
const workspaceContext = config.getWorkspaceContext();
const directories = workspaceContext.getDirectories();
// Should include the target directory plus the included directories
expect(directories).toHaveLength(3);
expect(directories).toContain(path.resolve(baseParams.targetDir));
expect(directories).toContain('/path/to/dir1');
expect(directories).toContain('/path/to/dir2');
expect(directories).toContain(resolved);
expect(directories).toContain(path.join(resolved, 'dir1'));
expect(directories).toContain(path.join(resolved, 'dir2'));
});
it('Config constructor should set telemetry to true when provided as true', () => {