diff --git a/app/services/users/suggest_for_sidebar.rb b/app/services/users/suggest_for_sidebar.rb index 278c4449f..edfbe03e7 100644 --- a/app/services/users/suggest_for_sidebar.rb +++ b/app/services/users/suggest_for_sidebar.rb @@ -13,6 +13,8 @@ module Users end def suggest + return User.none unless user + suggested_user_ids = Rails.cache.fetch(generate_cache_name, expires_in: 120.hours) do (reputable_user_ids + random_user_ids).uniq end diff --git a/spec/services/users/suggest_for_sidebar_spec.rb b/spec/services/users/suggest_for_sidebar_spec.rb index a674d7870..5199baa1c 100644 --- a/spec/services/users/suggest_for_sidebar_spec.rb +++ b/spec/services/users/suggest_for_sidebar_spec.rb @@ -15,4 +15,9 @@ RSpec.describe Users::SuggestForSidebar, type: :service do tags = create_list(:tag, 3) expect(described_class.new(user, tags).suggest).to be_empty end + + it "returns no user if not signed in" do + tags = "html" + expect(described_class.new(nil, tags).suggest).to be_empty + end end