Merge pull request #755 from byroot/hdel-splat

Super splat args for HDEL

Author: byroot

lib/redis.rb

@@ -2102,9 +2102,9 @@ def mapped_hmget(key, *fields)
   # @param [String] key
   # @param [String, Array<String>] field
   # @return [Fixnum] the number of fields that were removed from the hash
-  def hdel(key, field)
+  def hdel(key, *fields)
     synchronize do |client|
-      client.call([:hdel, key, field])
+      client.call([:hdel, key, *fields])
     end
   end
 

lib/redis/distributed.rb

@@ -692,8 +692,8 @@ def mapped_hmget(key, *fields)
     end
 
     # Delete one or more hash fields.
-    def hdel(key, field)
-      node_for(key).hdel(key, field)
+    def hdel(key, *fields)
+      node_for(key).hdel(key, *fields)
     end
 
     # Determine if a hash field exists.

test/lint/hashes.rb

@@ -30,6 +30,21 @@ def test_hdel
       assert_equal nil, r.hget("foo", "f1")
     end
 
+    def test_splat_hdel
+      target_version "2.3.9" do
+        r.hset("foo", "f1", "s1")
+        r.hset("foo", "f2", "s2")
+
+        assert_equal "s1", r.hget("foo", "f1")
+        assert_equal "s2", r.hget("foo", "f2")
+
+        assert_equal 2, r.hdel("foo", "f1", "f2")
+
+        assert_equal nil, r.hget("foo", "f1")
+        assert_equal nil, r.hget("foo", "f2")
+      end
+    end
+
     def test_variadic_hdel
       target_version "2.3.9" do
         r.hset("foo", "f1", "s1")