jaculabilis/danbooru
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

saved searches: fix call to redis.exists?

Browse Source 
https://github.com/redis/redis-rb/blob/master/CHANGELOG.md#421
This commit is contained in:
evazion
2020-06-19 12:13:56 -05:00
parent cd9e3e7f3d
commit 05f78f9bef
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/models/saved_search.rb
View File

@@ -115,7 +115,7 @@ class SavedSearch < ApplicationRecord
def populate(query, timeout: 10_000) def populate(query, timeout: 10_000)
redis_key = "search:#{query}" 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_ids = Post.with_timeout(timeout, [], query: query) do
Post.system_tag_match(query).limit(QUERY_LIMIT).pluck(:id) Post.system_tag_match(query).limit(QUERY_LIMIT).pluck(:id)

4
test/unit/post_query_builder_test.rb
View File

@@ -757,8 +757,8 @@ class PostQueryBuilderTest < ActiveSupport::TestCase
create(:saved_search, query: "aaa", labels: ["zzz"], user: CurrentUser.user) create(:saved_search, query: "aaa", labels: ["zzz"], user: CurrentUser.user)
create(:saved_search, query: "bbb", 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:aaa").returns(true)
Redis.any_instance.stubs(:exists).with("search:bbb").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:aaa").returns([@post1.id])
Redis.any_instance.stubs(:smembers).with("search:bbb").returns([@post2.id]) Redis.any_instance.stubs(:smembers).with("search:bbb").returns([@post2.id])