Made TestAccLoggingProjectSink_default use a separate project (Google…

…CloudPlatform#10386)
pengq-google · Apr 8, 2024 · 7f5098d · 7f5098d
1 parent 24d7c0c
commit 7f5098d
Showing 1 changed file with 27 additions and 6 deletions.
diff --git a/mmv1/third_party/terraform/services/logging/resource_logging_project_sink_test.go b/mmv1/third_party/terraform/services/logging/resource_logging_project_sink_test.go
@@ -13,6 +13,9 @@ import (
 func TestAccLoggingProjectSink_basic(t *testing.T) {
 	t.Parallel()
 
+	orgId := envvar.GetTestOrgFromEnv(t)
+	billingAccount := envvar.GetTestBillingAccountFromEnv(t)
+	projectId := "tf-test" + acctest.RandString(t, 10)
 	sinkName := "tf-test-sink-" + acctest.RandString(t, 10)
 	bucketName := "tf-test-sink-bucket-" + acctest.RandString(t, 10)
 
@@ -22,7 +25,7 @@ func TestAccLoggingProjectSink_basic(t *testing.T) {
 		CheckDestroy:             testAccCheckLoggingProjectSinkDestroyProducer(t),
 		Steps: []resource.TestStep{
 			{
-				Config: testAccLoggingProjectSink_basic(sinkName, envvar.GetTestProjectFromEnv(), bucketName),
+				Config: testAccLoggingProjectSink_basic(projectId, orgId, billingAccount, sinkName, bucketName, "false"),
 			},
 			{
 				ResourceName:      "google_logging_project_sink.basic",
@@ -36,6 +39,9 @@ func TestAccLoggingProjectSink_basic(t *testing.T) {
 func TestAccLoggingProjectSink_default(t *testing.T) {
 	t.Parallel()
 
+	orgId := envvar.GetTestOrgFromEnv(t)
+	billingAccount := envvar.GetTestBillingAccountFromEnv(t)
+	projectId := "tf-test" + acctest.RandString(t, 10)
 	sinkName := "_Default"
 	bucketName := "tf-test-sink-bucket-" + acctest.RandString(t, 10)
 
@@ -44,7 +50,8 @@ func TestAccLoggingProjectSink_default(t *testing.T) {
 		ProtoV5ProviderFactories: acctest.ProtoV5ProviderFactories(t),
 		Steps: []resource.TestStep{
 			{
-				Config: testAccLoggingProjectSink_basic(sinkName, envvar.GetTestProjectFromEnv(), bucketName),
+				// Default sink has a permadiff if any value is sent for "disabled" other than "true"
+				Config: testAccLoggingProjectSink_basic(projectId, orgId, billingAccount, sinkName, bucketName, "true"),
 			},
 			{
 				ResourceName:      "google_logging_project_sink.basic",
@@ -363,20 +370,34 @@ func testAccCheckLoggingProjectSinkDestroyProducer(t *testing.T) func(s *terrafo
 	}
 }
 
-func testAccLoggingProjectSink_basic(name, project, bucketName string) string {
+func testAccLoggingProjectSink_basic(projectId, orgId, billingAccount, sinkName, bucketName, disabled string) string {
 	return fmt.Sprintf(`
+resource "google_project" "project" {
+	project_id = "%s"
+	name       = "%s"
+	org_id     = "%s"
+  billing_account = "%s"
+}
+
+resource "google_project_service" "logging_service" {
+	project = google_project.project.project_id
+	service = "logging.googleapis.com"
+}
+
 resource "google_logging_project_sink" "basic" {
   name        = "%s"
-  project     = "%s"
+  disabled    = %s
+  project     = google_project_service.logging_service.project
   destination = "storage.googleapis.com/${google_storage_bucket.gcs-bucket.name}"
-  filter      = "logName=\"projects/%s/logs/compute.googleapis.com%%2Factivity_log\" AND severity>=ERROR"
+  filter      = "logName=\"projects/${google_project.project.project_id}/logs/compute.googleapis.com%%2Factivity_log\" AND severity>=ERROR"
 }
 
 resource "google_storage_bucket" "gcs-bucket" {
   name     = "%s"
+  project  = google_project.project.project_id
   location = "US"
 }
-`, name, project, project, bucketName)
+`, projectId, projectId, orgId, billingAccount, sinkName, disabled, bucketName)
 }
 
 func testAccLoggingProjectSink_described(name, project, bucketName string) string {