Fix declareOpts test
Summary: When we updated joi, the error message was changed. I removed the content to prevent similar errors in the future. Reviewed By: @amasad Differential Revision: D2424048
This commit is contained in:
parent
ce47e56b7b
commit
d3800c6615
|
@ -61,7 +61,7 @@ describe('declareOpts', function() {
|
||||||
|
|
||||||
expect(function() {
|
expect(function() {
|
||||||
validate({});
|
validate({});
|
||||||
}).toThrow('Error validating module options: foo is required');
|
}).toThrow();
|
||||||
});
|
});
|
||||||
|
|
||||||
it('should throw on invalid type', function() {
|
it('should throw on invalid type', function() {
|
||||||
|
@ -74,7 +74,7 @@ describe('declareOpts', function() {
|
||||||
|
|
||||||
expect(function() {
|
expect(function() {
|
||||||
validate({foo: 'lol'});
|
validate({foo: 'lol'});
|
||||||
}).toThrow('Error validating module options: foo must be a number');
|
}).toThrow();
|
||||||
});
|
});
|
||||||
|
|
||||||
it('should throw on extra options', function() {
|
it('should throw on extra options', function() {
|
||||||
|
@ -87,6 +87,6 @@ describe('declareOpts', function() {
|
||||||
|
|
||||||
expect(function() {
|
expect(function() {
|
||||||
validate({foo: 1, lol: 1});
|
validate({foo: 1, lol: 1});
|
||||||
}).toThrow('Error validating module options: lol is not allowed');
|
}).toThrow();
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
Loading…
Reference in New Issue