123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433 |
- # -*- coding: utf-8 -*-
- import os
- import shutil
- import sys
- import tempfile
- import time
- import unittest
- from cStringIO import StringIO
- from datetime import datetime, timedelta
- from django.conf import settings
- from django.core.exceptions import SuspiciousOperation
- from django.core.files.base import ContentFile, File
- from django.core.files.images import get_image_dimensions
- from django.core.files.storage import FileSystemStorage, get_storage_class
- from django.core.files.uploadedfile import UploadedFile
- from django.core.exceptions import ImproperlyConfigured
- from unittest import TestCase
- try:
- import threading
- except ImportError:
- import dummy_threading as threading
- # Try to import PIL in either of the two ways it can end up installed.
- # Checking for the existence of Image is enough for CPython, but
- # for PyPy, you need to check for the underlying modules
- try:
- from PIL import Image, _imaging
- except ImportError:
- try:
- import Image, _imaging
- except ImportError:
- Image = None
- class GetStorageClassTests(unittest.TestCase):
- def assertRaisesErrorWithMessage(self, error, message, callable,
- *args, **kwargs):
- self.assertRaises(error, callable, *args, **kwargs)
- try:
- callable(*args, **kwargs)
- except error, e:
- self.assertEqual(message, str(e))
- def test_get_filesystem_storage(self):
- """
- get_storage_class returns the class for a storage backend name/path.
- """
- self.assertEqual(
- get_storage_class('django.core.files.storage.FileSystemStorage'),
- FileSystemStorage)
- def test_get_invalid_storage_module(self):
- """
- get_storage_class raises an error if the requested import don't exist.
- """
- self.assertRaisesErrorWithMessage(
- ImproperlyConfigured,
- "NonExistingStorage isn't a storage module.",
- get_storage_class,
- 'NonExistingStorage')
- def test_get_nonexisting_storage_class(self):
- """
- get_storage_class raises an error if the requested class don't exist.
- """
- self.assertRaisesErrorWithMessage(
- ImproperlyConfigured,
- 'Storage module "django.core.files.storage" does not define a '\
- '"NonExistingStorage" class.',
- get_storage_class,
- 'django.core.files.storage.NonExistingStorage')
- def test_get_nonexisting_storage_module(self):
- """
- get_storage_class raises an error if the requested module don't exist.
- """
- self.assertRaisesErrorWithMessage(
- ImproperlyConfigured,
- 'Error importing storage module django.core.files.non_existing_'\
- 'storage: "No module named non_existing_storage"',
- get_storage_class,
- 'django.core.files.non_existing_storage.NonExistingStorage')
- class FileStorageTests(unittest.TestCase):
- storage_class = FileSystemStorage
- def setUp(self):
- self.temp_dir = tempfile.mktemp()
- os.makedirs(self.temp_dir)
- self.storage = self.storage_class(location=self.temp_dir,
- base_url='/test_media_url/')
- def tearDown(self):
- shutil.rmtree(self.temp_dir)
- def test_file_access_options(self):
- """
- Standard file access options are available, and work as expected.
- """
- self.failIf(self.storage.exists('storage_test'))
- f = self.storage.open('storage_test', 'w')
- f.write('storage contents')
- f.close()
- self.assert_(self.storage.exists('storage_test'))
- f = self.storage.open('storage_test', 'r')
- self.assertEqual(f.read(), 'storage contents')
- f.close()
- self.storage.delete('storage_test')
- self.failIf(self.storage.exists('storage_test'))
- def test_file_accessed_time(self):
- """
- File storage returns a Datetime object for the last accessed time of
- a file.
- """
- self.failIf(self.storage.exists('test.file'))
- f = ContentFile('custom contents')
- f_name = self.storage.save('test.file', f)
- atime = self.storage.accessed_time(f_name)
- self.assertEqual(atime, datetime.fromtimestamp(
- os.path.getatime(self.storage.path(f_name))))
- self.assertTrue(datetime.now() - self.storage.accessed_time(f_name) < timedelta(seconds=2))
- self.storage.delete(f_name)
- def test_file_created_time(self):
- """
- File storage returns a Datetime object for the creation time of
- a file.
- """
- self.failIf(self.storage.exists('test.file'))
- f = ContentFile('custom contents')
- f_name = self.storage.save('test.file', f)
- ctime = self.storage.created_time(f_name)
- self.assertEqual(ctime, datetime.fromtimestamp(
- os.path.getctime(self.storage.path(f_name))))
- self.assertTrue(datetime.now() - self.storage.created_time(f_name) < timedelta(seconds=2))
- self.storage.delete(f_name)
- def test_file_modified_time(self):
- """
- File storage returns a Datetime object for the last modified time of
- a file.
- """
- self.failIf(self.storage.exists('test.file'))
- f = ContentFile('custom contents')
- f_name = self.storage.save('test.file', f)
- mtime = self.storage.modified_time(f_name)
- self.assertEqual(mtime, datetime.fromtimestamp(
- os.path.getmtime(self.storage.path(f_name))))
- self.assertTrue(datetime.now() - self.storage.modified_time(f_name) < timedelta(seconds=2))
- self.storage.delete(f_name)
- def test_file_save_without_name(self):
- """
- File storage extracts the filename from the content object if no
- name is given explicitly.
- """
- self.failIf(self.storage.exists('test.file'))
- f = ContentFile('custom contents')
- f.name = 'test.file'
- storage_f_name = self.storage.save(None, f)
- self.assertEqual(storage_f_name, f.name)
- self.assert_(os.path.exists(os.path.join(self.temp_dir, f.name)))
- self.storage.delete(storage_f_name)
- def test_file_path(self):
- """
- File storage returns the full path of a file
- """
- self.failIf(self.storage.exists('test.file'))
- f = ContentFile('custom contents')
- f_name = self.storage.save('test.file', f)
- self.assertEqual(self.storage.path(f_name),
- os.path.join(self.temp_dir, f_name))
- self.storage.delete(f_name)
- def test_file_url(self):
- """
- File storage returns a url to access a given file from the Web.
- """
- self.assertEqual(self.storage.url('test.file'),
- '%s%s' % (self.storage.base_url, 'test.file'))
- self.storage.base_url = None
- self.assertRaises(ValueError, self.storage.url, 'test.file')
- def test_file_with_mixin(self):
- """
- File storage can get a mixin to extend the functionality of the
- returned file.
- """
- self.failIf(self.storage.exists('test.file'))
- class TestFileMixin(object):
- mixed_in = True
- f = ContentFile('custom contents')
- f_name = self.storage.save('test.file', f)
- self.assert_(isinstance(
- self.storage.open('test.file', mixin=TestFileMixin),
- TestFileMixin
- ))
- self.storage.delete('test.file')
- def test_listdir(self):
- """
- File storage returns a tuple containing directories and files.
- """
- self.failIf(self.storage.exists('storage_test_1'))
- self.failIf(self.storage.exists('storage_test_2'))
- self.failIf(self.storage.exists('storage_dir_1'))
- f = self.storage.save('storage_test_1', ContentFile('custom content'))
- f = self.storage.save('storage_test_2', ContentFile('custom content'))
- os.mkdir(os.path.join(self.temp_dir, 'storage_dir_1'))
- dirs, files = self.storage.listdir('')
- self.assertEqual(set(dirs), set([u'storage_dir_1']))
- self.assertEqual(set(files),
- set([u'storage_test_1', u'storage_test_2']))
- self.storage.delete('storage_test_1')
- self.storage.delete('storage_test_2')
- os.rmdir(os.path.join(self.temp_dir, 'storage_dir_1'))
- def test_file_storage_prevents_directory_traversal(self):
- """
- File storage prevents directory traversal (files can only be accessed if
- they're below the storage location).
- """
- self.assertRaises(SuspiciousOperation, self.storage.exists, '..')
- self.assertRaises(SuspiciousOperation, self.storage.exists, '/etc/passwd')
- class CustomStorage(FileSystemStorage):
- def get_available_name(self, name):
- """
- Append numbers to duplicate files rather than underscores, like Trac.
- """
- parts = name.split('.')
- basename, ext = parts[0], parts[1:]
- number = 2
- while self.exists(name):
- name = '.'.join([basename, str(number)] + ext)
- number += 1
- return name
- class CustomStorageTests(FileStorageTests):
- storage_class = CustomStorage
- def test_custom_get_available_name(self):
- first = self.storage.save('custom_storage', ContentFile('custom contents'))
- self.assertEqual(first, 'custom_storage')
- second = self.storage.save('custom_storage', ContentFile('more contents'))
- self.assertEqual(second, 'custom_storage.2')
- self.storage.delete(first)
- self.storage.delete(second)
- class UnicodeFileNameTests(unittest.TestCase):
- def test_unicode_file_names(self):
- """
- Regression test for #8156: files with unicode names I can't quite figure
- out the encoding situation between doctest and this file, but the actual
- repr doesn't matter; it just shouldn't return a unicode object.
- """
- uf = UploadedFile(name=u'¿Cómo?',content_type='text')
- self.assertEqual(type(uf.__repr__()), str)
- # Tests for a race condition on file saving (#4948).
- # This is written in such a way that it'll always pass on platforms
- # without threading.
- class SlowFile(ContentFile):
- def chunks(self):
- time.sleep(1)
- return super(ContentFile, self).chunks()
- class FileSaveRaceConditionTest(TestCase):
- def setUp(self):
- self.storage_dir = tempfile.mkdtemp()
- self.storage = FileSystemStorage(self.storage_dir)
- self.thread = threading.Thread(target=self.save_file, args=['conflict'])
- def tearDown(self):
- shutil.rmtree(self.storage_dir)
- def save_file(self, name):
- name = self.storage.save(name, SlowFile("Data"))
- def test_race_condition(self):
- self.thread.start()
- name = self.save_file('conflict')
- self.thread.join()
- self.assert_(self.storage.exists('conflict'))
- self.assert_(self.storage.exists('conflict_1'))
- self.storage.delete('conflict')
- self.storage.delete('conflict_1')
- class FileStoragePermissions(TestCase):
- def setUp(self):
- self.old_perms = settings.FILE_UPLOAD_PERMISSIONS
- settings.FILE_UPLOAD_PERMISSIONS = 0666
- self.storage_dir = tempfile.mkdtemp()
- self.storage = FileSystemStorage(self.storage_dir)
- def tearDown(self):
- settings.FILE_UPLOAD_PERMISSIONS = self.old_perms
- shutil.rmtree(self.storage_dir)
- def test_file_upload_permissions(self):
- name = self.storage.save("the_file", ContentFile("data"))
- actual_mode = os.stat(self.storage.path(name))[0] & 0777
- self.assertEqual(actual_mode, 0666)
- class FileStoragePathParsing(TestCase):
- def setUp(self):
- self.storage_dir = tempfile.mkdtemp()
- self.storage = FileSystemStorage(self.storage_dir)
- def tearDown(self):
- shutil.rmtree(self.storage_dir)
- def test_directory_with_dot(self):
- """Regression test for #9610.
- If the directory name contains a dot and the file name doesn't, make
- sure we still mangle the file name instead of the directory name.
- """
- self.storage.save('dotted.path/test', ContentFile("1"))
- self.storage.save('dotted.path/test', ContentFile("2"))
- self.failIf(os.path.exists(os.path.join(self.storage_dir, 'dotted_.path')))
- self.assert_(os.path.exists(os.path.join(self.storage_dir, 'dotted.path/test')))
- self.assert_(os.path.exists(os.path.join(self.storage_dir, 'dotted.path/test_1')))
- def test_first_character_dot(self):
- """
- File names with a dot as their first character don't have an extension,
- and the underscore should get added to the end.
- """
- self.storage.save('dotted.path/.test', ContentFile("1"))
- self.storage.save('dotted.path/.test', ContentFile("2"))
- self.assert_(os.path.exists(os.path.join(self.storage_dir, 'dotted.path/.test')))
- # Before 2.6, a leading dot was treated as an extension, and so
- # underscore gets added to beginning instead of end.
- if sys.version_info < (2, 6):
- self.assert_(os.path.exists(os.path.join(self.storage_dir, 'dotted.path/_1.test')))
- else:
- self.assert_(os.path.exists(os.path.join(self.storage_dir, 'dotted.path/.test_1')))
- if Image is not None:
- class DimensionClosingBug(TestCase):
- """
- Test that get_image_dimensions() properly closes files (#8817)
- """
- def test_not_closing_of_files(self):
- """
- Open files passed into get_image_dimensions() should stay opened.
- """
- empty_io = StringIO()
- try:
- get_image_dimensions(empty_io)
- finally:
- self.assert_(not empty_io.closed)
- def test_closing_of_filenames(self):
- """
- get_image_dimensions() called with a filename should closed the file.
- """
- # We need to inject a modified open() builtin into the images module
- # that checks if the file was closed properly if the function is
- # called with a filename instead of an file object.
- # get_image_dimensions will call our catching_open instead of the
- # regular builtin one.
- class FileWrapper(object):
- _closed = []
- def __init__(self, f):
- self.f = f
- def __getattr__(self, name):
- return getattr(self.f, name)
- def close(self):
- self._closed.append(True)
- self.f.close()
- def catching_open(*args):
- return FileWrapper(open(*args))
- from django.core.files import images
- images.open = catching_open
- try:
- get_image_dimensions(os.path.join(os.path.dirname(__file__), "test1.png"))
- finally:
- del images.open
- self.assert_(FileWrapper._closed)
- class InconsistentGetImageDimensionsBug(TestCase):
- """
- Test that get_image_dimensions() works properly after various calls using a file handler (#11158)
- """
- def test_multiple_calls(self):
- """
- Multiple calls of get_image_dimensions() should return the same size.
- """
- from django.core.files.images import ImageFile
- img_path = os.path.join(os.path.dirname(__file__), "test.png")
- image = ImageFile(open(img_path, 'rb'))
- image_pil = Image.open(img_path)
- size_1, size_2 = get_image_dimensions(image), get_image_dimensions(image)
- self.assertEqual(image_pil.size, size_1)
- self.assertEqual(size_1, size_2)
|