Merge pull request #918 from tgxworld/fix_exists

Fix `Redis#exists?` to return boolean value for multiple key match.
redis · Jun 10, 2020 · 8982fa2 · 8982fa2
2 parents 850ab84 + 525a93e
commit 8982fa2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/lib/redis.rb b/lib/redis.rb
@@ -584,7 +584,9 @@ def _exists(*keys)
   # @return [Boolean]
   def exists?(*keys)
     synchronize do |client|
-      client.call([:exists, *keys], &Boolify)
+      client.call([:exists, *keys]) do |value|
+        value > 0
+      end
     end
   end
 

diff --git a/test/lint/value_types.rb b/test/lint/value_types.rb
@@ -40,6 +40,10 @@ def test_exists?
       r.set("{1}foo", "s1")
 
       assert_equal true, r.exists?("{1}foo")
+
+      r.set("{1}bar", "s1")
+
+      assert_equal true, r.exists?("{1}foo", "{1}bar")
     end
 
     def test_type