kareem 2.3.0 → 2.3.1

package/.travis.yml

@@ -1,5 +1,7 @@
 language: node_js
 node_js:
+  - "12"
+  - "10"
   - "9"
   - "8"
   - "7"

package/index.js

@@ -395,6 +395,10 @@ Kareem.prototype.pre = function(name, isAsync, fn, error, unshift) {
     ++pres.numAsync;
   }
 
+  if (typeof fn !== 'function') {
+    throw new Error('pre() requires a function, got "' + typeof fn + '"');
+  }
+
   if (unshift) {
     pres.unshift(Object.assign({}, options, { fn: fn, isAsync: isAsync }));
   } else {
@@ -413,6 +417,10 @@ Kareem.prototype.post = function(name, options, fn, unshift) {
     options = {};
   }
 
+  if (typeof fn !== 'function') {
+    throw new Error('post() requires a function, got "' + typeof fn + '"');
+  }
+
   if (unshift) {
     hooks.unshift(Object.assign({}, options, { fn: fn }));
   } else {

package/package.json

@@ -1,31 +1,21 @@
 {
   "name": "kareem",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "description": "Next-generation take on pre/post function hooks",
   "main": "index.js",
   "scripts": {
-    "test": "./node_modules/mocha/bin/mocha ./test/*",
-    "test-travis": "./node_modules/istanbul/lib/cli.js cover ./node_modules/mocha/bin/_mocha -- -R spec ./test/*"
+    "test": "mocha ./test/*",
+    "test-travis": "nyc mocha ./test/*"
   },
   "repository": {
     "type": "git",
     "url": "git://github.com/vkarpov15/kareem.git"
   },
   "devDependencies": {
-    "acquit": "0.5.1",
-    "gulp": "3.8.10",
-    "gulp-mocha": "2.0.0",
-    "gulp-jscs": "1.4.0",
-    "istanbul": "0.4.5",
-    "jscs": "1.9.0",
-    "mocha": "3.2.0",
+    "nyc": "11.x",
+    "mocha": "5.x",
     "standard-version": "4.4.0"
   },
-  "jscsConfig": {
-    "preset": "airbnb",
-    "requireMultipleVarDecl": null,
-    "disallowMultipleVarDecl": true
-  },
   "author": "Valeri Karpov <val@karpov.io>",
   "license": "Apache-2.0"
 }

package/test/post.test.js

@@ -40,6 +40,10 @@ describe('execPost', function() {
     assert.equal(hooks._posts.get('cook')[0].bar, 'baz');
   });
 
+  it('throws error if no function', function() {
+    assert.throws(() => hooks.post('test'), /got "undefined"/);
+  });
+ 
   it('multiple posts', function(done) {
     hooks.post('cook', function(eggs, callback) {
       setTimeout(

package/test/pre.test.js

@@ -65,6 +65,10 @@ describe('execPre', function() {
     assert.strictEqual(hooks._pres.get('cook')[1].fn, f1);
   });
 
+  it('throws error if no function', function() {
+    assert.throws(() => hooks.pre('test'), /got "undefined"/);
+  });
+
   it('arbitrary options', function() {
     const f1 = function() {};
     const f2 = function() {};