joi 4.8.1 → 4.9.0

package/test/object.js

@@ -27,7 +27,7 @@ describe('object', function () {
 
         Joi.object().validate('{"hi":true}', function (err, value) {
 
-            expect(err).to.not.exist;
+            expect(err).to.not.exist();
             expect(value.hi).to.equal(true);
             done();
         });
@@ -37,7 +37,7 @@ describe('object', function () {
 
         Joi.object().validate('a string', function (err, value) {
 
-            expect(err).to.exist;
+            expect(err).to.exist();
             expect(value).to.equal('a string');
             done();
         });
@@ -92,11 +92,11 @@ describe('object', function () {
 
         Joi.object().validate({ a: 4 }, function (err, value) {
 
-            expect(err).to.not.exist;
+            expect(err).to.not.exist();
 
             Joi.object(undefined).validate({ a: 4 }, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 done();
             });
         });
@@ -106,7 +106,7 @@ describe('object', function () {
 
         Joi.object(null).validate({ a: 4 }, function (err, value) {
 
-            expect(err).to.not.exist;
+            expect(err).to.not.exist();
             done();
         });
     });
@@ -133,8 +133,8 @@ describe('object', function () {
 
         Joi.object({ a: Joi.object() }).validate(undefined, function (err, value) {
 
-            expect(err).to.not.exist;
-            expect(value).to.not.exist;
+            expect(err).to.not.exist();
+            expect(value).to.not.exist();
             done();
         });
     });
@@ -143,7 +143,7 @@ describe('object', function () {
 
         Joi.object().validate([1, 2, 3], function (err, value) {
 
-            expect(err).to.exist;
+            expect(err).to.exist();
             done();
         });
     });
@@ -203,6 +203,18 @@ describe('object', function () {
         ], done);
     });
 
+    it('should validate constructor when type is set', function (done) {
+
+        var schema = Joi.object().type(RegExp);
+        Helper.validate(schema, [
+            [{ item: 'something' }, false],
+            ['', false],
+            [new Date(), false],
+            [/abcd/, true],
+            [new RegExp(), true]
+        ], done);
+    });
+
     it('should traverse an object and validate all properties in the top level', function (done) {
 
         var schema = Joi.object({
@@ -308,7 +320,7 @@ describe('object', function () {
         var obj = { a: 5, b: 'value' };
         schema.validate(obj, function (err, value) {
 
-            expect(err).to.exist;
+            expect(err).to.exist();
             done();
         });
     });
@@ -333,10 +345,10 @@ describe('object', function () {
             var b = a.keys();
             a.validate({ b: 3 }, function (err, value) {
 
-                expect(err).to.exist;
+                expect(err).to.exist();
                 b.validate({ b: 3 }, function (err, value) {
 
-                    expect(err).to.not.exist;
+                    expect(err).to.not.exist();
                     done();
                 });
             });
@@ -348,10 +360,10 @@ describe('object', function () {
             var b = a.keys({});
             a.validate({ b: 3 }, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 b.validate({ b: 3 }, function (err, value) {
 
-                    expect(err).to.exist;
+                    expect(err).to.exist();
                     done();
                 });
             });
@@ -363,10 +375,10 @@ describe('object', function () {
             var b = a.keys({ b: 2 });
             a.validate({ a: 1, b: 2 }, function (err, value) {
 
-                expect(err).to.exist;
+                expect(err).to.exist();
                 b.validate({ a: 1, b: 2 }, function (err, value) {
 
-                    expect(err).to.not.exist;
+                    expect(err).to.not.exist();
                     done();
                 });
             });
@@ -418,7 +430,7 @@ describe('object', function () {
 
             Joi.compile(schema).validate({ test1: 'a', test2: 'b' }, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 done();
             });
         });
@@ -440,7 +452,7 @@ describe('object', function () {
 
             Joi.object().rename('a', 'b').rename('c', 'b').rename('d', 'b').options({ abortEarly: false }).validate({ a: 1, c: 1, d: 1 }, function (err, value) {
 
-                expect(err).to.exist;
+                expect(err).to.exist();
                 expect(err.message).to.equal('value cannot rename child c because multiple renames are disabled and another key was already renamed to b. value cannot rename child d because multiple renames are disabled and another key was already renamed to b');
                 done();
             });
@@ -457,7 +469,7 @@ describe('object', function () {
 
             Joi.compile(schema).validate(obj, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 expect(value.a).to.equal(10);
                 expect(value.b).to.equal(10);
                 done();
@@ -485,7 +497,7 @@ describe('object', function () {
 
             schema.validate({ test: 'b', test1: 'a' }, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 done();
             });
         });
@@ -502,7 +514,7 @@ describe('object', function () {
             var data = { arr: [{ uno: '1', dos: '2' }] };
             Joi.object(schema).validate(data, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 expect(value.arr[0].one).to.equal('1');
                 expect(value.arr[0].two).to.equal('2');
                 done();
@@ -519,8 +531,8 @@ describe('object', function () {
 
             schema1.validate(input1, function (err1, value1) {
 
-                expect(err1).to.not.exist;
-                expect(value1.b).to.not.exist;
+                expect(err1).to.not.exist();
+                expect(value1.b).to.not.exist();
                 expect(value1.a).to.equal(5);
 
                 var schema2 = Joi.object({ a: Joi.number(), b: Joi.any() }).rename('b', 'a');
@@ -528,8 +540,8 @@ describe('object', function () {
 
                 schema2.validate(input2, function (err2, value2) {
 
-                    expect(err2).to.not.exist;
-                    expect(value2.b).to.not.exist;
+                    expect(err2).to.not.exist();
+                    expect(value2.b).to.not.exist();
                     expect(value2.a).to.equal(5);
 
                     done();
@@ -547,7 +559,7 @@ describe('object', function () {
 
             Joi.validate(input, schema, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 expect(value.foo2).to.equal('test');
 
                 done();
@@ -563,8 +575,8 @@ describe('object', function () {
 
             schema.validate(input, function (err, value) {
 
-                expect(err).to.not.exist;
-                expect(value['']).to.not.exist;
+                expect(err).to.not.exist();
+                expect(value['']).to.not.exist();
                 expect(value.notEmpty).to.equal('something');
                 done();
             });
@@ -580,7 +592,7 @@ describe('object', function () {
 
             schema.validate(input, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 expect(Object.keys(value)).to.include('a');
                 expect(value.a).to.equal('something');
                 done();
@@ -689,7 +701,7 @@ describe('object', function () {
 
             Joi.validate({ bb: 'y', 5: 'x' }, schema, { abortEarly: false }, function (err, value) {
 
-                expect(err).to.exist;
+                expect(err).to.exist();
                 expect(err.message).to.equal('5 must be a boolean. bb must be one of x');
 
                 Helper.validate(schema, [
@@ -714,7 +726,7 @@ describe('object', function () {
 
             Joi.validate({ x: { bb: 'y', 5: 'x' } }, schema, { abortEarly: false }, function (err, value) {
 
-                expect(err).to.exist;
+                expect(err).to.exist();
                 expect(err.message).to.equal('5 must be a boolean. bb must be one of x');
                 done();
             });
@@ -726,7 +738,7 @@ describe('object', function () {
 
             Joi.validate({ a: 5 }, schema, { abortEarly: false }, function (err, value) {
 
-                expect(err).to.exist;
+                expect(err).to.exist();
                 expect(err.message).to.equal('a is not allowed');
                 done();
             });
@@ -738,7 +750,7 @@ describe('object', function () {
 
             Joi.validate({ a1: 5, a2: 6 }, schema, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 done();
             });
         });
@@ -871,7 +883,7 @@ describe('object', function () {
                 }
             }).validate({ a: { b: { c: 1 } } }, function (err, value) {
 
-                expect(err).to.exist;
+                expect(err).to.exist();
                 expect(err.message).to.equal('value must contain at least one of x, y');
                 done();
             });
@@ -894,7 +906,7 @@ describe('object', function () {
 
             schema.validate({ a: { b: 'x', c: 5 }, d: { e: 6 } }, function (err, value) {
 
-                expect(err).to.exist;
+                expect(err).to.exist();
                 expect(err.message).to.equal('value validation failed because d.e failed to equal to a.c');
 
                 Helper.validate(schema, [
@@ -917,7 +929,7 @@ describe('object', function () {
 
             schema.validate({ a: { b: 'x', c: 5 }, d: { e: 6 } }, function (err, value) {
 
-                expect(err).to.exist;
+                expect(err).to.exist();
                 expect(err.message).to.equal('value validation failed because d.e failed to equal to a.c');
 
                 Helper.validate(schema, [
@@ -978,10 +990,69 @@ describe('object', function () {
                 }
             }, function (err) {
 
-                expect(err).to.exist;
+                expect(err).to.exist();
                 expect(err.message).to.equal('value validation failed because d.e failed to pass the assertion test');
                 done();
             });
         });
     });
+
+    describe('#type', function () {
+
+        it('uses constructor name for default type name', function (done) {
+
+            function Foo () {}
+
+            var schema = Joi.object().type(Foo);
+            schema.validate({}, function (err) {
+
+                expect(err).to.exist();
+                expect(err.message).to.equal('value must be an instance of Foo');
+                done();
+            });
+        });
+
+        it('uses custom type name if supplied', function (done) {
+
+            var Foo = function () {};
+
+            var schema = Joi.object().type(Foo, 'Bar');
+            schema.validate({}, function (err) {
+
+                expect(err).to.exist();
+                expect(err.message).to.equal('value must be an instance of Bar');
+                done();
+            });
+        });
+
+        it('overrides constructor name with custom name', function (done) {
+
+            function Foo () {}
+
+            var schema = Joi.object().type(Foo, 'Bar');
+            schema.validate({}, function (err) {
+
+                expect(err).to.exist();
+                expect(err.message).to.equal('value must be an instance of Bar');
+                done();
+            });
+        });
+
+        it('throws when constructor is not a function', function (done) {
+
+            expect(function () {
+
+                var schema = Joi.object().type('');
+            }).to.throw('type must be a constructor function');
+            done();
+        });
+
+        it('uses the constructor name in the schema description', function (done) {
+
+            var description = Joi.object().type(RegExp).describe();
+
+            expect(description.rules).to.deep.include({ name: 'type', arg: 'RegExp' });
+            done();
+        });
+    });
 });

package/test/ref.js

@@ -32,7 +32,7 @@ describe('ref', function () {
 
         schema.validate({ a: 5, b: 6 }, function (err, value) {
 
-            expect(err).to.exist;
+            expect(err).to.exist();
             expect(err.message).to.equal('a must be one of ref:b');
 
             Helper.validate(schema, [
@@ -53,7 +53,7 @@ describe('ref', function () {
 
         schema.validate({ a: 5, '': 6 }, function (err, value) {
 
-            expect(err).to.exist;
+            expect(err).to.exist();
             expect(err.message).to.equal('a must be one of ref:');
 
             Helper.validate(schema, [
@@ -76,7 +76,7 @@ describe('ref', function () {
 
         schema.validate({ a: 5, b: { c: 6 } }, function (err, value) {
 
-            expect(err).to.exist;
+            expect(err).to.exist();
             expect(err.message).to.equal('a must be one of ref:b.c');
 
             Helper.validate(schema, [
@@ -103,7 +103,7 @@ describe('ref', function () {
         var input = { a: 5, b: { c: 5 } };
         schema.validate(input, function (err, value) {
 
-            expect(err).to.not.exist;
+            expect(err).to.not.exist();
 
             var parent = Joi.object({
                 e: schema
@@ -111,7 +111,7 @@ describe('ref', function () {
 
             parent.validate({ e: input }, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 done();
             });
         });
@@ -131,7 +131,7 @@ describe('ref', function () {
 
         schema.validate({ a: 5, b: { c: 5 } }, function (err, value) {
 
-            expect(err).to.not.exist;
+            expect(err).to.not.exist();
             done();
         });
     });
@@ -147,7 +147,7 @@ describe('ref', function () {
 
         ab.validate({ a: { c: '5' }, b: 5 }, function (err, value) {
 
-            expect(err).to.not.exist;
+            expect(err).to.not.exist();
 
             var ba = Joi.object({
                 b: Joi.ref('a.c'),
@@ -158,7 +158,7 @@ describe('ref', function () {
 
             ba.validate({ a: { c: '5' }, b: 5 }, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 done();
             });
         });
@@ -173,7 +173,7 @@ describe('ref', function () {
 
         schema.validate({ b: 6 }, function (err, value) {
 
-            expect(err).to.not.exist;
+            expect(err).to.not.exist();
             expect(value).to.deep.equal({ a: 6, b: 6 });
             done();
         });
@@ -188,7 +188,7 @@ describe('ref', function () {
 
         ab.validate({ b: '6' }, function (err, value) {
 
-            expect(err).to.not.exist;
+            expect(err).to.not.exist();
             expect(value).to.deep.equal({ a: 6, b: 6 });
 
             var ba = Joi.object({
@@ -198,7 +198,7 @@ describe('ref', function () {
 
             ba.validate({ b: '6' }, function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 expect(value).to.deep.equal({ a: 6, b: 6 });
                 done();
             });
@@ -263,7 +263,7 @@ describe('ref', function () {
 
         Joi.validate({ b: 6 }, schema, { context: { x: 22 } }, function (err, value) {
 
-            expect(err).to.not.exist;
+            expect(err).to.not.exist();
             expect(value).to.deep.equal({ a: 22, b: 6 });
             done();
         });
@@ -278,7 +278,7 @@ describe('ref', function () {
 
         Joi.validate({ b: 6 }, schema, { context: { x: 22 } }, function (err, value) {
 
-            expect(err).to.not.exist;
+            expect(err).to.not.exist();
             expect(value).to.deep.equal({ a: 22, b: 6 });
             done();
         });
@@ -293,7 +293,7 @@ describe('ref', function () {
 
         Joi.validate({ a: 5, b: 6 }, schema, { context: { x: 22 } }, function (err, value) {
 
-            expect(err).to.exist;
+            expect(err).to.exist();
             expect(err.message).to.equal('a must be one of context:x');
 
             Helper.validateOptions(schema, [
@@ -353,7 +353,7 @@ describe('ref', function () {
     it('describes schema with ref', function (done) {
 
         var desc = Joi.compile(Joi.ref('a.b')).describe();
-        expect(Joi.isRef(desc.valids[0])).to.be.true;
+        expect(Joi.isRef(desc.valids[0])).to.be.true();
         done();
     });
 

package/test/string.js

@@ -295,7 +295,7 @@ describe('string', function () {
             var schema = Joi.string().lowercase();
             schema.validate('UPPER TO LOWER', function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 expect(value).to.equal('upper to lower');
                 done();
             });
@@ -333,7 +333,7 @@ describe('string', function () {
             var schema = Joi.string().uppercase();
             schema.validate('lower to upper', function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 expect(value).to.equal('LOWER TO UPPER');
                 done();
             });
@@ -370,7 +370,7 @@ describe('string', function () {
             var schema = Joi.string().trim();
             schema.validate(' trim this ', function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 expect(value).to.equal('trim this');
                 done();
             });
@@ -381,7 +381,7 @@ describe('string', function () {
             var schema = Joi.string().trim().allow('');
             schema.validate('     ', function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 expect(value).to.equal('');
                 done();
             });
@@ -650,7 +650,7 @@ describe('string', function () {
             var text = Joi.string().invalid('joi');
             text.validate('joi', function (err, value) {
 
-                expect(err).to.exist;
+                expect(err).to.exist();
                 done();
             });
         });
@@ -660,7 +660,7 @@ describe('string', function () {
             var text = Joi.string().allow('hapi');
             text.validate('result', function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 done();
             });
         });
@@ -670,7 +670,7 @@ describe('string', function () {
             var text = Joi.string().min(3);
             text.validate('joi', function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
                 done();
             });
         });
@@ -680,11 +680,11 @@ describe('string', function () {
             var text = Joi.string().min(3).required();
             text.validate('joi', function (err, value) {
 
-                expect(err).to.not.exist;
+                expect(err).to.not.exist();
 
                 text.validate('', function (err, value) {
 
-                    expect(err).to.exist;
+                    expect(err).to.exist();
                     done();
                 });
             });