fix(object): fix error when updating bucket with empty tagsSet

Co-authored-by: DuarteRoberto <[email protected]>
Co-authored-by: paulden <[email protected]>
Co-authored-by: pomverte <[email protected]>
Co-authored-by: ralbon <[email protected]>
Co-authored-by: schaltiemi <[email protected]>

Signed-off-by: Julien ALEXANDRE <[email protected]>

Author: ndrpnt

scaleway/resource_object_bucket.go

@@ -172,12 +172,18 @@ func resourceScalewayObjectBucketUpdate(ctx context.Context, d *schema.ResourceD
 	if d.HasChange("tags") {
 		tagsSet := expandObjectBucketTags(d.Get("tags"))
 
-		_, err = s3Client.PutBucketTaggingWithContext(ctx, &s3.PutBucketTaggingInput{
-			Bucket: scw.StringPtr(bucketName),
-			Tagging: &s3.Tagging{
-				TagSet: tagsSet,
-			},
-		})
+		if len(tagsSet) > 0 {
+			_, err = s3Client.PutBucketTaggingWithContext(ctx, &s3.PutBucketTaggingInput{
+				Bucket: scw.StringPtr(bucketName),
+				Tagging: &s3.Tagging{
+					TagSet: tagsSet,
+				},
+			})
+		} else {
+			_, err = s3Client.DeleteBucketTaggingWithContext(ctx, &s3.DeleteBucketTaggingInput{
+				Bucket: scw.StringPtr(bucketName),
+			})
+		}
 		if err != nil {
 			return diag.FromErr(err)
 		}

scaleway/resource_object_bucket_test.go

@@ -47,6 +47,10 @@ func TestAccScalewayObjectBucket_Basic(t *testing.T) {
 					resource "scaleway_object_bucket" "ams-bucket" {
 						name = "%s"
 						region = "nl-ams"
+						tags = {
+							foo = "bar"
+							baz = "qux"
+						}
 					}
 
 					resource "scaleway_object_bucket" "par-bucket" {
@@ -62,6 +66,9 @@ func TestAccScalewayObjectBucket_Basic(t *testing.T) {
 					resource.TestCheckResourceAttr("scaleway_object_bucket.base", "endpoint", fmt.Sprintf("https://%s.s3.%s.scw.cloud", testBucketName, "fr-par")),
 
 					resource.TestCheckResourceAttr("scaleway_object_bucket.ams-bucket", "name", testBucketNameAms),
+					resource.TestCheckResourceAttr("scaleway_object_bucket.ams-bucket", "tags.%", "2"),
+					resource.TestCheckResourceAttr("scaleway_object_bucket.ams-bucket", "tags.foo", "bar"),
+					resource.TestCheckResourceAttr("scaleway_object_bucket.ams-bucket", "tags.baz", "qux"),
 					resource.TestCheckResourceAttr("scaleway_object_bucket.ams-bucket", "endpoint", fmt.Sprintf("https://%s.s3.%s.scw.cloud", testBucketNameAms, "nl-ams")),
 
 					resource.TestCheckResourceAttr("scaleway_object_bucket.par-bucket", "name", testBucketNamePar),
@@ -74,11 +81,23 @@ func TestAccScalewayObjectBucket_Basic(t *testing.T) {
 						name = "%s"
 						acl = "%s"
 					}
-				`, testBucketName, testBucketUpdatedACL),
+
+					resource "scaleway_object_bucket" "ams-bucket" {
+						name = "%s"
+						region = "nl-ams"
+						tags = {
+							foo = "bar"
+						}
+					}
+				`, testBucketName, testBucketUpdatedACL, testBucketNameAms),
 				Check: resource.ComposeTestCheckFunc(
 					resource.TestCheckResourceAttr("scaleway_object_bucket.base", "name", testBucketName),
 					resource.TestCheckResourceAttr("scaleway_object_bucket.base", "acl", testBucketUpdatedACL),
 					resource.TestCheckResourceAttr("scaleway_object_bucket.base", "tags.%", "0"),
+
+					resource.TestCheckResourceAttr("scaleway_object_bucket.ams-bucket", "name", testBucketNameAms),
+					resource.TestCheckResourceAttr("scaleway_object_bucket.ams-bucket", "tags.%", "1"),
+					resource.TestCheckResourceAttr("scaleway_object_bucket.ams-bucket", "tags.foo", "bar"),
 				),
 			},
 		},