update deps and test on node 11

Author: cjihrig

.travis.yml

@@ -2,7 +2,8 @@ language: node_js
 
 node_js:
   - "8"
-  - "9"
+  - "10"
+  - "11"
   - "node"
 
 sudo: false

lib/index.js

@@ -167,10 +167,10 @@ internals.addMethod = function (names, fn) {
     internals.addMethod(word, method);
 });
 
-internals.addMethod('error', function (/*type, message*/) {
+internals.addMethod('error', function (...args /*type, message*/) {
 
-    const type = arguments.length && typeof arguments[0] !== 'string' && !(arguments[0] instanceof RegExp) ? arguments[0] : Error;
-    const lastArg = arguments[1] || arguments[0];
+    const type = args.length && typeof args[0] !== 'string' && !(args[0] instanceof RegExp) ? args[0] : Error;
+    const lastArg = args[1] || args[0];
     const message = typeof lastArg === 'string' || lastArg instanceof RegExp ? lastArg : null;
     const err = this._ref;
 
@@ -366,13 +366,13 @@ internals.satisfy = function (validator) {
 internals.addMethod(['satisfy', 'satisfies'], internals.satisfy);
 
 
-internals.throw = function (/* type, message */) {
+internals.throw = function (...args /* type, message */) {
 
     internals.assert(this, typeof this._ref === 'function', 'Can only assert throw on functions');
-    internals.assert(this, !this._flags.not || !arguments.length, 'Cannot specify arguments when expecting not to throw');
+    internals.assert(this, !this._flags.not || !args.length, 'Cannot specify arguments when expecting not to throw');
 
-    const type = arguments.length && typeof arguments[0] !== 'string' && !(arguments[0] instanceof RegExp) ? arguments[0] : null;
-    const lastArg = arguments[1] || arguments[0];
+    const type = args.length && typeof args[0] !== 'string' && !(args[0] instanceof RegExp) ? args[0] : null;
+    const lastArg = args[1] || args[0];
     const message = typeof lastArg === 'string' || lastArg instanceof RegExp ? lastArg : null;
 
     let thrown = false;
@@ -402,13 +402,13 @@ internals.throw = function (/* type, message */) {
 internals.addMethod(['throw', 'throws'], internals.throw);
 
 
-internals.reject = async function (/* type, message */) {
+internals.reject = async function (...args/* type, message */) {
 
     try {
         internals.assert(this, internals.isPromise(this._ref), 'Can only assert reject on promises');
 
-        const type = arguments.length && typeof arguments[0] !== 'string' && !(arguments[0] instanceof RegExp) ? arguments[0] : null;
-        const lastArg = arguments[1] || arguments[0];
+        const type = args.length && typeof args[0] !== 'string' && !(args[0] instanceof RegExp) ? args[0] : null;
+        const lastArg = args[1] || args[0];
         const message = typeof lastArg === 'string' || lastArg instanceof RegExp ? lastArg : null;
 
         let thrown = null;

package.json

@@ -16,7 +16,7 @@
         "hoek": "5.x.x"
     },
     "devDependencies": {
-        "lab": "15.x.x",
+        "lab": "17.x.x",
         "markdown-toc": "1.1.x"
     },
     "scripts": {

test/index.js

@@ -311,7 +311,7 @@ describe('expect()', () => {
         }
 
         Code.settings.comparePrototypes = origPrototype;
-        Hoek.assert(exception2.message === 'Expected {} to equal specified value: {}', exception2);
+        Hoek.assert(exception2.message === `Expected {} to equal specified value: ${Util.format(Object.create(null))}`, exception2);
     });
 
     describe('assertion', () => {
@@ -322,7 +322,7 @@ describe('expect()', () => {
 
                 const grab = function () {
 
-                    return arguments;
+                    return arguments;   // eslint-disable-line prefer-rest-params
                 };
 
                 try {
@@ -407,7 +407,7 @@ describe('expect()', () => {
             it('validates correct type', () => {
 
                 try {
-                    Code.expect(new Buffer([1])).to.be.a.buffer();
+                    Code.expect(Buffer.from([1])).to.be.a.buffer();
                 }
                 catch (err) {
                     var exception = err;
@@ -759,7 +759,7 @@ describe('expect()', () => {
             it('invalidates incorrect type', () => {
 
                 try {
-                    Code.expect(new Buffer([20])).to.be.an.object();
+                    Code.expect(Buffer.from([20])).to.be.an.object();
                 }
                 catch (err) {
                     var exception = err;
@@ -1054,6 +1054,7 @@ describe('expect()', () => {
                 catch (err) {
                     var exception1 = err;
                 }
+
                 Hoek.assert(exception1.message === 'Can only assert include with a single parameter', exception1);
 
                 try {
@@ -1239,7 +1240,7 @@ describe('expect()', () => {
             it('validates buffer', () => {
 
                 try {
-                    Code.expect(new Buffer('')).to.be.empty();
+                    Code.expect(Buffer.from('')).to.be.empty();
                 }
                 catch (err) {
                     var exception = err;
@@ -1302,7 +1303,7 @@ describe('expect()', () => {
             it('validates buffer', () => {
 
                 try {
-                    Code.expect(new Buffer('a')).to.have.length(1);
+                    Code.expect(Buffer.from('a')).to.have.length(1);
                 }
                 catch (err) {
                     var exception = err;