instance.reload() ignores 'include' option

From the docs: “Options that are passed on to Model.find”

But the following does not work (see gist):

https://gist.github.com/matmar10/d460f0fdfa6071dbb2c1

// this works
User.findOne({
  include: [{ as: 'roles', model: UserRole }],
  where: { id: 1 }
}).then(function (user) {
  // has `roles` property OK
  console.log(user);
});

// this does not work
User.findOne({
  where: { id: 1 }
}).then(function (user) {
  user.reload({
    include: [{ as: 'roles', model: UserRole }]
  }).then(function (user) {
    // does NOT have `roles` property
    console.log(user);
  });
});

[davesag]

I just tested this and reload with include works fine. I believe this ticket can be closed.

[matmar10]

Shouldn’t it try to honor the include option passed to reload, if it exists? Something like:

Instance.prototype.reload = function(options) {
     return this.Model.find({
           where: where,
           limit: 1,
           include: options.include || this.options.include || null
     }, options).then(function(reload) {
           self.set(reload.dataValues, {raw: true, reset: true});
     }).return(self);
}