Merge pull request #99 from gnulnx/jf/delete_many

Jf/delete many

AUTHORS.rst

@@ -17,4 +17,5 @@ wtracyliu / wtracyliu <https://github.com/wtracyliu>
 Florent Messa / thoas <https://github.com/thoas>
 Markus Kaiserswerth / mkai <https://github.com/mkai>
 Michael Lindemuth / mlindemu <https://github.com/mlindemu>
+John Furr / gnulnx <https://github.com/gnulnx>
 Christian Pedersen / chripede <https://github.com/chripede>

redis_cache/backends/single.py

@@ -34,7 +34,8 @@ class RedisCache(BaseRedisCache):
     def delete_many(self, keys, version=None):
         """Remove multiple keys at once."""
         versioned_keys = self.make_keys(keys, version=version)
-        self._delete_many(self.master_client, versioned_keys)
+        if versioned_keys:
+            self._delete_many(self.master_client, versioned_keys)
 
     def clear(self, version=None):
         """Flush cache keys.

tests/testapp/tests/base_tests.py

@@ -358,6 +358,8 @@ class BaseRedisTestCase(SetupMixin):
         self.assertIsNone(self.cache.get("key1"))
         self.assertIsNone(self.cache.get("key2"))
         self.assertEqual(self.cache.get("key3"), "ham")
+        # Test that passing an empty list fails silently
+        self.cache.delete_many([])
 
     def test_clear(self):
         # The cache can be emptied using clear