From d142544159dee7a93e31c886737e12c5bca8844a Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Wed, 10 May 2017 09:45:43 -0400 Subject: Spec coverage and refactor of digest mailer worker (#2961) --- app/workers/digest_mailer_worker.rb | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) (limited to 'app') 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 -- cgit