diff --git a/packages/accounts-mongo/src/index.js b/packages/accounts-mongo/src/index.js index 4a72fa9eb..5fbb6a11a 100644 --- a/packages/accounts-mongo/src/index.js +++ b/packages/accounts-mongo/src/index.js @@ -150,15 +150,12 @@ class Mongo { } async setProfile(userId: string, profile: Object): Promise { - const ret = await this.collection.update({ _id: userId }, { + await this.collection.update({ _id: userId }, { $set: { profile, [this.options.timestamps.updatedAt]: Date.now(), }, }); - if (ret.result.nModified === 0) { - throw new Error('User not found'); - } const user = await this.findUserById(userId); return user && user.profile; } diff --git a/packages/accounts-mongo/src/index.spec.js b/packages/accounts-mongo/src/index.spec.js index d0cc89a7c..53a7d9133 100644 --- a/packages/accounts-mongo/src/index.spec.js +++ b/packages/accounts-mongo/src/index.spec.js @@ -15,19 +15,22 @@ const session = { userAgent: 'user agent', }; +function dropDatabase(cb) { + db.dropDatabase((err) => { + if (err) return cb(err); + return cb(); + }); +} + function createConnection(cb) { const url = 'mongodb://localhost:27017/accounts-mongo-tests'; mongodb.MongoClient.connect(url, (err, dbArg) => { db = dbArg; mongo = new Mongo(db); - cb(err); - }); -} - -function dropDatabase(cb) { - db.dropDatabase((err) => { if (err) return cb(err); - return cb(); + return dropDatabase((error) => { + cb(error); + }); }); } @@ -292,15 +295,6 @@ describe('Mongo', () => { }); describe('setProfile', () => { - it('should throw if user is not found', async () => { - try { - await mongo.setProfile('unknowuser', {}); - throw new Error(); - } catch (err) { - expect(err.message).toEqual('User not found'); - } - }); - it('should change profile', async () => { const userId = await mongo.createUser(user); await delay(10);