diff --git a/django/test/client.py b/django/test/client.py index a3a8f21a637..b1ff33e3d30 100644 --- a/django/test/client.py +++ b/django/test/client.py @@ -279,8 +279,9 @@ class RequestFactory(object): def get(self, path, data=None, secure=False, **extra): "Construct a GET request." + data = {} if data is None else data r = { - 'QUERY_STRING': urlencode(data or {}, doseq=True), + 'QUERY_STRING': urlencode(data, doseq=True), } r.update(extra) return self.generic('GET', path, secure=secure, **r) @@ -289,7 +290,8 @@ class RequestFactory(object): secure=False, **extra): "Construct a POST request." - post_data = self._encode_data(data or {}, content_type) + data = {} if data is None else data + post_data = self._encode_data(data, content_type) return self.generic('POST', path, post_data, content_type, secure=secure, **extra) @@ -297,8 +299,9 @@ class RequestFactory(object): def head(self, path, data=None, secure=False, **extra): "Construct a HEAD request." + data = {} if data is None else data r = { - 'QUERY_STRING': urlencode(data or {}, doseq=True), + 'QUERY_STRING': urlencode(data, doseq=True), } r.update(extra) return self.generic('HEAD', path, secure=secure, **r) diff --git a/docs/releases/1.7.1.txt b/docs/releases/1.7.1.txt index f356d9d8d50..215e2fc2111 100644 --- a/docs/releases/1.7.1.txt +++ b/docs/releases/1.7.1.txt @@ -129,3 +129,6 @@ Bugfixes * Fixed :djadmin:`makemigrations` to detect changes to :attr:`Meta.db_table ` (:ticket:`23629`). + +* Fixed a regression when feeding the Django test client with an empty data + string (:ticket:`21740`). diff --git a/tests/test_client_regress/tests.py b/tests/test_client_regress/tests.py index e5dfffe5706..b3656e49633 100644 --- a/tests/test_client_regress/tests.py +++ b/tests/test_client_regress/tests.py @@ -1228,6 +1228,16 @@ class RequestMethodStringDataTests(TestCase): self.assertEqual(response.status_code, 200) self.assertEqual(response.content, b'request method: PATCH') + def test_empty_string_data(self): + "Request a view with empty string data via request method GET/POST/HEAD" + # Regression test for #21740 + response = self.client.get('/body/', data='', content_type='application/json') + self.assertEqual(response.content, b'') + response = self.client.post('/body/', data='', content_type='application/json') + self.assertEqual(response.content, b'') + response = self.client.head('/body/', data='', content_type='application/json') + self.assertEqual(response.content, b'') + @override_settings(ROOT_URLCONF='test_client_regress.urls',) class QueryStringTests(TestCase):