provider/statuscake: fix StatusCake tests (#10660)

* Update vendored statuscake SDK

* Set all attributes when upserting statuscake tests

1 files changed, 44 insertions, 3 deletions

diff --git a/resource_statuscaketest_test.go b/resource_statuscaketest_test.go
index bec1a45..de30254 100644
--- a/resource_statuscaketest_test.go
+++ b/resource_statuscaketest_test.go
@@ -22,6 +22,7 @@ func TestAccStatusCake_basic(t *testing.T) {
                                 Config: testAccTestConfig_basic,
                                 Check: resource.ComposeTestCheckFunc(
                                         testAccTestCheckExists("statuscake_test.google", &test),
+                                        testAccTestCheckAttributes("statuscake_test.google", &test),
                                 ),
                         },
                 },
@@ -47,9 +48,10 @@ func TestAccStatusCake_withUpdate(t *testing.T) {
                                 Config: testAccTestConfig_update,
                                 Check: resource.ComposeTestCheckFunc(
                                         testAccTestCheckExists("statuscake_test.google", &test),
+                                        testAccTestCheckAttributes("statuscake_test.google", &test),
                                         resource.TestCheckResourceAttr("statuscake_test.google", "check_rate", "500"),
                                         resource.TestCheckResourceAttr("statuscake_test.google", "paused", "true"),
-                                        resource.TestCheckResourceAttr("statuscake_test.google", "contact_id", "23456"),
+                                        resource.TestCheckResourceAttr("statuscake_test.google", "contact_id", "0"),
                                 ),
                         },
                 },
@@ -75,7 +77,7 @@ func testAccTestCheckExists(rn string, test *statuscake.Test) resource.TestCheck
 
                 gotTest, err := client.Tests().Detail(testId)
                 if err != nil {
-                        return fmt.Errorf("error getting project: %s", err)
+                        return fmt.Errorf("error getting test: %s", err)
                 }
 
                 *test = *gotTest
@@ -84,6 +86,46 @@ func testAccTestCheckExists(rn string, test *statuscake.Test) resource.TestCheck
         }
 }
 
+func testAccTestCheckAttributes(rn string, test *statuscake.Test) resource.TestCheckFunc {
+        return func(s *terraform.State) error {
+                attrs := s.RootModule().Resources[rn].Primary.Attributes
+
+                check := func(key, stateValue, testValue string) error {
+                        if testValue != stateValue {
+                                return fmt.Errorf("different values for %s in state (%s) and in statuscake (%s)",
+                                        key, stateValue, testValue)
+                        }
+                        return nil
+                }
+
+                for key, value := range attrs {
+                        var err error
+
+                        switch key {
+                        case "website_name":
+                                err = check(key, value, test.WebsiteName)
+                        case "website_url":
+                                err = check(key, value, test.WebsiteURL)
+                        case "check_rate":
+                                err = check(key, value, strconv.Itoa(test.CheckRate))
+                        case "test_type":
+                                err = check(key, value, test.TestType)
+                        case "paused":
+                                err = check(key, value, strconv.FormatBool(test.Paused))
+                        case "timeout":
+                                err = check(key, value, strconv.Itoa(test.Timeout))
+                        case "contact_id":
+                                err = check(key, value, strconv.Itoa(test.ContactID))
+                        }
+
+                        if err != nil {
+                                return err
+                        }
+                }
+                return nil
+        }
+}
+
 func testAccTestCheckDestroy(test *statuscake.Test) resource.TestCheckFunc {
         return func(s *terraform.State) error {
                 client := testAccProvider.Meta().(*statuscake.Client)
@@ -113,6 +155,5 @@ resource "statuscake_test" "google" {
         test_type = "HTTP"
         check_rate = 500
         paused = true
-        contact_id = 23456
 }
 `