fix arrLength validation

README.md

@@ -264,6 +264,14 @@ Use "%s" in the message to have the field name or label printed in the message:
 
         Checks the field value max length.
 
+    arrLength(minLength, maxLength[, messages])
+    - minLength (integer): The min character to test for.
+    - maxLength (integer): The max character to test for.
+    - messages(array): Custom messages for min/max error
+
+        For example ["Too short string", "Too long string"]
+        If messages is not provided, default will be ["%s is too short"", "%s is too long""]
+        Checks the field value in min/max length.
 
 *Other*
 

lib/field.js

@@ -106,17 +106,17 @@ Field.prototype.array = function () {
   return this.array();
 };
 
-Field.prototype.arrLength = function (from, to) {
-  return this.add(function (arr) {
+Field.prototype.arrLength = function (from, to, messages) {
+  return this.add(function (value) {
     if (value.length < from) {
-      return { error: message || e.message || "%s is too short" };
+      return { error: (messages && messages[0]) || "%s is too short" };
     }
     if (value.length > to) {
-      return { error: message || e.message || "%s is too long" };
+      return { error: (messages && messages[1]) || "%s is too long" };
     }
     return { valid: true };
   });
-}
+};
 
 // HYBRID METHODS
 

test/validate.test.js

@@ -722,5 +722,48 @@ module.exports = {
       validate("field[even][more][inner]").required().equals("fail")
     )(request, {});
     assert.equal(request.form.errors.length, 2);
+  },
+
+  'validation : arrLength : failed default message': function() {
+    var request = { body: { field: "value" }};
+    form(validate("field").arrLength(10, 20))(request, {});
+    assert.equal(request.form.errors.length, 1);
+    assert.equal(request.form.errors[0], "field is too short");
+
+    request = { body: { field: "value is so long so it is failed" }};
+    form(validate("field").arrLength(4, 20))(request, {});
+    assert.equal(request.form.errors.length, 1);
+    assert.equal(request.form.errors[0], "field is too long");
+
+    request = { body: { field: "value is so long so it is failed" }};
+    form(validate("field").arrLength(10, 20, []))(request, {});
+    assert.equal(request.form.errors.length, 1);
+    assert.equal(request.form.errors[0], "field is too long");
+  },
+
+  'validation : arrLength : failed custom message': function() {
+    var request = { body: { field: "value" }};
+    form(validate("field").arrLength(10, 20, ["!!! %s SO SHORT!!!", "!!! %s SO LONG!!!"]))(request, {});
+    assert.equal(request.form.errors.length, 1);
+    assert.equal(request.form.errors[0], "!!! field SO SHORT!!!");
+
+    request = { body: { field: "value is so long so it is failed" }};
+    form(validate("field").arrLength(10, 20, ["!!! %s SO SHORT!!!", "!!! %s SO LONG!!!"]))(request, {});
+    assert.equal(request.form.errors.length, 1);
+    assert.equal(request.form.errors[0], "!!! field SO LONG!!!");
+  },
+
+  'validation : arrLength : success': function() {
+    var request = { body: { field: "value" }};
+    form(validate("field").arrLength(1, 5))(request, {});
+    assert.equal(request.form.errors.length, 0);
+
+    request = { body: { field: "value" }};
+    form(validate("field").arrLength(1, 15, ["short", "long"]))(request, {});
+    assert.equal(request.form.errors.length, 0);
+
+    request = { body: { field: "" }};
+    form(validate("field").arrLength(0, 15, ["short", "long"]))(request, {});
+    assert.equal(request.form.errors.length, 0);
   }
 };