diff options
| author | Tom Christie | 2014-12-12 15:37:43 +0000 | 
|---|---|---|
| committer | Tom Christie | 2014-12-12 15:37:43 +0000 | 
| commit | baaa356489dd51d7c68161db40e99cd59b1124c3 (patch) | |
| tree | 23dc5c4cbe1065580ff88ddd1bfa6dcda956ac68 /tests/test_parsers.py | |
| parent | 5e6052811716a494e995a84c497579867ee6acaa (diff) | |
| parent | fd473aa905337908b41c9a1087967a19f0558f89 (diff) | |
| download | django-rest-framework-baaa356489dd51d7c68161db40e99cd59b1124c3.tar.bz2 | |
Merge master
Diffstat (limited to 'tests/test_parsers.py')
| -rw-r--r-- | tests/test_parsers.py | 32 | 
1 files changed, 30 insertions, 2 deletions
| diff --git a/tests/test_parsers.py b/tests/test_parsers.py index 32fb0595..54455cf6 100644 --- a/tests/test_parsers.py +++ b/tests/test_parsers.py @@ -4,7 +4,8 @@ from __future__ import unicode_literals  from django import forms  from django.core.files.uploadhandler import MemoryFileUploadHandler  from django.test import TestCase -from rest_framework.compat import StringIO +from django.utils.six.moves import StringIO +from rest_framework.exceptions import ParseError  from rest_framework.parsers import FormParser, FileUploadParser @@ -44,13 +45,40 @@ class TestFileUploadParser(TestCase):          self.parser_context = {'request': request, 'kwargs': {}}      def test_parse(self): -        """ Make sure the `QueryDict` works OK """ +        """ +        Parse raw file upload. +        """          parser = FileUploadParser()          self.stream.seek(0)          data_and_files = parser.parse(self.stream, None, self.parser_context)          file_obj = data_and_files.files['file']          self.assertEqual(file_obj._size, 14) +    def test_parse_missing_filename(self): +        """ +        Parse raw file upload when filename is missing. +        """ +        parser = FileUploadParser() +        self.stream.seek(0) +        self.parser_context['request'].META['HTTP_CONTENT_DISPOSITION'] = '' +        with self.assertRaises(ParseError): +            parser.parse(self.stream, None, self.parser_context) + +    def test_parse_missing_filename_multiple_upload_handlers(self): +        """ +        Parse raw file upload with multiple handlers when filename is missing. +        Regression test for #2109. +        """ +        parser = FileUploadParser() +        self.stream.seek(0) +        self.parser_context['request'].upload_handlers = ( +            MemoryFileUploadHandler(), +            MemoryFileUploadHandler() +        ) +        self.parser_context['request'].META['HTTP_CONTENT_DISPOSITION'] = '' +        with self.assertRaises(ParseError): +            parser.parse(self.stream, None, self.parser_context) +      def test_get_filename(self):          parser = FileUploadParser()          filename = parser.get_filename(self.stream, None, self.parser_context) | 
