add validation schema to shot

lib/index.js

@@ -2,12 +2,19 @@
 
 // Load modules
 
+const Joi = require('joi');
 const Request = require('./request');
 const Response = require('./response');
-
+const Schema = require('./schema');
 
 exports.inject = function (dispatchFunc, options, callback) {
 
+    Joi.assert({
+        dispatchFunc: dispatchFunc,
+        options: options,
+        callback: callback
+    }, Schema);
+
     const req = new Request(options);
     const res = new Response(req, callback);
 

lib/schema.js

@@ -0,0 +1,29 @@
+'use strict';
+
+const Joi = require('joi');
+
+const internals = {};
+
+internals.url = Joi.alternatives(Joi.string().required(), Joi.object().keys({
+    protocol: Joi.string(),
+    hostname: Joi.string(),
+    port: Joi.any(),
+    pathname: Joi.string().required(),
+    query: Joi.any()
+}).required());
+
+internals.options = Joi.object().keys({
+    url: internals.url.required(),
+    headers: Joi.object(),
+    payload: Joi.any(),
+    simulate: Joi.object(), // add these
+    authority: Joi.string(),
+    remoteAddress: Joi.string(),
+    method: Joi.string().regex(/^[a-zA-Z0-9!#\$%&'\*\+\-\.^_`\|~]+$/)
+}).min(1);
+
+module.exports = Joi.object().keys({
+    dispatchFunc: Joi.func().required(),
+    options: Joi.alternatives(internals.options, internals.url).required(),
+    callback: Joi.func()
+});

package.json

@@ -13,6 +13,9 @@
   "engines": {
     "node": ">=4.0.0"
   },
+  "dependencies": {
+    "joi": "9.x.x"
+  },
   "devDependencies": {
     "code": "3.x.x",
     "lab": "10.x.x"

test/index.js

@@ -750,6 +750,31 @@ describe('_read()', () => {
             done();
         });
     });
+
+    it('errors for invalid input options', (done) => {
+
+        try {
+            Shot.inject({}, {}, (res) => {});
+        }
+        catch (err) {
+
+            expect(err).to.exist();
+            expect(err.isJoi).to.be.true();
+            done();
+        }
+    });
+
+    it('errors for missing url', (done) => {
+
+        try {
+            Shot.inject((req, res) => {}, {}, (res) => {});
+        }
+        catch (err) {
+            expect(err).to.exist();
+            expect(err.isJoi).to.be.true();
+            done();
+        }
+    });
 });