fix(dependencies): bumped to the latest version of joi

Author: travi

package-lock.json

@@ -75,18 +75,19 @@
     "nyc": "^14.1.1",
     "proxyquire": "^2.1.3",
     "rollup": "^1.20.3",
+    "rollup-plugin-auto-external": "^2.0.0",
     "rollup-plugin-babel": "^3.0.7",
     "rollup-plugin-node-resolve": "^5.2.0",
     "sinon": "^7.4.2"
   },
   "dependencies": {
+    "@hapi/joi": "^16.1.1",
     "@octokit/rest": "^16.28.9",
     "boom": "^7.3.0",
     "btoa": "1.2.1",
     "delay": "^4.3.0",
     "hoek": "^6.1.3",
     "http-status-codes": "1.3.2",
-    "joi": "^14.3.1",
     "lodash.groupby": "4.6.0",
     "lodash.values": "4.3.0"
   }

package.json

@@ -1,18 +1,12 @@
 /* eslint import/no-extraneous-dependencies: ["error", {"devDependencies": true}] */
 import babel from 'rollup-plugin-babel';
 import nodeResolve from 'rollup-plugin-node-resolve';
+import autoExternal from 'rollup-plugin-auto-external';
 
 export default {
   input: 'src/plugin.js',
-  external: [
-    'http-status-codes',
-    'delay',
-    'joi',
-    'hoek',
-    'boom',
-    '@octokit/rest'
-  ],
   plugins: [
+    autoExternal(),
     nodeResolve({
       module: true,
       jsnext: true

rollup.config.js

@@ -1,12 +1,13 @@
-import * as joi from 'joi';
+import * as joi from '@hapi/joi';
 import hoek from 'hoek';
 import validatePayloadAndProcess from './handler';
 
 function validate(options) {
-  const validated = joi.validate(options, joi.object({
+  const schema = joi.object({
     github: joi.object({token: joi.string().required()}).required(),
     acceptAction: joi.string().valid('merge', 'squash', 'rebase').required()
-  }).required());
+  }).required();
+  const validated = schema.validate(options);
 
   hoek.assert(!validated.error, validated.error);
 

src/plugin.js

@@ -60,35 +60,30 @@ suite('plugin', () => {
 
     test('that an error is thrown if the github config is not an object', () => assert.isRejected(
       plugin.register({}, {github: ''}, next),
-      '"github" must be an object'
+      '"github" must be of type object'
     ));
 
     test('that an error is thrown if the github token is not provided', () => assert.isRejected(
       plugin.register({}, {github: {}}, next),
-      '"token" is required'
+      '"github.token" is required'
     ));
 
     test('that an error is thrown if the github token is an inappropriate type', () => assert.isRejected(
       plugin.register({}, {github: {token: any.integer()}}, next),
-      '"token" must be a string'
+      '"github.token" must be a string'
     ));
 
     test(
       'that an error is thrown if the `acceptAction` setting is not provided',
       () => assert.isRejected(
         plugin.register({}, {github: {token: any.string()}}, next),
-        'child "acceptAction" fails because ["acceptAction" is required]'
+        '"acceptAction" is required'
       )
     );
 
-    test('that an error is thrown if the accept-action is not provided as an appropriate type', () => assert.isRejected(
-      plugin.register({}, {github: {token: any.string()}, acceptAction: any.integer()}, next),
-      'child "acceptAction" fails because ["acceptAction" must be a string]'
-    ));
-
     test('that an error is thrown if an invalid accept-action is not provided', () => assert.isRejected(
       plugin.register({}, {github: {token: any.string()}, acceptAction: any.string()}, next),
-      'child "acceptAction" fails because ["acceptAction" must be one of [merge, squash, rebase]]'
+      '"acceptAction" must be one of [merge, squash, rebase]'
     ));
 
     test('that no error is thrown if a valid accept-type is provided', () => Promise.all([