openstack/deb-python-django-compressor
Code Issues Proposed changes

CachedCompilerFilter: also cache empty files

Browse Source
This commit is contained in:
Johannes Linke
2015-09-10 20:37:23 +02:00
parent 7b1e442832
commit 0c59d1be25
2 changed files with 11 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
compressor/filters/base.py
View File

@@ -224,7 +224,7 @@ class CachedCompilerFilter(CompilerFilter):
if self.mimetype in settings.COMPRESS_CACHEABLE_PRECOMPILERS:
key = self.get_cache_key()
data = cache.get(key)
if data:
if data is not None:
return data
filtered = super(CachedCompilerFilter, self).input(**kwargs)
cache.set(key, filtered, settings.COMPRESS_REBUILD_TIMEOUT)

11
compressor/tests/test_filters.py
View File

@@ -10,7 +10,7 @@ from django.test import TestCase
from django.utils import unittest
from django.test.utils import override_settings
from compressor.cache import get_hashed_mtime, get_hashed_content
from compressor.cache import cache, get_hashed_mtime, get_hashed_content
from compressor.conf import settings
from compressor.css import CssCompressor
from compressor.utils import find_command
@@ -134,6 +134,15 @@ class PrecompilerTestCase(TestCase):
self.assertEqual("body { color:#990; }", compiler.input())
self.assertIsNotNone(compiler.infile) # Not cached
def test_precompiler_caches_empty_files(self):
command = '%s %s -f {infile} -o {outfile}' % (sys.executable, self.test_precompiler)
compiler = CachedCompilerFilter(command=command, **self.cached_precompiler_args)
self.assertEqual("body { color:#990; }", compiler.input())
cache.set(compiler.get_cache_key(), "")
compiler = CachedCompilerFilter(command=command, **self.cached_precompiler_args)
self.assertEqual("", compiler.input())
class CssMinTestCase(TestCase):
def test_cssmin_filter(self):