directly return promise or…? (which is better?)

changePassword.js

userSchema.methods.changePassword = function (password) {
  // option 1
  return new Promise((resolve, reject) => {
    let evaluatePassword = passwordStrength(password)
    if (evaluatePassword.state) {
      return this.generateHash(password).then((hash) => {
        this.password = hash
        return this
      })
    } else {
      throw new Error(evaluatePassword.message)
    }
  })

  // option 2
  let evaluatePassword = passwordStrength(password)
  if (evaluatePassword.state) {
    return this.generateHash(password).then((hash) => {
      this.password = hash
      return this
    })
  } else {
    throw new Error(evaluatePassword.message)
  }
}