diff options
| author | Santiago Basulto <santiago.basulto@gmail.com> | 2018-05-04 20:37:01 -0300 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2019-01-16 13:38:47 -0500 |
| commit | 4fc35a9c3efdc9154efce28cb23cb84f8834517e (patch) | |
| tree | a4f7b10244cb933d827cf72ef57dc11e68c1a6ca /tests | |
| parent | aa5d0a5a90a690dc6f8fdbbffba143e32c86e40a (diff) | |
Fixed #20147 -- Added HttpRequest.headers.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/requests/tests.py | 84 | ||||
| -rw-r--r-- | tests/utils_tests/test_datastructures.py | 83 |
2 files changed, 164 insertions, 3 deletions
diff --git a/tests/requests/tests.py b/tests/requests/tests.py index c5efdad3c7..720178a8b4 100644 --- a/tests/requests/tests.py +++ b/tests/requests/tests.py @@ -5,7 +5,7 @@ from urllib.parse import urlencode from django.core.exceptions import DisallowedHost from django.core.handlers.wsgi import LimitedStream, WSGIRequest from django.http import HttpRequest, RawPostDataException, UnreadablePostError -from django.http.request import split_domain_port +from django.http.request import HttpHeaders, split_domain_port from django.test import RequestFactory, SimpleTestCase, override_settings from django.test.client import FakePayload @@ -830,3 +830,85 @@ class BuildAbsoluteURITests(SimpleTestCase): for location, expected_url in tests: with self.subTest(location=location): self.assertEqual(request.build_absolute_uri(location=location), expected_url) + + +class RequestHeadersTests(SimpleTestCase): + ENVIRON = { + # Non-headers are ignored. + 'PATH_INFO': '/somepath/', + 'REQUEST_METHOD': 'get', + 'wsgi.input': BytesIO(b''), + 'SERVER_NAME': 'internal.com', + 'SERVER_PORT': 80, + # These non-HTTP prefixed headers are included. + 'CONTENT_TYPE': 'text/html', + 'CONTENT_LENGTH': '100', + # All HTTP-prefixed headers are included. + 'HTTP_ACCEPT': '*', + 'HTTP_HOST': 'example.com', + 'HTTP_USER_AGENT': 'python-requests/1.2.0', + } + + def test_base_request_headers(self): + request = HttpRequest() + request.META = self.ENVIRON + self.assertEqual(dict(request.headers), { + 'Content-Type': 'text/html', + 'Content-Length': '100', + 'Accept': '*', + 'Host': 'example.com', + 'User-Agent': 'python-requests/1.2.0', + }) + + def test_wsgi_request_headers(self): + request = WSGIRequest(self.ENVIRON) + self.assertEqual(dict(request.headers), { + 'Content-Type': 'text/html', + 'Content-Length': '100', + 'Accept': '*', + 'Host': 'example.com', + 'User-Agent': 'python-requests/1.2.0', + }) + + def test_wsgi_request_headers_getitem(self): + request = WSGIRequest(self.ENVIRON) + self.assertEqual(request.headers['User-Agent'], 'python-requests/1.2.0') + self.assertEqual(request.headers['user-agent'], 'python-requests/1.2.0') + self.assertEqual(request.headers['Content-Type'], 'text/html') + self.assertEqual(request.headers['Content-Length'], '100') + + def test_wsgi_request_headers_get(self): + request = WSGIRequest(self.ENVIRON) + self.assertEqual(request.headers.get('User-Agent'), 'python-requests/1.2.0') + self.assertEqual(request.headers.get('user-agent'), 'python-requests/1.2.0') + self.assertEqual(request.headers.get('Content-Type'), 'text/html') + self.assertEqual(request.headers.get('Content-Length'), '100') + + +class HttpHeadersTests(SimpleTestCase): + def test_basic(self): + environ = { + 'CONTENT_TYPE': 'text/html', + 'CONTENT_LENGTH': '100', + 'HTTP_HOST': 'example.com', + } + headers = HttpHeaders(environ) + self.assertEqual(sorted(headers), ['Content-Length', 'Content-Type', 'Host']) + self.assertEqual(headers, { + 'Content-Type': 'text/html', + 'Content-Length': '100', + 'Host': 'example.com', + }) + + def test_parse_header_name(self): + tests = ( + ('PATH_INFO', None), + ('HTTP_ACCEPT', 'Accept'), + ('HTTP_USER_AGENT', 'User-Agent'), + ('HTTP_X_FORWARDED_PROTO', 'X-Forwarded-Proto'), + ('CONTENT_TYPE', 'Content-Type'), + ('CONTENT_LENGTH', 'Content-Length'), + ) + for header, expected in tests: + with self.subTest(header=header): + self.assertEqual(HttpHeaders.parse_header_name(header), expected) diff --git a/tests/utils_tests/test_datastructures.py b/tests/utils_tests/test_datastructures.py index 806ad23721..0513df69b2 100644 --- a/tests/utils_tests/test_datastructures.py +++ b/tests/utils_tests/test_datastructures.py @@ -6,8 +6,8 @@ import copy from django.test import SimpleTestCase from django.utils.datastructures import ( - DictWrapper, ImmutableList, MultiValueDict, MultiValueDictKeyError, - OrderedSet, + CaseInsensitiveMapping, DictWrapper, ImmutableList, MultiValueDict, + MultiValueDictKeyError, OrderedSet, ) @@ -148,3 +148,82 @@ class DictWrapperTests(SimpleTestCase): "Normal: %(a)s. Modified: %(xx_a)s" % d, 'Normal: a. Modified: *a' ) + + +class CaseInsensitiveMappingTests(SimpleTestCase): + def setUp(self): + self.dict1 = CaseInsensitiveMapping({ + 'Accept': 'application/json', + 'content-type': 'text/html', + }) + + def test_create_with_invalid_values(self): + msg = 'dictionary update sequence element #1 has length 4; 2 is required' + with self.assertRaisesMessage(ValueError, msg): + CaseInsensitiveMapping([('Key1', 'Val1'), 'Key2']) + + def test_create_with_invalid_key(self): + msg = 'Element key 1 invalid, only strings are allowed' + with self.assertRaisesMessage(ValueError, msg): + CaseInsensitiveMapping([(1, '2')]) + + def test_list(self): + self.assertEqual(sorted(list(self.dict1)), sorted(['Accept', 'content-type'])) + + def test_dict(self): + self.assertEqual(dict(self.dict1), {'Accept': 'application/json', 'content-type': 'text/html'}) + + def test_repr(self): + dict1 = CaseInsensitiveMapping({'Accept': 'application/json'}) + dict2 = CaseInsensitiveMapping({'content-type': 'text/html'}) + self.assertEqual(repr(dict1), repr({'Accept': 'application/json'})) + self.assertEqual(repr(dict2), repr({'content-type': 'text/html'})) + + def test_str(self): + dict1 = CaseInsensitiveMapping({'Accept': 'application/json'}) + dict2 = CaseInsensitiveMapping({'content-type': 'text/html'}) + self.assertEqual(str(dict1), str({'Accept': 'application/json'})) + self.assertEqual(str(dict2), str({'content-type': 'text/html'})) + + def test_equal(self): + self.assertEqual(self.dict1, {'Accept': 'application/json', 'content-type': 'text/html'}) + self.assertNotEqual(self.dict1, {'accept': 'application/jso', 'Content-Type': 'text/html'}) + self.assertNotEqual(self.dict1, 'string') + + def test_items(self): + other = {'Accept': 'application/json', 'content-type': 'text/html'} + self.assertEqual(sorted(self.dict1.items()), sorted(other.items())) + + def test_copy(self): + copy = self.dict1.copy() + self.assertIs(copy, self.dict1) + self.assertEqual(copy, self.dict1) + + def test_getitem(self): + self.assertEqual(self.dict1['Accept'], 'application/json') + self.assertEqual(self.dict1['accept'], 'application/json') + self.assertEqual(self.dict1['aCCept'], 'application/json') + self.assertEqual(self.dict1['content-type'], 'text/html') + self.assertEqual(self.dict1['Content-Type'], 'text/html') + self.assertEqual(self.dict1['Content-type'], 'text/html') + + def test_in(self): + self.assertIn('Accept', self.dict1) + self.assertIn('accept', self.dict1) + self.assertIn('aCCept', self.dict1) + self.assertIn('content-type', self.dict1) + self.assertIn('Content-Type', self.dict1) + + def test_del(self): + self.assertIn('Accept', self.dict1) + msg = "'CaseInsensitiveMapping' object does not support item deletion" + with self.assertRaisesMessage(TypeError, msg): + del self.dict1['Accept'] + self.assertIn('Accept', self.dict1) + + def test_set(self): + self.assertEqual(len(self.dict1), 2) + msg = "'CaseInsensitiveMapping' object does not support item assignment" + with self.assertRaisesMessage(TypeError, msg): + self.dict1['New Key'] = 1 + self.assertEqual(len(self.dict1), 2) |
