Merge pull request #164 from jpenna/master

Enum validates `false` value

README.md

@@ -193,6 +193,8 @@ If the `len` property is combined with the `min` and `max` range properties, `le
 
 #### Enumerable
 
+> Since version 3.0.0 if you want to validate the values `0` or `false` inside `enum` types, you have to include them explicitly.
+
 To validate a value from a list of possible values use the `enum` type with a `enum` property listing the valid values for the field, for example:
 
 ```javascript
@@ -420,6 +422,18 @@ var Schema = require('async-validator').default;
 Schema.warning = function(){};
 ```
 
+### How to check if it is `true`
+
+Use `enum` type passing `true` as option.
+
+```js
+{
+  type: 'enum',
+  enum: [true],
+  message: '',
+}
+```
+
 ## Test Case
 
 ```

__tests__/enum.spec.js

@@ -0,0 +1,18 @@
+import Schema from '../src/';
+
+describe('enum', () => {
+  it('run validation on `false`', (done) => {
+    new Schema({
+      v: {
+        type: 'enum',
+        enum: [true],
+      },
+    }).validate({
+      v: false,
+    }, (errors) => {
+      expect(errors.length).toBe(1);
+      expect(errors[0].message).toBe('v must be one of true');
+      done();
+    });
+  });
+});

package.json

@@ -1,6 +1,6 @@
 {
   "name": "async-validator",
-  "version": "2.0.1",
+  "version": "3.0.0",
   "description": "validate form asynchronous",
   "keywords": [
     "validator",

src/validator/enum.js

@@ -22,7 +22,7 @@ function enumerable(rule, value, callback, source, options) {
       return callback();
     }
     rules.required(rule, value, source, errors, options);
-    if (value) {
+    if (value !== undefined) {
       rules[ENUM](rule, value, source, errors, options);
     }
   }