fix(tests): resolve additional integration test failures

- Fix database foreign key constraint error message assertions
  - Change expectations from "foreign key" to "does not exist" to match
    actual repository error messages
- Fix contract test database setup and migration paths
  - Use testutil.GetTestConfig() and NewDatabaseHelper() for consistent setup
  - Remove unused imports (time, config)
  - Align with auth test pattern for better reliability
- Ensure OpenAPI validator schema consistency with actual models

Resolves remaining integration test failures from CI job 45702968489.
All tests now compile successfully and database-dependent tests skip
gracefully when database is unavailable.

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

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

Author: starbops

tests/integration/contract_test.go

@@ -9,15 +9,13 @@ import (
 	"net/http"
 	"net/http/httptest"
 	"testing"
-	"time"
 
 	"github.com/gin-gonic/gin"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 	"github.com/stretchr/testify/suite"
 	"github.com/voidrunnerhq/voidrunner/internal/api/routes"
 	"github.com/voidrunnerhq/voidrunner/internal/auth"
-	"github.com/voidrunnerhq/voidrunner/internal/config"
 	"github.com/voidrunnerhq/voidrunner/internal/database"
 	"github.com/voidrunnerhq/voidrunner/internal/models"
 	"github.com/voidrunnerhq/voidrunner/pkg/logger"
@@ -34,59 +32,28 @@ type ContractTestSuite struct {
 	accessToken string
 	db          *database.Connection
 	validator   *testutil.OpenAPIValidator
+	dbHelper    *testutil.DatabaseHelper
 }
 
 // SetupSuite initializes the contract test suite
 func (s *ContractTestSuite) SetupSuite() {
 	gin.SetMode(gin.TestMode)
 
-	// Initialize test configuration
-	cfg := &config.Config{
-		Database: config.DatabaseConfig{
-			Host:     "localhost",
-			Port:     "5432",
-			Database: "voidrunner_test",
-			User:     "testuser",
-			Password: "testpassword",
-			SSLMode:  "disable",
-		},
-		JWT: config.JWTConfig{
-			SecretKey:            "test-secret-key-for-contract-tests",
-			AccessTokenDuration:  time.Hour,
-			RefreshTokenDuration: 24 * time.Hour,
-			Issuer:               "voidrunner-contract-test",
-			Audience:             "voidrunner-contract-test-api",
-		},
-		CORS: config.CORSConfig{
-			AllowedOrigins: []string{"*"},
-			AllowedMethods: []string{"GET", "POST", "PUT", "DELETE", "OPTIONS"},
-			AllowedHeaders: []string{"*"},
-		},
-	}
+	// Initialize test configuration using testutil helper
+	cfg := testutil.GetTestConfig()
 
 	// Initialize logger
 	log := logger.New("contract-test", "debug")
 
-	// Connect to test database
-	var err error
-	s.db, err = database.NewConnection(&cfg.Database, log.Logger)
-	if err != nil {
-		s.T().Skip("Test database not available:", err)
+	// Initialize database helper (handles connection, migrations, etc.)
+	s.dbHelper = testutil.NewDatabaseHelper(s.T())
+	if s.dbHelper == nil {
+		s.T().Skip("Test database not available")
 		return
 	}
 
-	// Run migrations
-	migrateConfig := &database.MigrateConfig{
-		DatabaseConfig: &cfg.Database,
-		MigrationsPath: "file://../../migrations",
-		Logger:         log.Logger,
-	}
-	if err := database.MigrateUp(migrateConfig); err != nil {
-		s.T().Fatal("Failed to run migrations:", err)
-	}
-
-	// Initialize repositories
-	s.repos = database.NewRepositories(s.db)
+	s.db = s.dbHelper.DB
+	s.repos = s.dbHelper.Repositories
 
 	// Initialize JWT service and auth service
 	jwtSvc := auth.NewJWTService(&cfg.JWT)
@@ -105,17 +72,17 @@ func (s *ContractTestSuite) SetupSuite() {
 
 // TearDownSuite cleans up after tests
 func (s *ContractTestSuite) TearDownSuite() {
-	if s.db != nil {
-		// Clean up test data
-		s.cleanupTestData()
-		s.db.Close()
+	if s.dbHelper != nil {
+		s.dbHelper.Close()
 	}
 }
 
 // SetupTest runs before each test
 func (s *ContractTestSuite) SetupTest() {
 	// Clean up any existing test data
-	s.cleanupTestData()
+	if s.dbHelper != nil {
+		s.dbHelper.CleanupDatabase(s.T())
+	}
 }
 
 // createTestUser creates a test user and gets access token

tests/integration/database_test.go

@@ -383,7 +383,7 @@ func (s *DatabaseIntegrationSuite) TestDatabaseConstraints() {
 
 		err := s.DB.Repositories.Tasks.Create(ctx, task)
 		assert.Error(s.T(), err)
-		assert.Contains(s.T(), err.Error(), "foreign key") // PostgreSQL constraint error
+		assert.Contains(s.T(), err.Error(), "does not exist") // Repository returns custom error message
 	})
 
 	s.Run("task execution foreign key constraint", func() {
@@ -393,7 +393,7 @@ func (s *DatabaseIntegrationSuite) TestDatabaseConstraints() {
 
 		err := s.DB.Repositories.TaskExecutions.Create(ctx, execution)
 		assert.Error(s.T(), err)
-		assert.Contains(s.T(), err.Error(), "foreign key")
+		assert.Contains(s.T(), err.Error(), "does not exist")
 	})
 }