fix: Enhance Jest configuration for CI compatibility and fix E2E test timeouts

- Enhanced CI environment detection with support for multiple CI platforms
- Added explicit timeouts to E2E tests (15-45 seconds based on operation complexity)
- Updated Jest configuration with CI-specific settings (single worker, longer timeouts)
- Fixed missing timeouts in basic.test.ts that could cause CI failures
- Improved test stability across different environments

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: Mrassimo

jest.config.js

@@ -1,4 +1,23 @@
 /** @type {import('jest').Config} */
+
+// Enhanced CI detection for various environments
+function isCI() {
+  return !!(
+    process.env.CI ||                    // Standard CI environment variable
+    process.env.GITHUB_ACTIONS ||       // GitHub Actions
+    process.env.GITLAB_CI ||            // GitLab CI
+    process.env.CIRCLECI ||             // CircleCI
+    process.env.TRAVIS ||               // Travis CI
+    process.env.BUILDKITE ||            // Buildkite
+    process.env.JENKINS_URL ||          // Jenkins
+    process.env.TEAMCITY_VERSION ||     // TeamCity
+    process.env.TF_BUILD ||             // Azure DevOps
+    process.env.CODEBUILD_BUILD_ID ||   // AWS CodeBuild
+    process.env.BUILD_ID ||             // Generic build ID
+    process.env.NODE_ENV === 'test'     // Test environment
+  );
+}
+
 module.exports = {
   preset: 'ts-jest',
   testEnvironment: 'node',
@@ -31,18 +50,18 @@ module.exports = {
       statements: 65,
     },
   },
-  // Fast test execution
-  testTimeout: 5000,
-  maxWorkers: '50%',
+  // CI-friendly test execution - detect various CI environments
+  testTimeout: isCI() ? 30000 : 10000, // Longer timeout for CI environments
+  maxWorkers: isCI() ? 1 : '50%', // Single worker in CI to avoid resource conflicts
 
   // Clean test environment
   clearMocks: true,
   restoreMocks: true,
   resetMocks: true,
 
-  // No hanging tests allowed
-  forceExit: false,
-  detectOpenHandles: false,
+  // Better error detection and handling
+  forceExit: isCI() ? true : false, // Force exit in CI to prevent hanging
+  detectOpenHandles: true, // Enable to detect process leaks
 
   // Coverage reporting
   coverageDirectory: 'coverage',
@@ -53,8 +72,8 @@ module.exports = {
   cache: true,
   cacheDirectory: '<rootDir>/.jest-cache',
 
-  // Error handling
-  bail: 1, // Stop on first test failure for faster feedback
+  // Error handling - show all failures in CI, bail locally for speed
+  bail: isCI() ? false : 1,
 
   // Custom matchers and setup
   setupFilesAfterEnv: ['<rootDir>/tests/setup.ts'],

tests/e2e/basic.test.ts

@@ -28,25 +28,41 @@ Charlie,35,92`;
   });
 
   it('should execute CLI help command', () => {
-    const result = execSync(`node "${cliPath}" --help`, { encoding: 'utf8' });
+    const result = execSync(`node "${cliPath}" --help`, { 
+      encoding: 'utf8',
+      timeout: 15000, // 15 seconds timeout for CI compatibility
+      maxBuffer: 1024 * 1024 // 1MB buffer
+    });
     expect(result).toContain('Usage: datapilot');
     expect(result).toContain('Commands:');
   });
 
   it('should execute CLI version command', () => {
-    const result = execSync(`node "${cliPath}" --version`, { encoding: 'utf8' });
+    const result = execSync(`node "${cliPath}" --version`, { 
+      encoding: 'utf8',
+      timeout: 15000, // 15 seconds timeout for CI compatibility
+      maxBuffer: 1024 * 1024 // 1MB buffer
+    });
     expect(result).toContain('1.7.0');
   });
 
   it('should analyze CSV file with overview command', () => {
-    const result = execSync(`node "${cliPath}" overview "${testCsv}" -f json`, { encoding: 'utf8' });
+    const result = execSync(`node "${cliPath}" overview "${testCsv}" -f json`, { 
+      encoding: 'utf8',
+      timeout: 30000, // 30 seconds timeout for data processing
+      maxBuffer: 1024 * 1024 // 1MB buffer
+    });
     expect(result).toContain('Report written to:');
     expect(result).toContain('Selected parser: csv');
     expect(result).toContain('Detected format: csv');
   });
 
   it('should run complete analysis with all command', () => {
-    const result = execSync(`node "${cliPath}" all "${testCsv}"`, { encoding: 'utf8' });
+    const result = execSync(`node "${cliPath}" all "${testCsv}"`, { 
+      encoding: 'utf8',
+      timeout: 45000, // 45 seconds timeout for complete analysis
+      maxBuffer: 2 * 1024 * 1024 // 2MB buffer for full analysis output
+    });
     expect(result).toContain('Report written to:');
     expect(result).toContain('Sequential execution completed successfully');
   });

tests/setup.ts

@@ -14,8 +14,8 @@ global.console = {
   error: originalConsole.error, // Keep errors for debugging
 };
 
-// Global test timeout (should be fast)
-jest.setTimeout(5000);
+// Global test timeout - defer to jest.config.js for CI compatibility
+// jest.setTimeout is handled by jest.config.js based on environment
 
 // Clean up after each test
 afterEach(() => {