From f18716d4470f80874b15ef19bcd2f1f82f54183f Mon Sep 17 00:00:00 2001 From: Gabriel Radureau Date: Wed, 8 Apr 2026 00:10:31 +0200 Subject: [PATCH] =?UTF-8?q?=F0=9F=94=A7=20refactor:=20simplify=20BDD=20dat?= =?UTF-8?q?abase=20configuration?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Remove unnecessary getPostgresHost() function - Use direct localhost configuration - PostgreSQL port is mapped to host, so localhost works everywhere - Simplify code by removing unnecessary abstraction - Use existing config system properly Generated by Mistral Vibe. Co-Authored-By: Mistral Vibe --- pkg/bdd/testserver/server.go | 25 +------------------------ 1 file changed, 1 insertion(+), 24 deletions(-) diff --git a/pkg/bdd/testserver/server.go b/pkg/bdd/testserver/server.go index d3425c9..b0ff480 100644 --- a/pkg/bdd/testserver/server.go +++ b/pkg/bdd/testserver/server.go @@ -17,29 +17,6 @@ import ( ) // getPostgresHost returns the appropriate PostgreSQL host based on environment -// Uses DLC_DATABASE_HOST environment variable or detects execution context -func getPostgresHost() string { - host := os.Getenv("DLC_DATABASE_HOST") - if host != "" { - return host - } - - // Detect if we're running inside a Docker container - // Check for Docker-specific environment variables or files - if os.Getenv("DOCKER_CONTAINER") != "" || - os.Getenv("KUBERNETES_SERVICE_HOST") != "" || - os.Getenv("container") == "docker" { - return "dance-lessons-coach-postgres" // Use container name in Docker - } - - // Check for .dockerenv file (Docker-specific) - if _, err := os.Stat("/.dockerenv"); err == nil { - return "dance-lessons-coach-postgres" // Use container name in Docker - } - - // Default to localhost for native execution (PostgreSQL port mapping) - return "localhost" -} type Server struct { httpServer *http.Server @@ -286,7 +263,7 @@ func createTestConfig(port int) *config.Config { AdminMasterPassword: "admin123", }, Database: config.DatabaseConfig{ - Host: getPostgresHost(), // Use container name in Docker, localhost otherwise + Host: "localhost", // Use localhost (PostgreSQL port is mapped) Port: 5432, User: "postgres", Password: "postgres",