3472cab7d6
Fixes #10042 Fixes #9989 Another panic was found with this resource. IT essentially was causing a panic when no certificates were found. This was due to the casting of status to []string There are times when there are no statuses passed in. Made the error message a lot more generic now rather than having something like this ``` No certificate with statuses [] for domain mytestdomain.com found in this region. ``` This now becomes: ``` No certificate for domain mytestdomain.com found in this region. ``` Also, added a test to show that the panic is gone ``` % make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAwsAcmCertificateDataSource_' ==> Checking that code complies with gofmt requirements... go generate $(go list ./... | grep -v /terraform/vendor/) 2016/11/11 15:11:33 Generated command/internal_plugin_list.go TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAwsAcmCertificateDataSource_ -timeout 120m === RUN TestAccAwsAcmCertificateDataSource_noMatchReturnsError --- PASS: TestAccAwsAcmCertificateDataSource_noMatchReturnsError (6.07s) PASS ok github.com/hashicorp/terraform/builtin/providers/aws6.094s ``` |
||
|---|---|---|
| .. | ||
| archive | ||
| atlas | ||
| aws | ||
| azure | ||
| azurerm | ||
| bitbucket | ||
| chef | ||
| clc | ||
| cloudflare | ||
| cloudstack | ||
| cobbler | ||
| consul | ||
| datadog | ||
| digitalocean | ||
| dme | ||
| dnsimple | ||
| docker | ||
| dyn | ||
| fastly | ||
| github | ||
| grafana | ||
| heroku | ||
| influxdb | ||
| librato | ||
| logentries | ||
| mailgun | ||
| mysql | ||
| nomad | ||
| null | ||
| openstack | ||
| packet | ||
| pagerduty | ||
| postgresql | ||
| powerdns | ||
| rabbitmq | ||
| random | ||
| rundeck | ||
| scaleway | ||
| softlayer | ||
| statuscake | ||
| template | ||
| terraform | ||
| test | ||
| tls | ||
| triton | ||
| ultradns | ||
| vault | ||
| vcd | ||
| vsphere | ||