FIX: Include pending queued users regardless of their score (#9372)

FIX: Include pending queued users regardless of their score (#9372)

diff --git a/app/models/reviewable.rb b/app/models/reviewable.rb
index 78d4b98..771b37f 100644
--- a/app/models/reviewable.rb
+++ b/app/models/reviewable.rb
@@ -476,7 +476,10 @@ class Reviewable < ActiveRecord::Base
     result = result.where("created_at <= ?", to_date) if to_date
 
     if min_score > 0 && status == :pending && type.nil?
-      result = result.where("score >= ? OR type = ?", min_score, ReviewableQueuedPost.name)
+      result = result.where(
+        "score >= ? OR type IN (?)",
+        min_score, [ReviewableQueuedPost.name, ReviewableUser.name]
+      )
     elsif min_score > 0
       result = result.where("score >= ?", min_score)
     end
diff --git a/spec/models/reviewable_spec.rb b/spec/models/reviewable_spec.rb
index 9b46eb7..5fd051f 100644
--- a/spec/models/reviewable_spec.rb
+++ b/spec/models/reviewable_spec.rb
@@ -181,10 +181,11 @@ RSpec.describe Reviewable, type: :model do
             SiteSetting.reviewable_default_visibility = :high
             Reviewable.set_priorities(high: 10)
             @queued_post = Fabricate(:reviewable_queued_post, score: 0, target: post)
+            @queued_user = Fabricate(:reviewable_user, score: 0)
           end
 
           it 'includes queued posts when searching for pending reviewables' do
-            expect(Reviewable.list_for(user)).to contain_exactly(@queued_post)
+            expect(Reviewable.list_for(user)).to contain_exactly(@queued_post, @queued_user)
           end
 
           it 'excludes pending queued posts when applying a different status filter' do

GitHub sha: e52e7f6e

This commit appears in #9372 which was approved by eviltrout. It was merged by romanrizzi.