Merge pull request #22044 from saveriomiroddi/sav-allow_null_values_in_compact_function
Allow null values in Compact function
This commit is contained in:
commit
c1844c093d
|
@ -246,7 +246,7 @@ var CompactFunc = function.New(&function.Spec{
|
||||||
|
|
||||||
for it := listVal.ElementIterator(); it.Next(); {
|
for it := listVal.ElementIterator(); it.Next(); {
|
||||||
_, v := it.Element()
|
_, v := it.Element()
|
||||||
if v.AsString() == "" {
|
if v.IsNull() || v.AsString() == "" {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
outputList = append(outputList, v)
|
outputList = append(outputList, v)
|
||||||
|
|
|
@ -572,6 +572,7 @@ func TestCompact(t *testing.T) {
|
||||||
cty.StringVal("test"),
|
cty.StringVal("test"),
|
||||||
cty.StringVal(""),
|
cty.StringVal(""),
|
||||||
cty.StringVal("test"),
|
cty.StringVal("test"),
|
||||||
|
cty.NullVal(cty.String),
|
||||||
}),
|
}),
|
||||||
cty.ListVal([]cty.Value{
|
cty.ListVal([]cty.Value{
|
||||||
cty.StringVal("test"),
|
cty.StringVal("test"),
|
||||||
|
@ -588,6 +589,14 @@ func TestCompact(t *testing.T) {
|
||||||
cty.ListValEmpty(cty.String),
|
cty.ListValEmpty(cty.String),
|
||||||
false,
|
false,
|
||||||
},
|
},
|
||||||
|
{
|
||||||
|
cty.ListVal([]cty.Value{
|
||||||
|
cty.NullVal(cty.String),
|
||||||
|
cty.NullVal(cty.String),
|
||||||
|
}),
|
||||||
|
cty.ListValEmpty(cty.String),
|
||||||
|
false,
|
||||||
|
},
|
||||||
{
|
{
|
||||||
cty.ListValEmpty(cty.String),
|
cty.ListValEmpty(cty.String),
|
||||||
cty.ListValEmpty(cty.String),
|
cty.ListValEmpty(cty.String),
|
||||||
|
@ -610,6 +619,7 @@ func TestCompact(t *testing.T) {
|
||||||
cty.StringVal("test"),
|
cty.StringVal("test"),
|
||||||
cty.UnknownVal(cty.String),
|
cty.UnknownVal(cty.String),
|
||||||
cty.StringVal(""),
|
cty.StringVal(""),
|
||||||
|
cty.NullVal(cty.String),
|
||||||
}),
|
}),
|
||||||
cty.UnknownVal(cty.List(cty.String)),
|
cty.UnknownVal(cty.List(cty.String)),
|
||||||
false,
|
false,
|
||||||
|
|
Loading…
Reference in New Issue