Skip to content

Commit 1bc5874

Browse files
committed
changed findAndUpdate to updateOne.
1 parent cf3158c commit 1bc5874

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

src/managers/dao.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -213,11 +213,11 @@ function addToArrayFieldById(userId, fieldName, fieldValue, cbk) {
213213
}
214214
}
215215
};
216-
usersCollection.findOneAndUpdate({ _id: userId }, data, { returnOriginal: false, projection: { password: 0 }}, function (err) {
216+
usersCollection.updateOne({ _id: userId }, data, function (err, res) {
217217
if (err) {
218218
return cbk(err, null);
219219
}
220-
return cbk(null, 1);
220+
return cbk(null, res.modifiedCount);
221221
});
222222
}
223223

tests-unit/dao.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -285,10 +285,10 @@ describe('user dao', function () {
285285
const expectedField = 'field1';
286286
const expectedValue = ['value1', 'value2'];
287287

288-
fakeCollection.findOneAndUpdate = function (query, update, options, cbk) {
288+
fakeCollection.updateOne = function (query, update, cbk) {
289289
assert.equal(query._id, expectedUser.id);
290290
assert.deepEqual(update.$push[expectedField], {$each: [expectedValue]});
291-
cbk(null, expectedUser);
291+
cbk(null, {modifiedCount: 1});
292292
};
293293

294294
dao.addToArrayFieldById(expectedUser.id, expectedField, expectedValue, function (err, added) {

0 commit comments

Comments
 (0)