fix: make dogpile.cache.memory preserve its init parameters

Make the `MemoryBackend` behave like the others, and stop removing keys from the initialization dict.

Fixes: Issue #273

Closes: #274
Pull-request: #274
Pull-request-sha: 759dc77

Change-Id: Ifd3f0d88b18a73a0ce287e1a514a6268daf157c4

Author: 0xDEC0DE

dogpile/cache/backends/memory.py

@@ -50,7 +50,7 @@ class MemoryBackend(CacheBackend):
     """
 
     def __init__(self, arguments):
-        self._cache = arguments.pop("cache_dict", {})
+        self._cache = arguments.get("cache_dict", {})
 
     def get(self, key):
         return self._cache.get(key, NO_VALUE)

tests/cache/test_memory_backend.py

@@ -4,6 +4,10 @@
 
 class MemoryBackendTest(_GenericBackendTestSuite):
     backend = "dogpile.cache.memory"
+    config_args = {"arguments": {"cache_dict": {}}}
+
+    def test_config_args_does_not_mutate(self):
+        assert "cache_dict" in self.config_args["arguments"]
 
 
 class MemoryBackendSerializerTest(
@@ -12,5 +16,5 @@ class MemoryBackendSerializerTest(
     pass
 
 
-class MemoryPickleBackendTest(_GenericBackendTestSuite):
+class MemoryPickleBackendTest(MemoryBackendTest):
     backend = "dogpile.cache.memory_pickle"