Make cloudbuild trigger disabled field a bool (#3398)

Signed-off-by: Modular Magician <magic-modules@google.com>
This commit is contained in:
The Magician 2019-04-08 13:35:32 -07:00 committed by emily
parent 13308c0ce7
commit f82494c34f
2 changed files with 63 additions and 5 deletions

View File

@ -148,7 +148,7 @@ func resourceCloudBuildTrigger() *schema.Resource {
Optional: true,
},
"disabled": {
Type: schema.TypeString,
Type: schema.TypeBool,
Optional: true,
},
"filename": {

View File

@ -16,7 +16,7 @@ func TestAccCloudBuildTrigger_basic(t *testing.T) {
CheckDestroy: testAccCheckCloudBuildTriggerDestroy,
Steps: []resource.TestStep{
{
Config: testGoogleCloudBuildTrigger_basic(),
Config: testAccCloudBuildTrigger_basic(),
},
{
ResourceName: "google_cloudbuild_trigger.build_trigger",
@ -24,7 +24,35 @@ func TestAccCloudBuildTrigger_basic(t *testing.T) {
ImportStateVerify: true,
},
{
Config: testGoogleCloudBuildTrigger_updated(),
Config: testAccCloudBuildTrigger_updated(),
},
{
ResourceName: "google_cloudbuild_trigger.build_trigger",
ImportState: true,
ImportStateVerify: true,
},
},
})
}
func TestAccCloudBuildTrigger_disable(t *testing.T) {
t.Parallel()
resource.Test(t, resource.TestCase{
PreCheck: func() { testAccPreCheck(t) },
Providers: testAccProviders,
CheckDestroy: testAccCheckCloudBuildTriggerDestroy,
Steps: []resource.TestStep{
{
Config: testAccCloudBuildTrigger_basic(),
},
{
ResourceName: "google_cloudbuild_trigger.build_trigger",
ImportState: true,
ImportStateVerify: true,
},
{
Config: testAccCloudBuildTrigger_basicDisabled(),
},
{
ResourceName: "google_cloudbuild_trigger.build_trigger",
@ -55,7 +83,7 @@ func TestAccCloudBuildTrigger_fullStep(t *testing.T) {
})
}
func testGoogleCloudBuildTrigger_basic() string {
func testAccCloudBuildTrigger_basic() string {
return fmt.Sprintf(`
resource "google_cloudbuild_trigger" "build_trigger" {
description = "acceptance test build trigger"
@ -84,6 +112,36 @@ resource "google_cloudbuild_trigger" "build_trigger" {
`)
}
func testAccCloudBuildTrigger_basicDisabled() string {
return fmt.Sprintf(`
resource "google_cloudbuild_trigger" "build_trigger" {
disabled = true
description = "acceptance test build trigger"
trigger_template {
branch_name = "master"
repo_name = "some-repo"
}
build {
images = ["gcr.io/$PROJECT_ID/$REPO_NAME:$COMMIT_SHA"]
tags = ["team-a", "service-b"]
step {
name = "gcr.io/cloud-builders/gsutil"
args = ["cp", "gs://mybucket/remotefile.zip", "localfile.zip"]
}
step {
name = "gcr.io/cloud-builders/go"
args = ["build", "my_package"]
env = ["env1=two"]
}
step {
name = "gcr.io/cloud-builders/docker"
args = ["build", "-t", "gcr.io/$PROJECT_ID/$REPO_NAME:$COMMIT_SHA", "-f", "Dockerfile", "."]
}
}
}
`)
}
func testAccCloudBuildTrigger_fullStep() string {
return fmt.Sprintf(`
resource "google_cloudbuild_trigger" "build_trigger" {
@ -110,7 +168,7 @@ resource "google_cloudbuild_trigger" "build_trigger" {
`)
}
func testGoogleCloudBuildTrigger_updated() string {
func testAccCloudBuildTrigger_updated() string {
return fmt.Sprintf(`
resource "google_cloudbuild_trigger" "build_trigger" {
description = "acceptance test build trigger updated"