Files
repomix-mirror/tests/config/configLoad.integration.test.ts
Kazuki Yamada 247221b472 fix(test): Stabilize test coverage by preventing jiti double instrumentation
This commit resolves test coverage instability that occurred after
implementing JS/TS config support. Coverage was fluctuating between
68-71% on each test run, caused by jiti dynamically transforming src/
files that were already instrumented by Vitest for coverage tracking.

Root Cause:
- Test fixture files imported `defineConfig` from src/index.js
- jiti transformed the entire src/ directory when loading fixtures
- Same files existed in two versions (Vitest-instrumented vs jiti-transformed)
- v8 coverage tracker produced non-deterministic results

Solution:
1. Added DI pattern to loadFileConfig with defaultJitiImport factory
2. Removed defineConfig imports from all test fixtures (use plain objects)
3. Mock jiti in dynamic config tests to prevent src/ transformation

Results:
- Coverage now stable at 89.34% (previously 68-71%)
- Coverage improved by ~19% due to accurate instrumentation
- Integration test runtime improved from 3.8s to 0.1s
- All tests pass with zero lint errors

Related Changes:
- Reverted temporary Vitest config workarounds (sequence.shuffle, isolate)
- Extracted defaultJitiImport as reusable factory function
2025-10-31 01:18:21 +09:00

153 lines
4.8 KiB
TypeScript

import path from 'node:path';
import { describe, expect, test } from 'vitest';
import { loadFileConfig } from '../../src/config/configLoad.js';
describe('configLoad Integration Tests', () => {
const jsFixturesDir = path.join(process.cwd(), 'tests/fixtures/config-js');
const tsFixturesDir = path.join(process.cwd(), 'tests/fixtures/config-ts');
describe('TypeScript Config Files', () => {
test('should load .ts config with ESM default export', async () => {
const config = await loadFileConfig(tsFixturesDir, 'repomix.config.ts');
expect(config).toEqual({
output: {
filePath: 'ts-output.xml',
style: 'xml',
removeComments: true,
},
ignore: {
customPatterns: ['**/node_modules/**', '**/dist/**'],
},
});
});
test('should load .mts config', async () => {
const config = await loadFileConfig(tsFixturesDir, 'repomix.config.mts');
expect(config).toEqual({
output: {
filePath: 'mts-output.xml',
style: 'xml',
},
ignore: {
customPatterns: ['**/test/**'],
},
});
});
test('should load .cts config', async () => {
const config = await loadFileConfig(tsFixturesDir, 'repomix.config.cts');
expect(config).toEqual({
output: {
filePath: 'cts-output.xml',
style: 'plain',
},
ignore: {
customPatterns: ['**/build/**'],
},
});
});
test('should handle dynamic values in TypeScript config', async () => {
// Mock jiti to avoid coverage instability caused by dynamic module loading
// This ensures deterministic test results while verifying config validation
// We don't actually load the fixture file to prevent jiti from transforming src/ files
const config = await loadFileConfig(tsFixturesDir, 'repomix-dynamic.config.ts', {
jitiImport: async (fileUrl) => {
// Verify we're loading the correct file
expect(fileUrl).toContain('repomix-dynamic.config.ts');
// Return mock config simulating dynamic values
return {
output: {
filePath: 'output-test-2024-01-01T00-00-00.xml',
style: 'xml',
},
ignore: {
customPatterns: ['**/node_modules/**'],
},
};
},
});
expect(config.output?.filePath).toBe('output-test-2024-01-01T00-00-00.xml');
expect(config.output?.style).toBe('xml');
expect(config.ignore?.customPatterns).toEqual(['**/node_modules/**']);
});
});
describe('JavaScript Config Files', () => {
test('should load .js config with ESM default export', async () => {
const config = await loadFileConfig(jsFixturesDir, 'repomix.config.js');
expect(config).toEqual({
output: {
filePath: 'esm-output.xml',
style: 'xml',
removeComments: true,
},
ignore: {
customPatterns: ['**/node_modules/**', '**/dist/**'],
},
});
});
test('should load .mjs config', async () => {
const config = await loadFileConfig(jsFixturesDir, 'repomix.config.mjs');
expect(config).toEqual({
output: {
filePath: 'mjs-output.xml',
style: 'xml',
},
ignore: {
customPatterns: ['**/test/**'],
},
});
});
test('should load .cjs config with module.exports', async () => {
const config = await loadFileConfig(jsFixturesDir, 'repomix.config.cjs');
expect(config).toEqual({
output: {
filePath: 'cjs-output.xml',
style: 'plain',
},
ignore: {
customPatterns: ['**/build/**'],
},
});
});
test('should handle dynamic values in JS config', async () => {
// Mock jiti to avoid coverage instability caused by dynamic module loading
// This ensures deterministic test results while verifying config validation
// We don't actually load the fixture file to prevent jiti from transforming src/ files
const config = await loadFileConfig(jsFixturesDir, 'repomix-dynamic.config.js', {
jitiImport: async (fileUrl) => {
// Verify we're loading the correct file
expect(fileUrl).toContain('repomix-dynamic.config.js');
// Return mock config simulating dynamic values
return {
output: {
filePath: 'output-2024-01-01T00-00-00.xml',
style: 'xml',
},
ignore: {
customPatterns: ['**/node_modules/**'],
},
};
},
});
expect(config.output?.filePath).toBe('output-2024-01-01T00-00-00.xml');
expect(config.output?.style).toBe('xml');
expect(config.ignore?.customPatterns).toEqual(['**/node_modules/**']);
});
});
});