fix(document): catch sync errors in document pre hooks and report as …

…error

Fix #5738

package.json

@@ -21,7 +21,7 @@
   "dependencies": {
     "async": "2.1.4",
     "bson": "~1.0.4",
-    "hooks-fixed": "2.0.0",
+    "hooks-fixed": "2.0.2",
     "kareem": "1.5.0",
     "mongodb": "2.2.33",
     "mpath": "0.3.0",

test/document.hooks.test.js

@@ -763,6 +763,26 @@ describe('document: hooks:', function() {
     done();
   });
 
+  it('sync exceptions get passed as errors (gh-5738)', function(done) {
+    var bookSchema = new Schema({ title: String });
+
+    /* eslint-disable no-unused-vars */
+    bookSchema.pre('save', function(next) {
+      throw new Error('woops!');
+    });
+
+    var Book = mongoose.model('gh5738', bookSchema);
+
+    var book = new Book({});
+
+    book.title = 'Professional AngularJS';
+    book.save(function(error) {
+      assert.ok(error);
+      assert.equal(error.message, 'woops!');
+      done();
+    });
+  });
+
   it('nested subdocs only fire once (gh-3281)', function(done) {
     var L3Schema = new Schema({
       title: String