Merge branch 'develop'

app/workers/reviewer_reminder.rb

@@ -26,7 +26,7 @@ class ReviewerReminder
              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
+             where rev.vote = 0 and c.review_status = 0
                and u.role != 'manager' and u.active is not false;"
       ActiveRecord::Base.connection.exec_query(sql)
     end

test/fixtures/candidates.yml

@@ -137,6 +137,7 @@ wade: # Wade has completed AND submitted the test
   completed_at: <%= DateTime.current - 8.days %>
   reminded: false
   test_hash: BkSkpapJnkz2N
+  review_status: 2
 
 jorge: # Jorge has completed AND submitted the test
   name: Jorge Holmes

test/workers/reviewer_reminder_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 class ReviewerReminderTest < ActiveSupport::TestCase
   test "collection is created with results" do
     reminders = ReviewerReminder.new
-    assert_equal 8, reminders.size
+    assert_equal 7, reminders.size
   end
 
   test "each reminder has needed attributes" do