diff options
author | Matt Jankowski <mjankowski@thoughtbot.com> | 2017-05-10 09:45:43 -0400 |
---|---|---|
committer | Eugen Rochko <eugen@zeonfederated.com> | 2017-05-10 15:45:43 +0200 |
commit | d142544159dee7a93e31c886737e12c5bca8844a (patch) | |
tree | 6de1b55769c8def200d34b2a10de54966ea89619 /app/workers/digest_mailer_worker.rb | |
parent | 7ac092513c3c298ad80cb80f7bee76ca8ed56235 (diff) |
Spec coverage and refactor of digest mailer worker (#2961)
Diffstat (limited to 'app/workers/digest_mailer_worker.rb')
-rw-r--r-- | app/workers/digest_mailer_worker.rb | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/app/workers/digest_mailer_worker.rb b/app/workers/digest_mailer_worker.rb index dedb21e4e..028db89a9 100644 --- a/app/workers/digest_mailer_worker.rb +++ b/app/workers/digest_mailer_worker.rb @@ -5,10 +5,21 @@ class DigestMailerWorker sidekiq_options queue: 'mailers' + attr_reader :user + def perform(user_id) - user = User.find(user_id) - return unless user.settings.notification_emails['digest'] + @user = User.find(user_id) + deliver_digest if user_receives_digest? + end + + private + + def deliver_digest NotificationMailer.digest(user.account).deliver_now! user.touch(:last_emailed_at) end + + def user_receives_digest? + user.settings.notification_emails['digest'] + end end |