Ticket #15806: patch_with_tests_for_15806.diff

django/core/cache/backends/filebased.py

@@ -118 +118 @@
             return
 
         if self._cull_frequency == 0:
-            doomed = filelist
+            doomed = (os.path.join(self._dir, k) for k in filelist)
         else:
             doomed = [os.path.join(self._dir, k) for (i, k) in enumerate(filelist) if i % self._cull_frequency == 0]
 

tests/regressiontests/cache/tests.py

@@ -408 +408 @@
         self.assertEqual(self.cache.get('key3'), 'sausage')
         self.assertEqual(self.cache.get('key4'), 'lobster bisque')
 
-    def perform_cull_test(self, initial_count, final_count):
+    def perform_cull_test(self, initial_count, final_count, cache=None):
         """This is implemented as a utility method, because only some of the backends
         implement culling. The culling algorithm also varies slightly, so the final
         number of entries will vary between backends"""
+        if cache is None:
+            cache = self.cache
+
         # Create initial cache key entries. This will overflow the cache, causing a cull
         for i in range(1, initial_count):
-            self.cache.set('cull%d' % i, 'value', 1000)
+            cache.set('cull%d' % i, 'value', 1000)
         count = 0
         # Count how many keys are left in the cache.
         for i in range(1, initial_count):
-            if self.cache.has_key('cull%d' % i):
+            if cache.has_key('cull%d' % i):
                 count = count + 1
         self.assertEqual(count, final_count)
 
@@ -848 +851 @@
     def setUp(self):
         self.dirname = tempfile.mkdtemp()
         self.cache = get_cache(self.backend_name, LOCATION=self.dirname, OPTIONS={'MAX_ENTRIES': 30})
+        self.full_cull_cache = get_cache(self.backend_name,
+                LOCATION=self.dirname,
+                OPTIONS={'MAX_ENTRIES': 30, 'CULL_FREQUENCY': 0})
         self.prefix_cache = get_cache(self.backend_name, LOCATION=self.dirname, KEY_PREFIX='cacheprefix')
         self.v2_cache = get_cache(self.backend_name, LOCATION=self.dirname, VERSION=2)
         self.custom_key_cache = get_cache(self.backend_name, LOCATION=self.dirname, KEY_FUNCTION=custom_key_func)
@@ -879 +885 @@
         self.assertTrue(not os.path.exists(os.path.dirname(keypath)))
         self.assertTrue(not os.path.exists(os.path.dirname(os.path.dirname(keypath))))
 
+    def test_full_cull(self):
+        self.perform_cull_test(32, 1, self.full_cull_cache)
+
     def test_cull(self):
         self.perform_cull_test(50, 29)