fix: resolve test conflicts with .env environment variables

- Add Jest global setup/teardown hooks to backup and restore .env file
- Clear OCO_ environment variables during test execution
- Modify test resetEnv function to prevent OCO_ env vars from interfering
- Prevent .env file from interfering with test expectations

This fixes the issue where tests were failing due to environment variables
from the user's .env file overriding the expected default config values.

Author: jcppkkk

jest.config.ts

@@ -11,6 +11,8 @@ const config: Config = {
   moduleDirectories: ['node_modules', 'src'],
   preset: 'ts-jest/presets/default-esm',
   setupFilesAfterEnv: ['<rootDir>/test/jest-setup.ts'],
+  globalSetup: '<rootDir>/test/jest-global-setup.ts',
+  globalTeardown: '<rootDir>/test/jest-global-teardown.ts',
   testEnvironment: 'node',
   testRegex: ['.*\\.test\\.ts$'],
   // Tell Jest to ignore the specific duplicate package.json files

test/jest-global-setup.ts

@@ -0,0 +1,20 @@
+const fs = require('fs');
+const path = require('path');
+
+module.exports = async function globalSetup() {
+  const envPath = path.join(process.cwd(), '.env');
+  const backupPath = path.join(process.cwd(), '.env.test-backup');
+
+  // Backup .env file if it exists and clear related env vars
+  if (fs.existsSync(envPath)) {
+    fs.renameSync(envPath, backupPath);
+    console.log('Backed up .env file for testing');
+
+    // Also clear any OCO_ environment variables that might have been loaded
+    Object.keys(process.env).forEach(key => {
+      if (key.startsWith('OCO_')) {
+        delete process.env[key];
+      }
+    });
+  }
+}

test/jest-global-teardown.ts

@@ -0,0 +1,13 @@
+const fs = require('fs');
+const path = require('path');
+
+module.exports = async function globalTeardown() {
+  const envPath = path.join(process.cwd(), '.env');
+  const backupPath = path.join(process.cwd(), '.env.test-backup');
+
+  // Restore .env file
+  if (fs.existsSync(backupPath)) {
+    fs.renameSync(backupPath, envPath);
+    console.log('Restored .env file after testing');
+  }
+}

test/unit/config.test.ts

@@ -15,7 +15,10 @@ describe('config', () => {
 
   function resetEnv(env: NodeJS.ProcessEnv) {
     Object.keys(process.env).forEach((key) => {
-      if (!(key in env)) {
+      if (key.startsWith('OCO_')) {
+        // Don't restore OCO_ environment variables to avoid test interference
+        delete process.env[key];
+      } else if (!(key in env)) {
         delete process.env[key];
       } else {
         process.env[key] = env[key];
@@ -122,7 +125,7 @@ describe('config', () => {
       expect(config.OCO_ONE_LINE_COMMIT).toEqual(false);
       expect(config.OCO_OMIT_SCOPE).toEqual(true);
     });
-    
+
     it('should handle custom HTTP headers correctly', async () => {
       globalConfigFile = await generateConfig('.opencommit', {
         OCO_API_CUSTOM_HEADERS: '{"X-Global-Header": "global-value"}'
@@ -139,7 +142,7 @@ describe('config', () => {
 
       expect(config).not.toEqual(null);
       expect(config.OCO_API_CUSTOM_HEADERS).toEqual({"Authorization": "Bearer token123", "X-Custom-Header": "test-value"});
-      
+
       // No need to parse JSON again since it's already an object
       const parsedHeaders = config.OCO_API_CUSTOM_HEADERS;
       expect(parsedHeaders).toHaveProperty('Authorization', 'Bearer token123');