Skip to content

Commit 39c35c9

Browse files
committed
Fix duplicate TEST_TEMP_DIR definition causing CI failures
- Import TEST_TEMP_DIR from shared setup instead of redefining - Remove unused _claudeDir variable - This fixes race conditions and path conflicts in CI environment
1 parent 167fb68 commit 39c35c9

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

claude-config-composer/tests/integration/cli.test.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,11 @@
11
import { execSync } from 'child_process';
22
import fsSync from 'fs';
33
import fs from 'fs/promises';
4-
import os from 'os';
54
import path from 'path';
65
import { afterEach, beforeEach, describe, expect, it } from 'vitest';
6+
import { TEST_TEMP_DIR } from '../setup';
77

88
const CLI_PATH = path.resolve(__dirname, '../../dist/cli.js');
9-
const TEST_TEMP_DIR = path.join(os.tmpdir(), 'claude-config-test');
109

1110
describe('CLI Integration Tests', () => {
1211
let testDir: string;
@@ -118,7 +117,6 @@ describe('CLI Integration Tests', () => {
118117

119118
expect(result).toContain('Configuration generated in');
120119

121-
const _claudeDir = path.join(testDir, '.claude');
122120
const claudeMd = await fs.readFile(path.join(testDir, 'CLAUDE.md'), 'utf8');
123121

124122
// Should contain content from both configurations

0 commit comments

Comments
 (0)