Fix panic caused by incorrect type assert.

fastly/block_fastly_service_v1_gzip.go

@@ -127,10 +127,24 @@ func (h *GzipServiceAttributeHandler) Process(d *schema.ResourceData, latestVers
 		// this and so we've updated the below code to convert the type asserted
 		// int into a uint before passing the value to gofastly.Uint().
 		if v, ok := modified["content_types"]; ok {
-			opts.ContentTypes = gofastly.String(v.(string))
+			set := v.(*schema.Set)
+			if len(set.List()) > 0 {
+				var s []string
+				for _, elem := range set.List() {
+					s = append(s, elem.(string))
+				}
+				opts.ContentTypes = gofastly.String(strings.Join(s, " "))
+			}
 		}
 		if v, ok := modified["extensions"]; ok {
-			opts.Extensions = gofastly.String(v.(string))
+			set := v.(*schema.Set)
+			if len(set.List()) > 0 {
+				var s []string
+				for _, elem := range set.List() {
+					s = append(s, elem.(string))
+				}
+				opts.Extensions = gofastly.String(strings.Join(s, " "))
+			}
 		}
 		if v, ok := modified["cache_condition"]; ok {
 			opts.CacheCondition = gofastly.String(v.(string))

fastly/block_fastly_service_v1_gzip_test.go

@@ -127,6 +127,13 @@ func TestAccFastlyServiceV1_gzips_basic(t *testing.T) {
 		ContentTypes:   "text/javascript application/x-javascript application/javascript text/css text/html",
 	}
 
+	log4 := gofastly.Gzip{
+		ServiceVersion: 1,
+		Name:           "all",
+		Extensions:     "css",
+		ContentTypes:   "text/javascript application/x-javascript",
+	}
+
 	resource.ParallelTest(t, resource.TestCase{
 		PreCheck:     func() { testAccPreCheck(t) },
 		Providers:    testAccProviders,
@@ -145,7 +152,7 @@ func TestAccFastlyServiceV1_gzips_basic(t *testing.T) {
 			},
 
 			{
-				Config: testAccServiceV1GzipsConfig_update(name, domainName1),
+				Config: testAccServiceV1GzipsConfig_delete_create(name, domainName1),
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckServiceV1Exists("fastly_service_v1.foo", &service),
 					testAccCheckFastlyServiceV1GzipsAttributes(&service, []*gofastly.Gzip{&log3}),
@@ -155,6 +162,18 @@ func TestAccFastlyServiceV1_gzips_basic(t *testing.T) {
 						"fastly_service_v1.foo", "gzip.#", "1"),
 				),
 			},
+
+			{
+				Config: testAccServiceV1GzipsConfig_update(name, domainName1),
+				Check: resource.ComposeTestCheckFunc(
+					testAccCheckServiceV1Exists("fastly_service_v1.foo", &service),
+					testAccCheckFastlyServiceV1GzipsAttributes(&service, []*gofastly.Gzip{&log4}),
+					resource.TestCheckResourceAttr(
+						"fastly_service_v1.foo", "name", name),
+					resource.TestCheckResourceAttr(
+						"fastly_service_v1.foo", "gzip.#", "1"),
+				),
+			},
 		},
 	})
 }
@@ -240,7 +259,7 @@ resource "fastly_service_v1" "foo" {
 }`, name, domain)
 }
 
-func testAccServiceV1GzipsConfig_update(name, domain string) string {
+func testAccServiceV1GzipsConfig_delete_create(name, domain string) string {
 	return fmt.Sprintf(`
 resource "fastly_service_v1" "foo" {
   name = "%s"
@@ -272,3 +291,32 @@ resource "fastly_service_v1" "foo" {
   force_destroy = true
 }`, name, domain)
 }
+
+func testAccServiceV1GzipsConfig_update(name, domain string) string {
+	return fmt.Sprintf(`
+resource "fastly_service_v1" "foo" {
+  name = "%s"
+
+  domain {
+    name    = "%s"
+    comment = "tf-testing-domain"
+  }
+
+  backend {
+    address = "aws.amazon.com"
+    name    = "amazon docs"
+  }
+
+  gzip {
+    name       = "all"
+    extensions = ["css"]
+
+    content_types = [
+      "application/x-javascript",
+      "text/javascript",
+    ]
+  }
+
+  force_destroy = true
+}`, name, domain)
+}