105 lines
3.2 KiB
TypeScript
105 lines
3.2 KiB
TypeScript
/**
|
|
* Installer Tests
|
|
*
|
|
* Tests for installer config-writer fixes:
|
|
* - readJsonFile error handling
|
|
*
|
|
* (The CLAUDE.md instructions block is no longer written — see issue
|
|
* #529. The marker-based install/uninstall self-heal is covered in
|
|
* `installer-targets.test.ts`.)
|
|
*/
|
|
|
|
import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
|
|
import * as fs from 'fs';
|
|
import * as path from 'path';
|
|
import * as os from 'os';
|
|
|
|
// We test the exported functions from config-writer
|
|
import {
|
|
writeMcpConfig,
|
|
} from '../src/installer/config-writer';
|
|
|
|
function createTempDir(): string {
|
|
return fs.mkdtempSync(path.join(os.tmpdir(), 'codegraph-installer-test-'));
|
|
}
|
|
|
|
function cleanupTempDir(dir: string): void {
|
|
if (fs.existsSync(dir)) {
|
|
fs.rmSync(dir, { recursive: true, force: true });
|
|
}
|
|
}
|
|
|
|
describe('Installer Config Writer', () => {
|
|
let origCwd: string;
|
|
let tempDir: string;
|
|
|
|
beforeEach(() => {
|
|
tempDir = createTempDir();
|
|
origCwd = process.cwd();
|
|
process.chdir(tempDir);
|
|
});
|
|
|
|
afterEach(() => {
|
|
process.chdir(origCwd);
|
|
cleanupTempDir(tempDir);
|
|
});
|
|
|
|
describe('readJsonFile error handling', () => {
|
|
it('should return empty object for non-existent file', () => {
|
|
// writeMcpConfig reads .mcp.json - if it doesn't exist, it should create it
|
|
writeMcpConfig('local');
|
|
|
|
const mcpJson = path.join(tempDir, '.mcp.json');
|
|
expect(fs.existsSync(mcpJson)).toBe(true);
|
|
|
|
const content = JSON.parse(fs.readFileSync(mcpJson, 'utf-8'));
|
|
expect(content.mcpServers).toBeDefined();
|
|
expect(content.mcpServers.codegraph).toBeDefined();
|
|
});
|
|
|
|
it('should handle corrupted JSON by creating backup', () => {
|
|
// Create a corrupted .mcp.json
|
|
const mcpJson = path.join(tempDir, '.mcp.json');
|
|
fs.writeFileSync(mcpJson, '{ this is not valid json !!!');
|
|
|
|
// Suppress console.warn during test
|
|
const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {});
|
|
|
|
// Should not throw - gracefully handles corruption
|
|
writeMcpConfig('local');
|
|
|
|
// Should have warned
|
|
expect(warnSpy).toHaveBeenCalled();
|
|
const warnMsg = warnSpy.mock.calls[0][0];
|
|
expect(warnMsg).toContain('Warning');
|
|
|
|
// Backup should exist
|
|
expect(fs.existsSync(mcpJson + '.backup')).toBe(true);
|
|
// Original backup content should be the corrupted content
|
|
const backup = fs.readFileSync(mcpJson + '.backup', 'utf-8');
|
|
expect(backup).toContain('this is not valid json');
|
|
|
|
// New file should be valid JSON with codegraph config
|
|
const content = JSON.parse(fs.readFileSync(mcpJson, 'utf-8'));
|
|
expect(content.mcpServers.codegraph).toBeDefined();
|
|
|
|
warnSpy.mockRestore();
|
|
});
|
|
|
|
it('should preserve existing valid config when adding codegraph', () => {
|
|
const mcpJson = path.join(tempDir, '.mcp.json');
|
|
fs.writeFileSync(mcpJson, JSON.stringify({
|
|
mcpServers: { other: { command: 'other-tool' } },
|
|
customField: 'preserved',
|
|
}, null, 2));
|
|
|
|
writeMcpConfig('local');
|
|
|
|
const content = JSON.parse(fs.readFileSync(mcpJson, 'utf-8'));
|
|
expect(content.mcpServers.codegraph).toBeDefined();
|
|
expect(content.mcpServers.other).toBeDefined();
|
|
expect(content.customField).toBe('preserved');
|
|
});
|
|
});
|
|
});
|