From 5976bba0b5dd67d2169641a5c3dab935155fb885 Mon Sep 17 00:00:00 2001 From: Alexandre Garand Date: Tue, 9 Jul 2019 10:20:59 +0200 Subject: fix the tests the ips are now fetched correctly a sleep was put in the destroy check to ensure that the api have time to destroy before it is checked --- mailgun/resource_mailgun_domain_test.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'mailgun/resource_mailgun_domain_test.go') diff --git a/mailgun/resource_mailgun_domain_test.go b/mailgun/resource_mailgun_domain_test.go index 952e013..a129b1f 100644 --- a/mailgun/resource_mailgun_domain_test.go +++ b/mailgun/resource_mailgun_domain_test.go @@ -21,7 +21,7 @@ type fullDomain struct { } func getFullDomain(mg *mailgun.MailgunImpl, domainName string) (*fullDomain, error) { - ctx, cancel := context.WithTimeout(context.Background(), time.Second*30) + ctx, cancel := context.WithTimeout(context.Background(), time.Second*120) defer cancel() mg = mailgun.NewMailgun(domainName, mg.APIKey()) @@ -42,7 +42,8 @@ func getFullDomain(mg *mailgun.MailgunImpl, domainName string) (*fullDomain, err return nil, fmt.Errorf("Error Getting mailgun domain tracking Details for %s: Error: %s", domainName, err) } - ipAddress, err := mg.ListDomainIPS(ctx) + ipAddress, err := getIps(ctx, mg) + if err != nil { return nil, fmt.Errorf("Error Getting mailgun domain ips2 for %s: Error: %s", domainName, err) } @@ -209,7 +210,7 @@ func testAccDomainCheckDestroy(domain *fullDomain) resource.TestCheckFunc { mg := testAccProvider.Meta().(*mailgun.MailgunImpl) ctx, cancel := context.WithTimeout(context.Background(), time.Second*30) defer cancel() - + time.Sleep(5 * time.Second) _, err := mg.GetDomain(ctx, domain.domainResponse.Domain.Name) if err == nil { return fmt.Errorf("domain still exists") -- cgit v1.2.3