Misc minor fixes (#3207)

modular-magician · chrisst · commit 81d23bd4b69a · 2019-03-08T16:52:04.000-08:00
Signed-off-by: Modular Magician &lt;magic-modules@google.com&gt;
diff --git a/google/data_source_google_iam_policy_test.go b/google/data_source_google_iam_policy_test.go
diff --git a/google/data_source_google_kms_crypto_key_test.go b/google/data_source_google_kms_crypto_key_test.go
@@ -16,8 +16,6 @@ func TestAccDataSourceGoogleKmsCryptoKey_basic(t *testing.T) {
 	keyParts := strings.Split(kms.CryptoKey.Name, "/")
 	cryptoKeyId := keyParts[len(keyParts)-1]
 
-	fmt.Println(testAccDataSourceGoogleKmsCryptoKey_basic(kms.KeyRing.Name, cryptoKeyId))
-
 	resource.Test(t, resource.TestCase{
 		PreCheck:  func() { testAccPreCheck(t) },
 		Providers: testAccProviders,
@@ -30,9 +28,6 @@ func TestAccDataSourceGoogleKmsCryptoKey_basic(t *testing.T) {
 	})
 }
 
-/*
-	This test should run in its own project, because KMS key rings and crypto keys are not deletable
-*/
 func testAccDataSourceGoogleKmsCryptoKey_basic(keyRingName, cryptoKeyName string) string {
 	return fmt.Sprintf(`
 data "google_kms_crypto_key" "kms_crypto_key" {
diff --git a/google/data_source_google_kms_key_ring_test.go b/google/data_source_google_kms_key_ring_test.go
@@ -27,9 +27,6 @@ func TestAccDataSourceGoogleKmsKeyRing_basic(t *testing.T) {
 	})
 }
 
-/*
-	This test should run in its own project, because keys and key rings are not deletable
-*/
 func testAccDataSourceGoogleKmsKeyRing_basic(keyRingName string) string {
 	return fmt.Sprintf(`
 

Original file line number	Diff line number	Diff line change
`@@ -27,9 +27,6 @@ func TestAccDataSourceGoogleKmsKeyRing_basic(t *testing.T) {`
`27`	`27`	`})`
`28`	`28`	`}`
`29`	`29`
`30`		`-/*`
`31`		`- This test should run in its own project, because keys and key rings are not deletable`
`32`		`-*/`
`33`	`30`	`func testAccDataSourceGoogleKmsKeyRing_basic(keyRingName string) string {`
`34`	`31`	return fmt.Sprintf(`
`35`	`32`