Generate tagging tests for CloudWatch and CloudWatch Logs

.changelog/37668.txt

@@ -0,0 +1,3 @@
+```release-note:bug
+resource/aws_cloudwatch_log_group: Correctly handles tag updates with empty string tags
+```

internal/generate/tagstests/main.go

@@ -226,7 +226,9 @@ type ResourceDatum struct {
 	PackageProviderNameUpper  string
 	Name                      string
 	TypeName                  string
+	DestroyTakesT             bool
 	ExistsTypeName            string
+	ExistsTakesT              bool
 	FileName                  string
 	Generator                 string
 	NoImport                  bool
@@ -394,6 +396,15 @@ func (v *visitor) processFuncDecl(funcDecl *ast.FuncDecl) {
 						d.AlternateRegionProvider = b
 					}
 				}
+
+				if attr, ok := args.Keyword["destroyTakesT"]; ok {
+					if b, err := strconv.ParseBool(attr); err != nil {
+						v.errs = append(v.errs, fmt.Errorf("invalid destroyTakesT value: %q at %s. Should be boolean value.", attr, fmt.Sprintf("%s.%s", v.packageName, v.functionName)))
+						continue
+					} else {
+						d.DestroyTakesT = b
+					}
+				}
 				if attr, ok := args.Keyword["existsType"]; ok {
 					if typeName, importSpec, err := parseIdentifierSpec(attr); err != nil {
 						v.errs = append(v.errs, fmt.Errorf("%s: %w", attr, fmt.Sprintf("%s.%s", v.packageName, v.functionName), err))
@@ -405,6 +416,14 @@ func (v *visitor) processFuncDecl(funcDecl *ast.FuncDecl) {
 						}
 					}
 				}
+				if attr, ok := args.Keyword["existsTakesT"]; ok {
+					if b, err := strconv.ParseBool(attr); err != nil {
+						v.errs = append(v.errs, fmt.Errorf("invalid existsTakesT value: %q at %s. Should be boolean value.", attr, fmt.Sprintf("%s.%s", v.packageName, v.functionName)))
+						continue
+					} else {
+						d.ExistsTakesT = b
+					}
+				}
 				if attr, ok := args.Keyword["generator"]; ok {
 					if attr == "false" {
 						generatorSeen = true

internal/generate/tagstests/test.go.gtpl

@@ -23,7 +23,7 @@
 {{ define "TestCaseSetupNoProviders" -}}
 	PreCheck:     func() { acctest.PreCheck(ctx, t){{ if .PreCheck }}; testAccPreCheck(ctx, t){{ end }} },
 	ErrorCheck:   acctest.ErrorCheck(t, names.{{ .PackageProviderNameUpper }}ServiceID),
-	CheckDestroy: testAccCheck{{ .Name }}Destroy(ctx),
+	CheckDestroy: testAccCheck{{ .Name }}Destroy(ctx{{ if .DestroyTakesT }}, t{{ end }}),
 {{- end }}
 
 {{ define "ImportBody" }}
@@ -48,7 +48,7 @@
 {{- end }}
 
 {{ define "ExistsCheck" }}
-	testAccCheck{{ .Name }}Exists(ctx, resourceName{{ if .ExistsTypeName}}, &v{{ end }}),
+	testAccCheck{{ .Name }}Exists(ctx, {{ if .ExistsTakesT }}t,{{ end }} resourceName{{ if .ExistsTypeName}}, &v{{ end }}),
 {{ end }}
 
 {{ define "AdditionalTfVars" -}}