fix(middleware): ensure sync errors in pre hooks always bubble up to the calling code

Fix #9659

Author: vkarpov15

package.json

@@ -21,7 +21,7 @@
   "dependencies": {
     "@types/mongodb": "^3.5.27",
     "bson": "^1.1.4",
-    "kareem": "2.3.1",
+    "kareem": "2.3.2",
     "mongodb": "3.6.3",
     "mongoose-legacy-pluralize": "1.0.2",
     "mpath": "0.8.0",

test/document.test.js

@@ -9788,4 +9788,21 @@ describe('document', function() {
     assert.ok(documentFromDefault === user);
     assert.equal(documentFromDefault.name, 'Hafez');
   });
+
+  it('handles pre hook throwing a sync error (gh-9659)', function() {
+    const TestSchema = new Schema({ name: String });
+
+    TestSchema.pre('save', function() {
+      throw new Error('test err');
+    });
+    const TestModel = db.model('Test', TestSchema);
+
+    return co(function*() {
+      const testObject = new TestModel({ name: 't' });
+
+      const err = yield testObject.save().then(() => null, err => err);
+      assert.ok(err);
+      assert.equal(err.message, 'test err');
+    });
+  });
 });

test/model.test.js

@@ -1603,14 +1603,14 @@ describe('Model', function() {
       let docMiddleware = 0;
       let queryMiddleware = 0;
 
-      schema.pre('remove', { query: true }, function() {
-        assert.ok(this instanceof Model.Query);
+      schema.pre('remove', { query: true, document: false }, function() {
         ++queryMiddleware;
+        assert.ok(this instanceof Model.Query);
       });
 
-      schema.pre('remove', { document: true }, function() {
-        assert.ok(this instanceof Model);
+      schema.pre('remove', { query: false, document: true }, function() {
         ++docMiddleware;
+        assert.ok(this instanceof Model);
       });
 
       const Model = db.model('Test', schema);