alibaba / terraform-provider

[DEPRECATED] This repo has been deprecated and please access https://github.com/terraform-providers/terraform-provider-alicloud
Apache License 2.0
384 stars 117 forks source link

Allow setting the scaling group balancing policy. #643

Closed ebabani closed 6 years ago

xiaozhu36 commented 6 years ago

Thanks for your contribute. Can you add a test case for this PR and submit the result of running this test case?

ebabani commented 6 years ago

Added a test, small change so I added it to an existing test case.

Results with a failing and passing run:

~/g/s/g/a/t/alicloud (ebabani/ess-multi-az-policy) $ env TF_ACC=1 go test -run TestAccAlicloudEssScalingGroup_vpc -v
=== RUN   TestAccAlicloudEssScalingGroup_vpc
--- FAIL: TestAccAlicloudEssScalingGroup_vpc (78.45s)
    testing.go:513: Step 0 error: Check failed: Check 7/7 error: alicloud_ess_scaling_group.foo: Attribute 'multi_az_policy' expected "BALANCEZ", got "BALANCE"
FAIL
exit status 1
FAIL    github.com/alibaba/terraform-provider/alicloud  78.488s
~/g/s/g/a/t/alicloud (ebabani/ess-multi-az-policy) $ env TF_ACC=1 go test -run TestAccAlicloudEssScalingGroup_vpc -v
=== RUN   TestAccAlicloudEssScalingGroup_vpc
--- PASS: TestAccAlicloudEssScalingGroup_vpc (213.03s)
PASS
ok      github.com/alibaba/terraform-provider/alicloud  213.096s
~/g/s/g/a/t/alicloud (ebabani/ess-multi-az-policy) $