Merge pull request #679 from candeira/fix-getitem

fix getitem syntax for redis get when value is the empty string

Author: andymccurdy

redis/client.py

@@ -885,7 +885,7 @@ def __getitem__(self, name):
         doesn't exist.
         """
         value = self.get(name)
-        if value:
+        if value is not None:
             return value
         raise KeyError(name)
 

tests/test_commands.py

@@ -341,6 +341,10 @@ def test_getitem_raises_keyerror_for_missing_key(self, r):
         with pytest.raises(KeyError):
             r['a']
 
+    def test_getitem_does_not_raise_keyerror_for_empty_string(self, r):
+        r['a'] = b("")
+        assert r['a'] == b("")
+
     def test_get_set_bit(self, r):
         # no value
         assert not r.getbit('a', 5)