fix(test): updates tests for the recent changes

Author: hoangphuc841

apps/operator/internal/controller/database/resources_test.go

@@ -29,7 +29,7 @@ func TestFormatPostgresURI(t *testing.T) {
 			host:     "localhost",
 			dbName:   "mydb",
 			port:     5432,
-			expected: "postgres://user:pass@localhost:5432/mydb",
+			expected: "postgres://user:pass@localhost:5432/mydb?sslmode=disable",
 		},
 		{
 			name:     "special chars in password",
@@ -38,7 +38,7 @@ func TestFormatPostgresURI(t *testing.T) {
 			host:     "db.example.com",
 			dbName:   "app_db",
 			port:     5432,
-			expected: "postgres://user:p%40ss%3Aword%2F@db.example.com:5432/app_db",
+			expected: "postgres://user:p%40ss%3Aword%2F@db.example.com:5432/app_db?sslmode=disable",
 		},
 		{
 			name:     "special chars in username",
@@ -47,7 +47,7 @@ func TestFormatPostgresURI(t *testing.T) {
 			host:     "postgres-host",
 			dbName:   "database",
 			port:     5432,
-			expected: "postgres://user%2Badmin:password@postgres-host:5432/database",
+			expected: "postgres://user%2Badmin:password@postgres-host:5432/database?sslmode=disable",
 		},
 		{
 			name:     "custom port",
@@ -56,7 +56,7 @@ func TestFormatPostgresURI(t *testing.T) {
 			host:     "db-instance",
 			dbName:   "prod_db",
 			port:     5433,
-			expected: "postgres://admin:secret123@db-instance:5433/prod_db",
+			expected: "postgres://admin:secret123@db-instance:5433/prod_db?sslmode=disable",
 		},
 		{
 			name:     "IPv6 host",
@@ -65,7 +65,7 @@ func TestFormatPostgresURI(t *testing.T) {
 			host:     "::1",
 			dbName:   "testdb",
 			port:     5432,
-			expected: "postgres://user:pass@::1:5432/testdb",
+			expected: "postgres://user:pass@::1:5432/testdb?sslmode=disable",
 		},
 	}
 

apps/operator/internal/cue/tekton_test.go

@@ -67,9 +67,11 @@ func TestRenderTektonResources_AllResources(t *testing.T) {
 		t.Fatalf("RenderTektonResources failed: %v", err)
 	}
 
-	// With webhookDomain set, we expect 11 objects:
-	// 6 Tasks (git-clone, kaniko-build, git-update-manifest, argocd-sync, db-migrate, postgrest-reload) + 1 Pipeline + 1 TriggerBinding + 1 TriggerTemplate + 1 EventListener + 1 Ingress
-	expectedCount := 11
+	// With webhookDomain set, we expect 13 objects:
+	// 6 Tasks (git-clone, kaniko-build, git-update-manifest, argocd-sync, db-migrate, postgrest-reload)
+	// + 2 Pipelines (from-code-to-cluster + db-migrate)
+	// + 1 TriggerBinding + 2 TriggerTemplates (gitea + db-migrate) + 1 EventListener + 1 Ingress
+	expectedCount := 13
 	if len(objects) != expectedCount {
 		t.Errorf("Expected %d objects, got %d", expectedCount, len(objects))
 		for i, obj := range objects {
@@ -80,9 +82,9 @@ func TestRenderTektonResources_AllResources(t *testing.T) {
 	// Verify each expected kind is present
 	expectedKinds := map[string]int{
 		"Task":            6,
-		"Pipeline":        1,
+		"Pipeline":        2,
 		"TriggerBinding":  1,
-		"TriggerTemplate": 1,
+		"TriggerTemplate": 2,
 		"EventListener":   1,
 		"Ingress":         1,
 	}
@@ -115,8 +117,9 @@ func TestRenderTektonResources_WithoutWebhook(t *testing.T) {
 		t.Fatalf("RenderTektonResources failed: %v", err)
 	}
 
-	// Without webhookDomain: 10 objects (no Ingress)
-	expectedCount := 10
+	// Without webhookDomain: 12 objects (no Ingress)
+	// 6 Tasks + 2 Pipelines + 1 TriggerBinding + 2 TriggerTemplates + 1 EventListener
+	expectedCount := 12
 	if len(objects) != expectedCount {
 		t.Errorf("Expected %d objects (no webhook), got %d", expectedCount, len(objects))
 		for i, obj := range objects {
@@ -255,17 +258,15 @@ func TestRenderTektonResources_BuildOnlyPipeline(t *testing.T) {
 		t.Fatalf("RenderTektonResources failed: %v", err)
 	}
 
-	// Verify pipeline name is "build-only"
-	foundPipeline := false
+	// Verify the primary pipeline with name "build-only" exists among rendered pipelines.
+	// Note: db-migrate pipeline is always rendered alongside the primary pipeline.
+	foundBuildOnly := false
 	for _, obj := range objects {
-		if obj.GetKind() == "Pipeline" {
-			if obj.GetName() != "build-only" {
-				t.Errorf("Expected pipeline name 'build-only', got %q", obj.GetName())
-			}
-			foundPipeline = true
+		if obj.GetKind() == "Pipeline" && obj.GetName() == "build-only" {
+			foundBuildOnly = true
 		}
 	}
-	if !foundPipeline {
-		t.Error("Pipeline not found in rendered objects")
+	if !foundBuildOnly {
+		t.Error("Expected primary pipeline 'build-only' not found in rendered objects")
 	}
 }