Merge pull request #30688 from hashicorp/backport/alisdair/fix-sum-fu…

…nc/annually-ample-dove Backport of functions: Fix sum() of all strings into v1.1
hashicorp · Mar 16, 2022 · c722eeb · c722eeb
2 parents b6aa8ae + 7a12c06
commit c722eeb
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/internal/lang/funcs/collection.go b/internal/lang/funcs/collection.go
@@ -527,6 +527,10 @@ var SumFunc = function.New(&function.Spec{
 		if s.IsNull() {
 			return cty.NilVal, function.NewArgErrorf(0, "argument must be list, set, or tuple of number values")
 		}
+		s, err = convert.Convert(s, cty.Number)
+		if err != nil {
+			return cty.NilVal, function.NewArgErrorf(0, "argument must be list, set, or tuple of number values")
+		}
 		for _, v := range arg[1:] {
 			if v.IsNull() {
 				return cty.NilVal, function.NewArgErrorf(0, "argument must be list, set, or tuple of number values")

diff --git a/internal/lang/funcs/collection_test.go b/internal/lang/funcs/collection_test.go
@@ -1629,6 +1629,15 @@ func TestSum(t *testing.T) {
 			cty.NilVal,
 			"can't compute sum of opposing infinities",
 		},
+		{
+			cty.ListVal([]cty.Value{
+				cty.StringVal("1"),
+				cty.StringVal("2"),
+				cty.StringVal("3"),
+			}),
+			cty.NumberIntVal(6),
+			"",
+		},
 	}
 
 	for _, test := range tests {