__getitem__ now raises a KeyError when a key doesn't exist

Author: andymccurdy

CHANGES

@@ -1,5 +1,7 @@
 * 2.2.5 (in development)
     * Support Jython, fixing #97. Thanks to Adam Vandenberg for the patch.
+    * Using __getitem__ now properly raises a KeyError when the key is not
+      found. Thanks Ionuț Arțăriși for the patch.
 * 2.2.4
     * WARNING: Potential backwards incompatible change - Changed order of
       parameters of ZREVRANGEBYSCORE to match those of the actual Redis command.

redis/client.py

@@ -452,9 +452,9 @@ def __getitem__(self, name):
         Return the value at key ``name``, raises a KeyError if the key
         doesn't exist.
         """
-        _name = self.get(name)
-        if _name:
-            return _name
+        value = self.get(name)
+        if value:
+            return value
         else:
             raise KeyError(name)
 

tests/lock.py

@@ -17,7 +17,7 @@ def test_lock(self):
         self.assert_(lock.acquire())
         self.assertEquals(self.client['foo'], str(Lock.LOCK_FOREVER))
         lock.release()
-        self.assertEquals(self.client['foo'], None)
+        self.assertEquals(self.client.get('foo'), None)
 
     def test_competing_locks(self):
         lock1 = self.client.lock('foo')
@@ -50,7 +50,7 @@ def test_non_blocking(self):
     def test_context_manager(self):
         with self.client.lock('foo'):
             self.assertEquals(self.client['foo'], str(Lock.LOCK_FOREVER))
-        self.assertEquals(self.client['foo'], None)
+        self.assertEquals(self.client.get('foo'), None)
 
     def test_float_timeout(self):
         lock1 = self.client.lock('foo', timeout=1.5)

tests/server_commands.py

@@ -41,6 +41,7 @@ def test_get_and_set(self):
     def test_getitem_and_setitem(self):
         self.client['a'] = 'bar'
         self.assertEquals(self.client['a'], 'bar')
+        self.assertRaises(KeyError, self.client.__getitem__, 'b')
 
     def test_delete(self):
         self.assertEquals(self.client.delete('a'), False)
@@ -50,7 +51,7 @@ def test_delete(self):
     def test_delitem(self):
         self.client['a'] = 'foo'
         del self.client['a']
-        self.assertEquals(self.client['a'], None)
+        self.assertEquals(self.client.get('a'), None)
 
     def test_config_get(self):
         data = self.client.config_get()
@@ -181,7 +182,7 @@ def test_msetnx(self):
         self.assert_(not self.client.msetnx(d2))
         for k,v in d.iteritems():
             self.assertEquals(self.client[k], v)
-        self.assertEquals(self.client['d'], None)
+        self.assertEquals(self.client.get('d'), None)
 
     def test_randomkey(self):
         self.assertEquals(self.client.randomkey(), None)
@@ -193,7 +194,7 @@ def test_randomkey(self):
     def test_rename(self):
         self.client['a'] = '1'
         self.assert_(self.client.rename('a', 'b'))
-        self.assertEquals(self.client['a'], None)
+        self.assertEquals(self.client.get('a'), None)
         self.assertEquals(self.client['b'], '1')
 
     def test_renamenx(self):