diff --git a/tests/unit/validators/length-test.js b/tests/unit/validators/length-test.js index e26eaf0..656adbc 100644 --- a/tests/unit/validators/length-test.js +++ b/tests/unit/validators/length-test.js @@ -15,7 +15,7 @@ test('no options', function(assert) { }); test('allow blank', function(assert) { - assert.expect(2); + assert.expect(4); options = { allowBlank: true, @@ -25,12 +25,18 @@ test('allow blank', function(assert) { result = validate('', cloneOptions(options)); assert.equal(processResult(result), true); + result = validate(undefined, cloneOptions(options)); + assert.equal(processResult(result), true); + + result = validate(' ', cloneOptions(options)); + assert.equal(processResult(result), 'This field is too short (minimum is 5 characters)'); + result = validate('test', cloneOptions(options)); assert.equal(processResult(result), 'This field is too short (minimum is 5 characters)'); }); test('allow none', function(assert) { - assert.expect(2); + assert.expect(3); options = { allowNone: true @@ -40,8 +46,12 @@ test('allow none', function(assert) { assert.equal(processResult(result), true); options.allowNone = false; + result = validate(null, cloneOptions(options)); assert.equal(processResult(result), 'This field is invalid'); + + result = validate('', cloneOptions(options)); + assert.equal(processResult(result), true); }); test('is', function(assert) { diff --git a/tests/unit/validators/presence-test.js b/tests/unit/validators/presence-test.js index 37541bd..3c27d42 100644 --- a/tests/unit/validators/presence-test.js +++ b/tests/unit/validators/presence-test.js @@ -25,13 +25,25 @@ test('presence - value blank', function(assert) { assert.equal(processResult(result), true); }); -test('presence with ignoreBlank - value blank', function(assert) { +test('presence - null value', function(assert) { assert.expect(1); + options = { presence: true }; + + result = validate(null, cloneOptions(options)); + assert.equal(processResult(result), 'This field can\'t be blank'); +}); + +test('presence with ignoreBlank - value blank', function(assert) { + assert.expect(2); + options = { presence: true, ignoreBlank: true }; result = validate(' ', cloneOptions(options)); assert.equal(processResult(result), 'This field can\'t be blank'); + + result = validate(null, cloneOptions(options)); + assert.equal(processResult(result), 'This field can\'t be blank'); }); test('presence - value not present', function(assert) {