Skip to content

Commit adf4280

Browse files
committed
provider/digitalocean: prevent collision on domain acctest
1 parent 983b342 commit adf4280

1 file changed

Lines changed: 9 additions & 7 deletions

File tree

builtin/providers/digitalocean/resource_digitalocean_domain_test.go

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,25 +5,27 @@ import (
55
"testing"
66

77
"github.com/digitalocean/godo"
8+
"github.com/hashicorp/terraform/helper/acctest"
89
"github.com/hashicorp/terraform/helper/resource"
910
"github.com/hashicorp/terraform/terraform"
1011
)
1112

1213
func TestAccDigitalOceanDomain_Basic(t *testing.T) {
1314
var domain godo.Domain
15+
domainName := fmt.Sprintf("foobar-test-terraform-%s.com", acctest.RandString(10))
1416

1517
resource.Test(t, resource.TestCase{
1618
PreCheck: func() { testAccPreCheck(t) },
1719
Providers: testAccProviders,
1820
CheckDestroy: testAccCheckDigitalOceanDomainDestroy,
1921
Steps: []resource.TestStep{
2022
resource.TestStep{
21-
Config: testAccCheckDigitalOceanDomainConfig_basic,
23+
Config: fmt.Sprintf(testAccCheckDigitalOceanDomainConfig_basic, domainName),
2224
Check: resource.ComposeTestCheckFunc(
2325
testAccCheckDigitalOceanDomainExists("digitalocean_domain.foobar", &domain),
24-
testAccCheckDigitalOceanDomainAttributes(&domain),
26+
testAccCheckDigitalOceanDomainAttributes(&domain, domainName),
2527
resource.TestCheckResourceAttr(
26-
"digitalocean_domain.foobar", "name", "foobar-test-terraform.com"),
28+
"digitalocean_domain.foobar", "name", domainName),
2729
resource.TestCheckResourceAttr(
2830
"digitalocean_domain.foobar", "ip_address", "192.168.0.10"),
2931
),
@@ -51,10 +53,10 @@ func testAccCheckDigitalOceanDomainDestroy(s *terraform.State) error {
5153
return nil
5254
}
5355

54-
func testAccCheckDigitalOceanDomainAttributes(domain *godo.Domain) resource.TestCheckFunc {
56+
func testAccCheckDigitalOceanDomainAttributes(domain *godo.Domain, name string) resource.TestCheckFunc {
5557
return func(s *terraform.State) error {
5658

57-
if domain.Name != "foobar-test-terraform.com" {
59+
if domain.Name != name {
5860
return fmt.Errorf("Bad name: %s", domain.Name)
5961
}
6062

@@ -94,6 +96,6 @@ func testAccCheckDigitalOceanDomainExists(n string, domain *godo.Domain) resourc
9496

9597
const testAccCheckDigitalOceanDomainConfig_basic = `
9698
resource "digitalocean_domain" "foobar" {
97-
name = "foobar-test-terraform.com"
98-
ip_address = "192.168.0.10"
99+
name = "%s"
100+
ip_address = "192.168.0.10"
99101
}`

0 commit comments

Comments
 (0)