diff --git a/app/models/saved_search.rb b/app/models/saved_search.rb index 722f47eb1..d992bf11d 100644 --- a/app/models/saved_search.rb +++ b/app/models/saved_search.rb @@ -115,7 +115,7 @@ class SavedSearch < ApplicationRecord def populate(query, timeout: 10_000) redis_key = "search:#{query}" - return if redis.exists(redis_key) + return if redis.exists?(redis_key) post_ids = Post.with_timeout(timeout, [], query: query) do Post.system_tag_match(query).limit(QUERY_LIMIT).pluck(:id) diff --git a/test/unit/post_query_builder_test.rb b/test/unit/post_query_builder_test.rb index d74b08e8e..055e6e765 100644 --- a/test/unit/post_query_builder_test.rb +++ b/test/unit/post_query_builder_test.rb @@ -757,8 +757,8 @@ class PostQueryBuilderTest < ActiveSupport::TestCase create(:saved_search, query: "aaa", labels: ["zzz"], user: CurrentUser.user) create(:saved_search, query: "bbb", user: CurrentUser.user) - Redis.any_instance.stubs(:exists).with("search:aaa").returns(true) - Redis.any_instance.stubs(:exists).with("search:bbb").returns(true) + Redis.any_instance.stubs(:exists?).with("search:aaa").returns(true) + Redis.any_instance.stubs(:exists?).with("search:bbb").returns(true) Redis.any_instance.stubs(:smembers).with("search:aaa").returns([@post1.id]) Redis.any_instance.stubs(:smembers).with("search:bbb").returns([@post2.id])