Fixed hset method for key=0

redis/client.py

@@ -3034,14 +3034,14 @@ def hlen(self, name):
     def hset(self, name, key=None, value=None, mapping=None):
         """
         Set ``key`` to ``value`` within hash ``name``,
-        Use ``mappings`` keyword args to set multiple key/value pairs
+        Use ``mapping`` keyword args to set multiple key/value pairs
         for a hash ``name``.
         Returns the number of fields that were added.
         """
-        if not key and not mapping:
+        if key is None and not mapping:
             raise DataError("'hset' with no key value pairs")
         items = []
-        if key:
+        if key is not None:
             items.extend((key, value))
         if mapping:
             for pair in mapping.items():

tests/test_commands.py

@@ -1612,6 +1612,10 @@ def test_hget_and_hset(self, r):
         # key inside of hash that doesn't exist returns null value
         assert r.hget('a', 'b') is None
 
+        # keys with bool(key) == False
+        assert r.hset('a', 0, 10) == 1
+        assert r.hset('a', '', 10) == 1
+
     def test_hset_with_multi_key_values(self, r):
         r.hset('a', mapping={'1': 1, '2': 2, '3': 3})
         assert r.hget('a', '1') == b'1'