diff --git a/app/workers/reviewer_reminder.rb b/app/workers/reviewer_reminder.rb new file mode 100644 index 0000000..58fbdb6 --- /dev/null +++ b/app/workers/reviewer_reminder.rb @@ -0,0 +1,32 @@ +# frozen_string_literal: true +class ReviewerReminder + def initialize + @collection = reminder_collection + end + + def count + @collection.count + end + alias size count + + def reminders + @reminders ||= @collection.to_hash.map { |r| OpenStruct.new(r) } + end + + def send_all + reminders.each do |reminder| + ReviewerMailer.reminder(reminder).deliver_now + end + end + + private + + def reminder_collection + sql = "select u.name, u.email, c.test_hash, c.project + from reviewer_votes rev + inner join users u on u.id = rev.user_id + inner join candidates c on c.id = rev.candidate_id + where rev.vote = 0 and rev.veto = 0 and u.active is not false;" + ActiveRecord::Base.connection.exec_query(sql) + end +end diff --git a/lib/tasks/reminders.rake b/lib/tasks/reminders.rake index 086a570..680dfde 100644 --- a/lib/tasks/reminders.rake +++ b/lib/tasks/reminders.rake @@ -5,4 +5,10 @@ namespace :reminders do reminders = CandidateReminder.new reminders.send_all end + + desc "send reminders to reviewers" + task reviewers: :environment do + reminders = ReviewerReminder.new + reminders.send_all + end end diff --git a/test/workers/reviewer_reminder_test.rb b/test/workers/reviewer_reminder_test.rb new file mode 100644 index 0000000..08101d3 --- /dev/null +++ b/test/workers/reviewer_reminder_test.rb @@ -0,0 +1,25 @@ +# frozen_string_literal: true +require 'test_helper' + +class ReviewerReminderTest < ActiveSupport::TestCase + test "collection is created with four results" do + reminders = ReviewerReminder.new + assert_equal 3, reminders.size + end + + test "each reminder has needed attributes" do + collection = ReviewerReminder.new + + assert_instance_of String, collection.reminders.first.name + assert_instance_of String, collection.reminders.first.email + assert_instance_of String, collection.reminders.first.test_hash + end + + test "send_all sends emails for each reviewer and test" do + collection = ReviewerReminder.new + + assert_difference("ActionMailer::Base.deliveries.size", collection.count) do + collection.send_all + end + end +end