Merge pull request #6 from ngocbd/copilot/fix-9c30279f-0189-479e-8e4f-f086f4c76102

Fix GitHub Actions test failures by implementing environment-specific configuration files

Author: ngocbd

.env.development

@@ -0,0 +1,25 @@
+# Development environment configuration for Container Engine
+# Database Configuration
+DATABASE_URL=postgresql://postgres:password@localhost:5432/container_engine
+
+# Redis Configuration  
+REDIS_URL=redis://localhost:6379
+
+# Server Configuration
+PORT=3000
+
+# JWT Configuration
+JWT_SECRET=development-jwt-secret-key-not-for-production
+JWT_EXPIRES_IN=3600
+
+# API Key Configuration
+API_KEY_PREFIX=ce_dev_
+
+# Kubernetes Configuration
+KUBERNETES_NAMESPACE=container-engine-dev
+
+# Domain Configuration
+DOMAIN_SUFFIX=dev.container-engine.app
+
+# Logging
+RUST_LOG=container_engine=debug,tower_http=debug

.env.integrate_test

@@ -0,0 +1,25 @@
+# Integration test environment configuration for Container Engine
+# Database Configuration
+DATABASE_URL=postgresql://postgres:password@localhost:5432/container_engine_test
+
+# Redis Configuration
+REDIS_URL=redis://localhost:6379
+
+# Server Configuration
+PORT=3001
+
+# JWT Configuration
+JWT_SECRET=test-jwt-secret-key-for-integration-tests
+JWT_EXPIRES_IN=3600
+
+# API Key Configuration
+API_KEY_PREFIX=ce_test_
+
+# Kubernetes Configuration
+KUBERNETES_NAMESPACE=container-engine-test
+
+# Domain Configuration
+DOMAIN_SUFFIX=test.container-engine.app
+
+# Logging
+RUST_LOG=container_engine=info,tower_http=info

.github/workflows/integration-tests.yml

@@ -78,16 +78,22 @@ jobs:
 
     - name: Set up test environment
       run: |
-        # Set environment variables for the application
+        # Set the environment variable to use the integration test configuration
+        echo "ENVIRONMENT=integrate_test" >> $GITHUB_ENV
+        
+        # Load configuration from .env.integrate_test file
+        if [ -f .env.integrate_test ]; then
+          while IFS= read -r line; do
+            # Skip comments and empty lines
+            if [[ $line =~ ^[^#]*= ]]; then
+              echo "$line" >> $GITHUB_ENV
+            fi
+          done < .env.integrate_test
+        fi
+        
+        # Override specific variables for GitHub Actions environment
         echo "DATABASE_URL=postgresql://postgres:password@localhost:5432/container_engine_test" >> $GITHUB_ENV
         echo "REDIS_URL=redis://localhost:6379" >> $GITHUB_ENV
-        echo "PORT=3000" >> $GITHUB_ENV
-        echo "JWT_SECRET=test-jwt-secret-key-for-github-actions" >> $GITHUB_ENV
-        echo "JWT_EXPIRES_IN=3600" >> $GITHUB_ENV
-        echo "API_KEY_PREFIX=ce_test_" >> $GITHUB_ENV
-        echo "KUBERNETES_NAMESPACE=test" >> $GITHUB_ENV
-        echo "DOMAIN_SUFFIX=test.local" >> $GITHUB_ENV
-        echo "RUST_LOG=container_engine=info,tower_http=info" >> $GITHUB_ENV
 
     - name: Run database migrations
       run: |
@@ -98,8 +104,8 @@ jobs:
         cargo run &
         echo $! > server.pid
         
-        # Wait for server to be ready
-        timeout 60 bash -c 'until curl -f http://localhost:3000/health; do sleep 2; done'
+        # Wait for server to be ready (using port 3001 for integration tests)
+        timeout 60 bash -c 'until curl -f http://localhost:3001/health; do sleep 2; done'
 
     - name: Run integration tests
       run: |

ENVIRONMENT_CONFIG.md

@@ -0,0 +1,53 @@
+# Environment Configuration
+
+This project uses environment-specific configuration files to manage different deployment and testing scenarios.
+
+## Environment Files
+
+- `.env.development` - Configuration for local development
+- `.env.integrate_test` - Configuration for integration testing  
+- `.env.example` - Example configuration template
+
+## Usage
+
+### Development
+```bash
+# Uses .env.development by default or when ENVIRONMENT=development
+cargo run
+
+# Explicitly set development environment
+ENVIRONMENT=development cargo run
+```
+
+### Integration Testing
+```bash
+# Run integration tests (automatically uses .env.integrate_test)
+ENVIRONMENT=integrate_test cargo run
+
+# Run Python integration tests
+python -m pytest tests/integrate/ -v
+```
+
+## Key Differences
+
+### Development (.env.development)
+- Port: 3000
+- Database: `container_engine` 
+- Namespace: `container-engine-dev`
+- Domain: `dev.container-engine.app`
+- Log level: debug
+
+### Integration Test (.env.integrate_test)
+- Port: 3001 (avoids conflicts)
+- Database: `container_engine_test`
+- Namespace: `container-engine-test` 
+- Domain: `test.container-engine.app`
+- Log level: info
+
+## Port Separation
+
+Different ports are used to prevent conflicts when running development server and tests simultaneously:
+- Development: port 3000
+- Integration tests: port 3001
+
+This ensures GitHub Actions and local testing won't encounter "port already in use" errors.

src/config.rs

@@ -15,8 +15,19 @@ pub struct Config {
 
 impl Config {
     pub fn new() -> Result<Self, Box<dyn std::error::Error>> {
-        // Load .env file if it exists
-        dotenv::dotenv().ok();
+        // Determine environment and load appropriate .env file
+        let environment = env::var("ENVIRONMENT").unwrap_or_else(|_| "development".to_string());
+        
+        let env_file = match environment.as_str() {
+            "test" | "integrate_test" => ".env.integrate_test",
+            "development" | "dev" => ".env.development",
+            _ => ".env.development", // default to development
+        };
+        
+        // Try to load the environment-specific file, fall back to default .env
+        if let Err(_) = dotenv::from_filename(env_file) {
+            dotenv::dotenv().ok();
+        }
 
         let config = Config {
             database_url: env::var("DATABASE_URL")

tests/.env.test

@@ -1,7 +1,7 @@
 # Test environment configuration for Container Engine integration tests
 
 # Server settings
-TEST_BASE_URL=http://localhost:3000
+TEST_BASE_URL=http://localhost:3001
 
 # Database settings (for test isolation)
 TEST_DB_HOST=localhost

tests/integrate/conftest.py

@@ -11,14 +11,23 @@
 import redis
 from dotenv import load_dotenv
 
-# Load environment variables
-load_dotenv()
+# Load environment-specific .env file for tests
+environment = os.getenv("ENVIRONMENT", "integrate_test")
+env_file = f".env.{environment}"
+
+# Try to load environment-specific file, fallback to .env.test, then .env
+if os.path.exists(env_file):
+    load_dotenv(env_file)
+elif os.path.exists("tests/.env.test"):
+    load_dotenv("tests/.env.test")
+else:
+    load_dotenv()
 
 class TestConfig:
     """Test configuration settings"""
 
     # Server settings
-    BASE_URL = os.getenv("TEST_BASE_URL", "http://localhost:3000")
+    BASE_URL = os.getenv("TEST_BASE_URL", "http://localhost:3001")  # Use port 3001 for tests
     HEALTH_ENDPOINT = "/health"
 
     # Database settings
@@ -138,9 +147,10 @@ def start_server(self):
         # Set environment variables for the server
         env = os.environ.copy()
         env.update({
+            "ENVIRONMENT": "integrate_test",  # This will load .env.integrate_test
             "DATABASE_URL": TestConfig.DATABASE_URL,
             "REDIS_URL": TestConfig.REDIS_URL,
-            "PORT": "3000",
+            "PORT": "3001",  # Use port 3001 to avoid conflicts
             "JWT_SECRET": "test-jwt-secret-key",
             "JWT_EXPIRES_IN": "3600",
             "API_KEY_PREFIX": "ce_test_",