test(query): repro #4836

Automattic · Jan 2, 2017 · d1492ce · d1492ce
1 parent 22552c5
commit d1492ce
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 1 deletion.
diff --git a/test/model.populate.test.js b/test/model.populate.test.js
@@ -4640,7 +4640,6 @@ describe('model: populate:', function() {
             return Person.populate(luke, { path: 'parent', model: 'gh4843' });
           }).
           then(function(luke) {
-            console.log('test', luke);
             assert.equal(luke.parent.name, 'Anakin');
             done();
           }).

diff --git a/test/query.middleware.test.js b/test/query.middleware.test.js
@@ -240,4 +240,26 @@ describe('query middleware', function() {
       });
     });
   });
+
+  it('error handlers with findOneAndUpdate and passRawResult (gh-4836)', function(done) {
+    var schema = new Schema({name: {type: String}});
+
+    var called = false;
+    var errorHandler = function(err, res, next) {
+      called = true;
+      next();
+    };
+
+    schema.post('findOneAndUpdate', errorHandler);
+
+    var Person = db.model('Person', schema);
+
+    Person.
+      findOneAndUpdate({name: 'name'}, {}, {upsert: true, passRawResult: true}).
+      exec(function(error) {
+        assert.ifError(error);
+        assert.ok(!called);
+        done();
+      });
+  });
 });