fix(User): ensure _mergeMagicFields returns data (#493)

Author: leeyeh

src/object.js

@@ -1033,7 +1033,7 @@ module.exports = function(AV) {
           output[key] = AV._parseDate(output[key]);
         }
       });
-      if (!output.updatedAt) {
+      if (output.createdAt && !output.updatedAt) {
         output.updatedAt = output.createdAt;
       }
       return output;

src/user.js

@@ -47,7 +47,7 @@ module.exports = function(AV) {
         this._sessionToken = attrs.sessionToken;
         delete attrs.sessionToken;
       }
-      AV.User.__super__._mergeMagicFields.call(this, attrs);
+      return AV.User.__super__._mergeMagicFields.call(this, attrs);
     },
 
     /**

test/object.js

@@ -107,6 +107,19 @@ describe('Objects', function(){
       parsedGameScore.get('score').should.eql(gameScore.get('score'));
     });
 
+    it('toJSON and parse (User)', () => {
+      const user = new AV.Object.createWithoutData('_User', 'objectId');
+      user.set('id', 'id');
+      user.set('score', 20);
+      const json = user.toJSON();
+      json.objectId.should.eql(user.id);
+      json.score.should.eql(user.get('score'));
+      const parsedUser = new AV.User(json, { parse: true });
+      parsedUser.id.should.eql(user.id);
+      parsedUser.get('id').should.eql(user.get('id'));
+      parsedUser.get('score').should.eql(user.get('score'));
+    });
+
     it('toJSON for nested objects', () => {
       const id = 'fakeObjectId';
       const a = AV.Object.createWithoutData('A', id, true);