Filter SSM items to require a name (#18)

Co-authored-by: pebers <[email protected]>
please-build · Jun 17, 2022 · 9b4d14c · 9b4d14c
1 parent 3ea2eda
commit 9b4d14c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/read_test.go b/read_test.go
@@ -488,6 +488,9 @@ func TestStringStringMapSection(t *testing.T) {
 	if val, present := res.Section["key3"]; !present || val != "" {
 		t.Errorf("res.Section.Name=%q present=%v; want %q", res.Section["key3"], present, "")
 	}
+	if len(res.Section) != 3 {
+		t.Errorf("Wrong number of items in section; expected 3, got %d", len(res.Section))
+	}
 }
 
 func TestFatalOnly(t *testing.T) {

diff --git a/set.go b/set.go
@@ -239,10 +239,12 @@ func set(c *warnings.Collector, cfg interface{}, sect, sub, name string,
 			if vSect.IsNil() {
 				vSect.Set(reflect.MakeMap(vst))
 			}
-			if sub != "" {
-				vSect.SetMapIndex(reflect.ValueOf(sub+" "+name), reflect.ValueOf(value))
-			} else {
-				vSect.SetMapIndex(reflect.ValueOf(name), reflect.ValueOf(value))
+			if name != "" {
+				if sub != "" {
+					vSect.SetMapIndex(reflect.ValueOf(sub+" "+name), reflect.ValueOf(value))
+				} else {
+					vSect.SetMapIndex(reflect.ValueOf(name), reflect.ValueOf(value))
+				}
 			}
 			return nil
 		}