Merge pull request #18 from alpacaaa/patch-string

String objects should be considered valid strings.

Author: jquense

src/boolean.js

@@ -19,7 +19,7 @@ function BooleanSchema(){
 inherits(BooleanSchema, MixedSchema, {
 
   _typeCheck(v){ 
-    return typeof v === 'boolean' 
+    return (typeof v === 'boolean') || (typeof v === 'object' && v instanceof Boolean)
   }
 })
 

src/number.js

@@ -22,7 +22,10 @@ function NumberSchema(){
 inherits(NumberSchema, SchemaObject, {
 
   _typeCheck(v) {
-    return typeof v === 'number' && !(v !== +v) //isNaN check
+    if ( typeof v === 'number' && !(v !== +v) ) return true
+    if ( typeof v === 'object' && v instanceof Number ) return true
+
+    return false
   },
 
   min(min, msg) {

src/string.js

@@ -24,7 +24,7 @@ function StringSchema(){
 inherits(StringSchema, MixedSchema, {
 
   _typeCheck(value) {
-     return typeof value === 'string'
+     return (typeof value === 'string') || (typeof value === 'object' && value instanceof String)
   },
 
   required(msg){
@@ -99,4 +99,4 @@ inherits(StringSchema, MixedSchema, {
         test: val => val == null || val === val.toUpperCase()
       })
   }
-})
+})

test/bool.js

@@ -40,6 +40,7 @@ describe('Boolean types', function(){
     inst.isType('true').should.equal(false)
     inst.isType(NaN).should.equal(false)
     inst.isType(34545).should.equal(false)
+    inst.isType(new Boolean(false)).should.equal(true)
     chai.expect(
       inst.isType(null)).to.equal(false)
     inst.nullable().isType(null).should.equal(true)

test/number.js

@@ -48,6 +48,7 @@ describe('Number types', function(){
     var inst = number()
 
     inst.isType(5).should.equal(true)
+    inst.isType(new Number(5)).should.equal(true)
     inst.isType(false).should.equal(false)
     inst.isType(null).should.equal(false)
     inst.isType(NaN).should.equal(false)

test/string.js

@@ -52,6 +52,7 @@ describe('String types', function(){
     var inst = string()
 
     inst.isType('5').should.equal(true)
+    inst.isType(new String('5')).should.equal(true)
     inst.isType(false).should.equal(false)
     inst.isType(null).should.equal(false)
     inst.nullable(false).isType(null).should.equal(false)
@@ -135,4 +136,4 @@ describe('String types', function(){
         .should.eventually.equal(false)
     ])
   })
-})
+})