kareem 2.3.0 → 2.3.4

package/.travis.yml

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

package/CHANGELOG.md

@@ -1,6 +1,19 @@
-# Change Log
+# Changelog
 
-All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
+<a name="2.3.4"></a>
+## 2.3.4 (2022-02-10)
+
+* perf: various performance improvements #27 #24 #23 #22 #21 #20
+
+<a name="2.3.3"></a>
+## 2.3.3 (2021-12-26)
+
+* fix: handle sync errors in `wrap()`
+
+<a name="2.3.2"></a>
+## 2.3.2 (2020-12-08)
+
+* fix: handle sync errors in pre hooks if there are multiple hooks
 
 <a name="2.3.0"></a>
 ## 2.3.0 (2018-09-24)

package/README.md

@@ -24,11 +24,9 @@ appropriate, giving you more fine-grained control over your function hooks.
 #### It runs without any hooks specified
 
 ```javascript
-    
-    hooks.execPre('cook', null, function() {
-      done();
-    });
-  
+hooks.execPre('cook', null, function() {
+  // ...
+});
 ```
 
 #### It runs basic serial pre hooks
@@ -38,44 +36,38 @@ when your pre hook is finished.
 
 
 ```javascript
-    
-    var count = 0;
-
-    hooks.pre('cook', function(done) {
-      ++count;
-      done();
-    });
-
-    hooks.execPre('cook', null, function() {
-      assert.equal(1, count);
-      done();
-    });
-  
+var count = 0;
+
+hooks.pre('cook', function(done) {
+  ++count;
+  done();
+});
+
+hooks.execPre('cook', null, function() {
+  assert.equal(1, count);
+});
 ```
 
 #### It can run multipe pre hooks
 
 ```javascript
-    
-    var count1 = 0;
-    var count2 = 0;
-
-    hooks.pre('cook', function(done) {
-      ++count1;
-      done();
-    });
-
-    hooks.pre('cook', function(done) {
-      ++count2;
-      done();
-    });
-
-    hooks.execPre('cook', null, function() {
-      assert.equal(1, count1);
-      assert.equal(1, count2);
-      done();
-    });
-  
+var count1 = 0;
+var count2 = 0;
+
+hooks.pre('cook', function(done) {
+  ++count1;
+  done();
+});
+
+hooks.pre('cook', function(done) {
+  ++count2;
+  done();
+});
+
+hooks.execPre('cook', null, function() {
+  assert.equal(1, count1);
+  assert.equal(1, count2);
+});
 ```
 
 #### It can run fully synchronous pre hooks
@@ -85,25 +77,22 @@ fully synchronous.
 
 
 ```javascript
-    
-    var count1 = 0;
-    var count2 = 0;
-
-    hooks.pre('cook', function() {
-      ++count1;
-    });
-
-    hooks.pre('cook', function() {
-      ++count2;
-    });
-
-    hooks.execPre('cook', null, function(error) {
-      assert.equal(null, error);
-      assert.equal(1, count1);
-      assert.equal(1, count2);
-      done();
-    });
-  
+var count1 = 0;
+var count2 = 0;
+
+hooks.pre('cook', function() {
+  ++count1;
+});
+
+hooks.pre('cook', function() {
+  ++count2;
+});
+
+hooks.execPre('cook', null, function(error) {
+  assert.equal(null, error);
+  assert.equal(1, count1);
+  assert.equal(1, count2);
+});
 ```
 
 #### It properly attaches context to pre hooks
@@ -112,27 +101,24 @@ Pre save hook functions are bound to the second parameter to `execPre()`
 
 
 ```javascript
-    
-    hooks.pre('cook', function(done) {
-      this.bacon = 3;
-      done();
-    });
-
-    hooks.pre('cook', function(done) {
-      this.eggs = 4;
-      done();
-    });
-
-    var obj = { bacon: 0, eggs: 0 };
-
-    // In the pre hooks, `this` will refer to `obj`
-    hooks.execPre('cook', obj, function(error) {
-      assert.equal(null, error);
-      assert.equal(3, obj.bacon);
-      assert.equal(4, obj.eggs);
-      done();
-    });
-  
+hooks.pre('cook', function(done) {
+  this.bacon = 3;
+  done();
+});
+
+hooks.pre('cook', function(done) {
+  this.eggs = 4;
+  done();
+});
+
+var obj = { bacon: 0, eggs: 0 };
+
+// In the pre hooks, `this` will refer to `obj`
+hooks.execPre('cook', obj, function(error) {
+  assert.equal(null, error);
+  assert.equal(3, obj.bacon);
+  assert.equal(4, obj.eggs);
+});
 ```
 
 #### It can execute parallel (async) pre hooks
@@ -144,38 +130,35 @@ async pre hooks have called `done()`.
 
 
 ```javascript
-    
-    hooks.pre('cook', true, function(next, done) {
-      this.bacon = 3;
-      next();
-      setTimeout(function() {
-        done();
-      }, 5);
-    });
-
-    hooks.pre('cook', true, function(next, done) {
-      next();
-      var _this = this;
-      setTimeout(function() {
-        _this.eggs = 4;
-        done();
-      }, 10);
-    });
-
-    hooks.pre('cook', function(next) {
-      this.waffles = false;
-      next();
-    });
-
-    var obj = { bacon: 0, eggs: 0 };
-
-    hooks.execPre('cook', obj, function() {
-      assert.equal(3, obj.bacon);
-      assert.equal(4, obj.eggs);
-      assert.equal(false, obj.waffles);
-      done();
-    });
-  
+hooks.pre('cook', true, function(next, done) {
+  this.bacon = 3;
+  next();
+  setTimeout(function() {
+    done();
+  }, 5);
+});
+
+hooks.pre('cook', true, function(next, done) {
+  next();
+  var _this = this;
+  setTimeout(function() {
+    _this.eggs = 4;
+    done();
+  }, 10);
+});
+
+hooks.pre('cook', function(next) {
+  this.waffles = false;
+  next();
+});
+
+var obj = { bacon: 0, eggs: 0 };
+
+hooks.execPre('cook', obj, function() {
+  assert.equal(3, obj.bacon);
+  assert.equal(4, obj.eggs);
+  assert.equal(false, obj.waffles);
+});
 ```
 
 #### It supports returning a promise
@@ -186,148 +169,162 @@ next middleware.
 
 
 ```javascript
-    
-    hooks.pre('cook', function() {
-      return new Promise(resolve => {
-        setTimeout(() => {
-          this.bacon = 3;
-          resolve();
-        }, 100);
-      });
-    });
-
-    var obj = { bacon: 0 };
-
-    hooks.execPre('cook', obj, function() {
-      assert.equal(3, obj.bacon);
-      done();
-    });
-  
+hooks.pre('cook', function() {
+  return new Promise(resolve => {
+    setTimeout(() => {
+      this.bacon = 3;
+      resolve();
+    }, 100);
+  });
+});
+
+var obj = { bacon: 0 };
+
+hooks.execPre('cook', obj, function() {
+  assert.equal(3, obj.bacon);
+});
 ```
 
 ## post hooks
 
+acquit:ignore:end
+
 #### It runs without any hooks specified
 
 ```javascript
-    
-    hooks.execPost('cook', null, [1], function(error, eggs) {
-      assert.ifError(error);
-      assert.equal(1, eggs);
-      done();
-    });
-  
+hooks.execPost('cook', null, [1], function(error, eggs) {
+  assert.ifError(error);
+  assert.equal(1, eggs);
+  done();
+});
 ```
 
 #### It executes with parameters passed in
 
 ```javascript
-    
-    hooks.post('cook', function(eggs, bacon, callback) {
-      assert.equal(1, eggs);
-      assert.equal(2, bacon);
-      callback();
-    });
-
-    hooks.execPost('cook', null, [1, 2], function(error, eggs, bacon) {
-      assert.ifError(error);
-      assert.equal(1, eggs);
-      assert.equal(2, bacon);
-      done();
-    });
-  
+hooks.post('cook', function(eggs, bacon, callback) {
+  assert.equal(1, eggs);
+  assert.equal(2, bacon);
+  callback();
+});
+
+hooks.execPost('cook', null, [1, 2], function(error, eggs, bacon) {
+  assert.ifError(error);
+  assert.equal(1, eggs);
+  assert.equal(2, bacon);
+});
 ```
 
 #### It can use synchronous post hooks
 
 ```javascript
-    
-    var execed = {};
-
-    hooks.post('cook', function(eggs, bacon) {
-      execed.first = true;
-      assert.equal(1, eggs);
-      assert.equal(2, bacon);
-    });
-
-    hooks.post('cook', function(eggs, bacon, callback) {
-      execed.second = true;
-      assert.equal(1, eggs);
-      assert.equal(2, bacon);
-      callback();
-    });
-
-    hooks.execPost('cook', null, [1, 2], function(error, eggs, bacon) {
-      assert.ifError(error);
-      assert.equal(2, Object.keys(execed).length);
-      assert.ok(execed.first);
-      assert.ok(execed.second);
-      assert.equal(1, eggs);
-      assert.equal(2, bacon);
-      done();
-    });
-  
+var execed = {};
+
+hooks.post('cook', function(eggs, bacon) {
+  execed.first = true;
+  assert.equal(1, eggs);
+  assert.equal(2, bacon);
+});
+
+hooks.post('cook', function(eggs, bacon, callback) {
+  execed.second = true;
+  assert.equal(1, eggs);
+  assert.equal(2, bacon);
+  callback();
+});
+
+hooks.execPost('cook', null, [1, 2], function(error, eggs, bacon) {
+  assert.ifError(error);
+  assert.equal(2, Object.keys(execed).length);
+  assert.ok(execed.first);
+  assert.ok(execed.second);
+  assert.equal(1, eggs);
+  assert.equal(2, bacon);
+});
+```
+
+#### It supports returning a promise
+
+You can also return a promise from your post hooks instead of calling
+`next()`. When the returned promise resolves, kareem will kick off the
+next middleware.
+
+
+```javascript
+hooks.post('cook', function(bacon) {
+  return new Promise(resolve => {
+    setTimeout(() => {
+      this.bacon = 3;
+      resolve();
+    }, 100);
+  });
+});
+
+var obj = { bacon: 0 };
+
+hooks.execPost('cook', obj, obj, function() {
+  assert.equal(obj.bacon, 3);
+});
 ```
 
 ## wrap()
 
+acquit:ignore:end
+
 #### It wraps pre and post calls into one call
 
 ```javascript
-    
-    hooks.pre('cook', true, function(next, done) {
-      this.bacon = 3;
-      next();
-      setTimeout(function() {
-        done();
-      }, 5);
-    });
-
-    hooks.pre('cook', true, function(next, done) {
-      next();
-      var _this = this;
-      setTimeout(function() {
-        _this.eggs = 4;
-        done();
-      }, 10);
-    });
-
-    hooks.pre('cook', function(next) {
-      this.waffles = false;
-      next();
-    });
-
-    hooks.post('cook', function(obj) {
-      obj.tofu = 'no';
-    });
-
-    var obj = { bacon: 0, eggs: 0 };
-
-    var args = [obj];
-    args.push(function(error, result) {
-      assert.ifError(error);
-      assert.equal(null, error);
-      assert.equal(3, obj.bacon);
-      assert.equal(4, obj.eggs);
-      assert.equal(false, obj.waffles);
-      assert.equal('no', obj.tofu);
-
-      assert.equal(obj, result);
-      done();
-    });
-
-    hooks.wrap(
-      'cook',
-      function(o, callback) {
-        assert.equal(3, obj.bacon);
-        assert.equal(4, obj.eggs);
-        assert.equal(false, obj.waffles);
-        assert.equal(undefined, obj.tofu);
-        callback(null, o);
-      },
-      obj,
-      args);
-  
+hooks.pre('cook', true, function(next, done) {
+  this.bacon = 3;
+  next();
+  setTimeout(function() {
+    done();
+  }, 5);
+});
+
+hooks.pre('cook', true, function(next, done) {
+  next();
+  var _this = this;
+  setTimeout(function() {
+    _this.eggs = 4;
+    done();
+  }, 10);
+});
+
+hooks.pre('cook', function(next) {
+  this.waffles = false;
+  next();
+});
+
+hooks.post('cook', function(obj) {
+  obj.tofu = 'no';
+});
+
+var obj = { bacon: 0, eggs: 0 };
+
+var args = [obj];
+args.push(function(error, result) {
+  assert.ifError(error);
+  assert.equal(null, error);
+  assert.equal(3, obj.bacon);
+  assert.equal(4, obj.eggs);
+  assert.equal(false, obj.waffles);
+  assert.equal('no', obj.tofu);
+
+  assert.equal(obj, result);
+});
+
+hooks.wrap(
+  'cook',
+  function(o, callback) {
+    assert.equal(3, obj.bacon);
+    assert.equal(4, obj.eggs);
+    assert.equal(false, obj.waffles);
+    assert.equal(undefined, obj.tofu);
+    callback(null, o);
+  },
+  obj,
+  args);
 ```
 
 ## createWrapper()
@@ -335,73 +332,70 @@ next middleware.
 #### It wraps wrap() into a callable function
 
 ```javascript
-    
-    hooks.pre('cook', true, function(next, done) {
-      this.bacon = 3;
-      next();
-      setTimeout(function() {
-        done();
-      }, 5);
-    });
-
-    hooks.pre('cook', true, function(next, done) {
-      next();
-      var _this = this;
-      setTimeout(function() {
-        _this.eggs = 4;
-        done();
-      }, 10);
-    });
-
-    hooks.pre('cook', function(next) {
-      this.waffles = false;
-      next();
-    });
-
-    hooks.post('cook', function(obj) {
-      obj.tofu = 'no';
-    });
-
-    var obj = { bacon: 0, eggs: 0 };
-
-    var cook = hooks.createWrapper(
-      'cook',
-      function(o, callback) {
-        assert.equal(3, obj.bacon);
-        assert.equal(4, obj.eggs);
-        assert.equal(false, obj.waffles);
-        assert.equal(undefined, obj.tofu);
-        callback(null, o);
-      },
-      obj);
-
-    cook(obj, function(error, result) {
-      assert.ifError(error);
-      assert.equal(3, obj.bacon);
-      assert.equal(4, obj.eggs);
-      assert.equal(false, obj.waffles);
-      assert.equal('no', obj.tofu);
-
-      assert.equal(obj, result);
-      done();
-    });
-  
+hooks.pre('cook', true, function(next, done) {
+  this.bacon = 3;
+  next();
+  setTimeout(function() {
+    done();
+  }, 5);
+});
+
+hooks.pre('cook', true, function(next, done) {
+  next();
+  var _this = this;
+  setTimeout(function() {
+    _this.eggs = 4;
+    done();
+  }, 10);
+});
+
+hooks.pre('cook', function(next) {
+  this.waffles = false;
+  next();
+});
+
+hooks.post('cook', function(obj) {
+  obj.tofu = 'no';
+});
+
+var obj = { bacon: 0, eggs: 0 };
+
+var cook = hooks.createWrapper(
+  'cook',
+  function(o, callback) {
+    assert.equal(3, obj.bacon);
+    assert.equal(4, obj.eggs);
+    assert.equal(false, obj.waffles);
+    assert.equal(undefined, obj.tofu);
+    callback(null, o);
+  },
+  obj);
+
+cook(obj, function(error, result) {
+  assert.ifError(error);
+  assert.equal(3, obj.bacon);
+  assert.equal(4, obj.eggs);
+  assert.equal(false, obj.waffles);
+  assert.equal('no', obj.tofu);
+
+  assert.equal(obj, result);
+});
 ```
 
 ## clone()
 
+acquit:ignore:end
+
 #### It clones a Kareem object
 
 ```javascript
-    
-    var k1 = new Kareem();
-    k1.pre('cook', function() {});
-    k1.post('cook', function() {});
-
-    var k2 = k1.clone();
-    assert.deepEqual(['cook'], Object.keys(k2._pres));
-    assert.deepEqual(['cook'], Object.keys(k2._posts));
-  
+var k1 = new Kareem();
+k1.pre('cook', function() {});
+k1.post('cook', function() {});
+
+var k2 = k1.clone();
+assert.deepEqual(Array.from(k2._pres.keys()), ['cook']);
+assert.deepEqual(Array.from(k2._posts.keys()), ['cook']);
 ```
 
 ## merge()
@@ -409,20 +403,18 @@ next middleware.
 #### It pulls hooks from another Kareem object
 
 ```javascript
-    
-    var k1 = new Kareem();
-    var test1 = function() {};
-    k1.pre('cook', test1);
-    k1.post('cook', function() {});
-
-    var k2 = new Kareem();
-    var test2 = function() {};
-    k2.pre('cook', test2);
-    var k3 = k2.merge(k1);
-    assert.equal(k3._pres['cook'].length, 2);
-    assert.equal(k3._pres['cook'][0].fn, test2);
-    assert.equal(k3._pres['cook'][1].fn, test1);
-    assert.equal(k3._posts['cook'].length, 1);
-  
+var k1 = new Kareem();
+var test1 = function() {};
+k1.pre('cook', test1);
+k1.post('cook', function() {});
+
+var k2 = new Kareem();
+var test2 = function() {};
+k2.pre('cook', test2);
+var k3 = k2.merge(k1);
+assert.equal(k3._pres.get('cook').length, 2);
+assert.equal(k3._pres.get('cook')[0].fn, test2);
+assert.equal(k3._pres.get('cook')[1].fn, test1);
+assert.equal(k3._posts.get('cook').length, 1);
 ```
 

package/docs.js

@@ -1,5 +1,7 @@
 var acquit = require('acquit');
 
+require('acquit-ignore')();
+
 var content = require('fs').readFileSync('./test/examples.test.js').toString();
 var blocks = acquit.parse(content);
 
@@ -29,7 +31,7 @@ for (var i = 0; i < blocks.length; ++i) {
       acquit.trimEachLine(it.comments[0]) + '\n\n' :
       '';
     mdOutput += '```javascript\n';
-    mdOutput += '    ' + it.code + '\n';
+    mdOutput += it.code + '\n';
     mdOutput += '```\n\n';
   }
 }

package/index.js

@@ -10,7 +10,7 @@ Kareem.prototype.execPre = function(name, context, args, callback) {
     callback = args;
     args = [];
   }
-  var pres = get(this._pres, name, []);
+  var pres = this._pres.get(name) || [];
   var numPres = pres.length;
   var numAsyncPres = pres.numAsync || 0;
   var currentPre = 0;
@@ -57,12 +57,13 @@ Kareem.prototype.execPre = function(name, context, args, callback) {
 
       callMiddlewareFunction(pre.fn, context, args, args[0]);
     } else {
-      let error = null;
       let maybePromise = null;
       try {
         maybePromise = pre.fn.call(context);
       } catch (err) {
-        error = err;
+        if (err != null) {
+          return callback(err);
+        }
       }
 
       if (isPromise(maybePromise)) {
@@ -74,11 +75,11 @@ Kareem.prototype.execPre = function(name, context, args, callback) {
             return;
           } else {
             return process.nextTick(function() {
-              callback(error);
+              callback(null);
             });
           }
         }
-        next(error);
+        next();
       }
     }
   };
@@ -108,7 +109,7 @@ Kareem.prototype.execPre = function(name, context, args, callback) {
 };
 
 Kareem.prototype.execPreSync = function(name, context, args) {
-  var pres = get(this._pres, name, []);
+  var pres = this._pres.get(name) || [];
   var numPres = pres.length;
 
   for (var i = 0; i < numPres; ++i) {
@@ -121,7 +122,7 @@ Kareem.prototype.execPost = function(name, context, args, options, callback) {
     callback = options;
     options = null;
   }
-  var posts = get(this._posts, name, []);
+  var posts = this._posts.get(name) || [];
   var numPosts = posts.length;
   var currentPost = 0;
 
@@ -150,7 +151,7 @@ Kareem.prototype.execPost = function(name, context, args, options, callback) {
 
     if (firstError) {
       if (post.length === numArgs + 2) {
-        var _cb = decorateNextFn(function(error) {
+        const _cb = decorateNextFn(function(error) {
           if (error) {
             firstError = error;
           }
@@ -209,7 +210,7 @@ Kareem.prototype.execPost = function(name, context, args, options, callback) {
           return callback.apply(null, [error].concat(args));
         }
 
-        next(error);
+        next();
       }
     }
   };
@@ -218,7 +219,7 @@ Kareem.prototype.execPost = function(name, context, args, options, callback) {
 };
 
 Kareem.prototype.execPostSync = function(name, context, args) {
-  const posts = get(this._posts, name, []);
+  const posts = this._posts.get(name) || [];
   const numPosts = posts.length;
 
   for (let i = 0; i < numPosts; ++i) {
@@ -241,14 +242,11 @@ Kareem.prototype.createWrapperSync = function(name, fn) {
 
 function _handleWrapError(instance, error, name, context, args, options, callback) {
   if (options.useErrorHandlers) {
-    var _options = { error: error };
-    return instance.execPost(name, context, args, _options, function(error) {
+    return instance.execPost(name, context, args, { error: error }, function(error) {
       return typeof callback === 'function' && callback(error);
     });
   } else {
-    return typeof callback === 'function' ?
-      callback(error) :
-      undefined;
+    return typeof callback === 'function' && callback(error);
   }
 }
 
@@ -275,7 +273,12 @@ Kareem.prototype.wrap = function(name, fn, context, args, options) {
 
     const end = (typeof lastArg === 'function' ? args.length - 1 : args.length);
     const numParameters = fn.length;
-    const ret = fn.apply(context, args.slice(0, end).concat(_cb));
+    let ret;
+    try {
+      ret = fn.apply(context, args.slice(0, end).concat(_cb));
+    } catch (err) {
+      return _cb(err);
+    }
 
     if (checkForPromise) {
       if (ret != null && typeof ret.then === 'function') {
@@ -294,8 +297,8 @@ Kareem.prototype.wrap = function(name, fn, context, args, options) {
     }
 
     function _cb() {
-      const args = arguments;
-      const argsWithoutError = Array.prototype.slice.call(arguments, 1);
+      const argsWithoutError = Array.from(arguments);
+      argsWithoutError.shift();
       if (options.nullResultByDefault && argsWithoutError.length === 0) {
         argsWithoutError.push(null);
       }
@@ -371,23 +374,21 @@ Kareem.prototype.createWrapper = function(name, fn, context, options) {
   }
   return function() {
     var _context = context || this;
-    var args = Array.prototype.slice.call(arguments);
-    _this.wrap(name, fn, _context, args, options);
+    _this.wrap(name, fn, _context, Array.from(arguments), options);
   };
 };
 
 Kareem.prototype.pre = function(name, isAsync, fn, error, unshift) {
   let options = {};
-  if (typeof isAsync === 'object' && isAsync != null) {
+  if (typeof isAsync === 'object' && isAsync !== null) {
     options = isAsync;
     isAsync = options.isAsync;
   } else if (typeof arguments[1] !== 'boolean') {
-    error = fn;
     fn = isAsync;
     isAsync = false;
   }
 
-  const pres = get(this._pres, name, []);
+  const pres = this._pres.get(name) || [];
   this._pres.set(name, pres);
 
   if (isAsync) {
@@ -395,6 +396,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 {
@@ -405,7 +410,7 @@ Kareem.prototype.pre = function(name, isAsync, fn, error, unshift) {
 };
 
 Kareem.prototype.post = function(name, options, fn, unshift) {
-  const hooks = get(this._posts, name, []);
+  const hooks = this._posts.get(name) || [];
 
   if (typeof options === 'function') {
     unshift = !!fn;
@@ -413,6 +418,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 {
@@ -442,7 +451,7 @@ Kareem.prototype.merge = function(other, clone) {
   var ret = clone ? this.clone() : this;
 
   for (let key of other._pres.keys()) {
-    const sourcePres = get(ret._pres, key, []);
+    const sourcePres = ret._pres.get(key) || [];
     const deduplicated = other._pres.get(key).
       // Deduplicate based on `fn`
       filter(p => sourcePres.map(_p => _p.fn).indexOf(p.fn) === -1);
@@ -452,7 +461,7 @@ Kareem.prototype.merge = function(other, clone) {
     ret._pres.set(key, combined);
   }
   for (let key of other._posts.keys()) {
-    const sourcePosts = get(ret._posts, key, []);
+    const sourcePosts = ret._posts.get(key) || [];
     const deduplicated = other._posts.get(key).
       filter(p => sourcePosts.indexOf(p) === -1);
     ret._posts.set(key, sourcePosts.concat(deduplicated));
@@ -461,13 +470,6 @@ Kareem.prototype.merge = function(other, clone) {
   return ret;
 };
 
-function get(map, key, def) {
-  if (map.has(key)) {
-    return map.get(key);
-  }
-  return def;
-}
-
 function callMiddlewareFunction(fn, context, args, next) {
   let maybePromise;
   try {

package/package.json

@@ -1,30 +1,21 @@
 {
   "name": "kareem",
-  "version": "2.3.0",
+  "version": "2.3.4",
   "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",
-    "standard-version": "4.4.0"
-  },
-  "jscsConfig": {
-    "preset": "airbnb",
-    "requireMultipleVarDecl": null,
-    "disallowMultipleVarDecl": true
+    "acquit": "1.x",
+    "acquit-ignore": "0.1.x",
+    "mocha": "9.2.0",
+    "nyc": "15.1.0"
   },
   "author": "Valeri Karpov <val@karpov.io>",
   "license": "Apache-2.0"

package/test/examples.test.js

@@ -17,7 +17,10 @@ describe('pre hooks', function() {
 
   it('runs without any hooks specified', function(done) {
     hooks.execPre('cook', null, function() {
+      // ...
+      // acquit:ignore:start
       done();
+      // acquit:ignore:end
     });
   });
 
@@ -34,7 +37,9 @@ describe('pre hooks', function() {
 
     hooks.execPre('cook', null, function() {
       assert.equal(1, count);
+      // acquit:ignore:start
       done();
+      // acquit:ignore:end
     });
   });
 
@@ -55,7 +60,9 @@ describe('pre hooks', function() {
     hooks.execPre('cook', null, function() {
       assert.equal(1, count1);
       assert.equal(1, count2);
+      // acquit:ignore:start
       done();
+      // acquit:ignore:end
     });
   });
 
@@ -78,7 +85,9 @@ describe('pre hooks', function() {
       assert.equal(null, error);
       assert.equal(1, count1);
       assert.equal(1, count2);
+      // acquit:ignore:start
       done();
+      // acquit:ignore:end
     });
   });
 
@@ -102,7 +111,9 @@ describe('pre hooks', function() {
       assert.equal(null, error);
       assert.equal(3, obj.bacon);
       assert.equal(4, obj.eggs);
+      // acquit:ignore:start
       done();
+      // acquit:ignore:end
     });
   });
 
@@ -140,7 +151,9 @@ describe('pre hooks', function() {
       assert.equal(3, obj.bacon);
       assert.equal(4, obj.eggs);
       assert.equal(false, obj.waffles);
+      // acquit:ignore:start
       done();
+      // acquit:ignore:end
     });
   });
 
@@ -162,7 +175,9 @@ describe('pre hooks', function() {
 
     hooks.execPre('cook', obj, function() {
       assert.equal(3, obj.bacon);
+      // acquit:ignore:start
       done();
+      // acquit:ignore:end
     });
   });
 });
@@ -193,7 +208,9 @@ describe('post hooks', function() {
       assert.ifError(error);
       assert.equal(1, eggs);
       assert.equal(2, bacon);
+      // acquit:ignore:start
       done();
+      // acquit:ignore:end
     });
   });
 
@@ -220,7 +237,33 @@ describe('post hooks', function() {
       assert.ok(execed.second);
       assert.equal(1, eggs);
       assert.equal(2, bacon);
+      // acquit:ignore:start
       done();
+      // acquit:ignore:end
+    });
+  });
+
+  /* You can also return a promise from your post hooks instead of calling
+   * `next()`. When the returned promise resolves, kareem will kick off the
+   * next middleware.
+   */
+  it('supports returning a promise', function(done) {
+    hooks.post('cook', function(bacon) {
+      return new Promise(resolve => {
+        setTimeout(() => {
+          this.bacon = 3;
+          resolve();
+        }, 100);
+      });
+    });
+
+    var obj = { bacon: 0 };
+
+    hooks.execPost('cook', obj, obj, function() {
+      assert.equal(obj.bacon, 3);
+      // acquit:ignore:start
+      done();
+      // acquit:ignore:end
     });
   });
 });
@@ -271,7 +314,9 @@ describe('wrap()', function() {
       assert.equal('no', obj.tofu);
 
       assert.equal(obj, result);
+      // acquit:ignore:start
       done();
+      // acquit:ignore:end
     });
 
     hooks.wrap(
@@ -343,7 +388,9 @@ describe('createWrapper()', function() {
       assert.equal('no', obj.tofu);
 
       assert.equal(obj, result);
+      // acquit:ignore:start
       done();
+      // acquit:ignore:end
     });
   });
 });

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() {};
@@ -283,6 +287,26 @@ describe('execPre', function() {
       done();
     });
   });
+
+  it('handles sync errors in pre if there are more hooks', function(done) {
+    var execed = {};
+
+    hooks.pre('cook', function() {
+      execed.first = true;
+      throw new Error('Oops!');
+    });
+
+    hooks.pre('cook', function() {
+      execed.second = true;
+    });
+
+    hooks.execPre('cook', null, function(err) {
+      assert.ok(err);
+      assert.ok(execed.first);
+      assert.equal(err.message, 'Oops!');
+      done();
+    });
+  });
 });
 
 describe('execPreSync', function() {

package/test/wrap.test.js

@@ -319,6 +319,30 @@ describe('wrap()', function() {
       25);
   });
 
+  it('catches sync errors', function(done) {
+    hooks.pre('cook', function(done) {
+      done();
+    });
+
+    hooks.post('cook', function(callback) {
+      callback();
+    });
+
+    var args = [];
+    args.push(function(error) {
+      assert.equal(error.message, 'oops!');
+      done();
+    });
+
+    hooks.wrap(
+      'cook',
+      function() {
+        throw new Error('oops!');
+      },
+      null,
+      args);
+  });
+
   it('sync wrappers', function() {
     var calledPre = 0;
     var calledFn = 0;