Skip to content

Commit bca56c4

Browse files
authored
[confmaptest] Add a test on empty slice to nil (#10966)
#### Description Add a test on empty slices being sanitized to `nil` in `ToStringMap`
1 parent a80ce1a commit bca56c4

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

confmap/confmaptest/configtest_test.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,13 @@ func TestLoadConf(t *testing.T) {
3030
assert.Equal(t, map[string]any{"floating": 3.14}, cfg.ToStringMap())
3131
}
3232

33+
func TestToStringMapSanitizeEmptySlice(t *testing.T) {
34+
cfg, err := LoadConf(filepath.Join("testdata", "empty-slice.yaml"))
35+
require.NoError(t, err)
36+
var nilSlice []interface{}
37+
assert.Equal(t, map[string]any{"slice": nilSlice}, cfg.ToStringMap())
38+
}
39+
3340
func TestValidateProviderScheme(t *testing.T) {
3441
assert.NoError(t, ValidateProviderScheme(&schemeProvider{scheme: "file"}))
3542
assert.NoError(t, ValidateProviderScheme(&schemeProvider{scheme: "s3"}))
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
slice: [] # empty slices are sanitized to nil in ToStringMap

0 commit comments

Comments
 (0)