Merge pull request #38 from drone/FFM-1290_group_match_warn_log

(FFM-1290) Ignored empty evaliation attibute

Author: Andrew Hayes

evaluation/target.go

@@ -31,6 +31,9 @@ func (t Target) GetAttrValue(attr string) reflect.Value {
 
 // GetOperator returns interface based on attribute value
 func (t Target) GetOperator(attr string) (types.ValueType, error) {
+	if attr == "" {
+		return nil, nil
+	}
 	value := t.GetAttrValue(attr)
 	switch value.Kind() {
 	case reflect.Bool:

evaluation/target_test.go

@@ -59,6 +59,13 @@ func TestTarget_GetOperator(t1 *testing.T) {
 			"weight": 99.99,
 		}},
 			args: struct{ attr string }{attr: "weight"}, want: types.Number(99.99)},
+		{name: "empty operator", fields: struct {
+			Identifier string
+			Name       *string
+			Anonymous  bool
+			Attributes map[string]interface{}
+		}{Identifier: "harness", Name: nil, Anonymous: false, Attributes: map[string]interface{}{}},
+			args: struct{ attr string }{attr: ""}, want: nil},
 	}
 	for _, tt := range tests {
 		val := tt