Merge pull request #1 from invopop/each-by-fix

Each function passes value or pointer to By rules

Author: samlown

each.go

@@ -78,10 +78,8 @@ func (r EachRule) getInterface(value reflect.Value) interface{} {
 		if value.IsNil() {
 			return nil
 		}
-		return value.Elem().Interface()
-	default:
-		return value.Interface()
 	}
+	return value.Interface()
 }
 
 func (r EachRule) getString(value reflect.Value) string {

each_test.go

@@ -72,3 +72,16 @@ func TestEachWithContext(t *testing.T) {
 		assertError(t, test.err, err, test.tag)
 	}
 }
+
+func TestEachAndBy(t *testing.T) {
+	var byAddr bool
+	var s string
+	_ = Each(By(func(v interface{}) error {
+		_, byAddr = v.(*string)
+		return nil
+	})).Validate([]*string{&s})
+
+	if !byAddr {
+		t.Fatal("slice of pointers does not get passed to `By` function by ref")
+	}
+}