Stop using err.lastErrorObject (it's not in the driver anymore).

mongo-js · Jul 25, 2014 · da252d9 · da252d9
1 parent f487584
commit da252d9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 12 deletions.
diff --git a/index.js b/index.js
@@ -214,7 +214,7 @@ Collection.prototype.findAndModify = function(options, callback) {
 		// property is added to the err argument instead.
 		// If the findAndModify command finds no matching document, when performing update or remove,
 		// no lastErrorObject is included (so we fake it).
-		(callback || noop)(err, doc, (err && err.lastErrorObject) || (obj && obj.lastErrorObject) || { n: 0 });
+		(callback || noop)(err, doc, (obj && obj.lastErrorObject) || { n: 0 });
 	}]);
 };
 

diff --git a/tests/test-find-and-modify.js b/tests/test-find-and-modify.js
@@ -63,17 +63,7 @@ insert([{
 						assert.ok(!err);
 						assert.equal(lastErrorObject.n, 0);
 
-						// Correct error handling
-						db.a.findAndModify({
-							update: { $illigal: 1 }
-						}, function(err, doc, lastErrorObject) {
-							assert(err instanceof Error);
-							assert.equal(doc, null);
-							assert.equal(lastErrorObject.n, 0);
-							assert.equal(typeof lastErrorObject.err, 'string');
-
-							done();
-						});
+						done();
 					});
 				});
 			});