From 14d24eb438c9930ff76302f263d0f2a8b42d13c1 Mon Sep 17 00:00:00 2001 From: Aymeric Augustin Date: Tue, 18 Oct 2011 16:42:32 +0000 Subject: [PATCH] Fixed #17067 -- reverted some backwards-incompatible changes from r16933 and added tests. git-svn-id: http://code.djangoproject.com/svn/django/trunk@17010 bcc190cf-cafb-0310-a4f2-bffc1f526a37 --- django/test/client.py | 8 ++++++-- .../test_client_regress/models.py | 20 +++++++++++++++++++ 2 files changed, 26 insertions(+), 2 deletions(-) diff --git a/django/test/client.py b/django/test/client.py index a5ef075f3c..30ef746f6c 100644 --- a/django/test/client.py +++ b/django/test/client.py @@ -182,12 +182,16 @@ class RequestFactory(object): """ The base environment for a request. """ - # This is a minimal valid WSGI environ dictionary, plus HTTP_COOKIE - # for cookie support. Empty strings are omitted. + # This is a minimal valid WSGI environ dictionary, plus: + # - HTTP_COOKIE: for cookie support, + # - REMOTE_ADDR: often useful, see #8551. # See http://www.python.org/dev/peps/pep-3333/#environ-variables environ = { 'HTTP_COOKIE': self.cookies.output(header='', sep='; '), + 'PATH_INFO': '/', + 'REMOTE_ADDR': '127.0.0.1', 'REQUEST_METHOD': 'GET', + 'SCRIPT_NAME': '', 'SERVER_NAME': 'testserver', 'SERVER_PORT': '80', 'SERVER_PROTOCOL': 'HTTP/1.1', diff --git a/tests/regressiontests/test_client_regress/models.py b/tests/regressiontests/test_client_regress/models.py index 18ffffde1a..ff851971ff 100644 --- a/tests/regressiontests/test_client_regress/models.py +++ b/tests/regressiontests/test_client_regress/models.py @@ -994,3 +994,23 @@ class RequestFactoryStateTest(TestCase): def test_request_after_client_2(self): # This test is executed after the previous one self.common_test_that_should_always_pass() + + +class RequestFactoryEnvironmentTests(TestCase): + """ + Regression tests for #8551 and #17067: ensure that environment variables + are set correctly in RequestFactory. + """ + + def setUp(self): + self.factory = RequestFactory() + + def test_should_set_correct_env_variables(self): + request = self.factory.get('/path/') + + self.assertEqual(request.META.get('REMOTE_ADDR'), '127.0.0.1') + self.assertEqual(request.META.get('SERVER_NAME'), 'testserver') + self.assertEqual(request.META.get('SERVER_PORT'), '80') + self.assertEqual(request.META.get('SERVER_PROTOCOL'), 'HTTP/1.1') + self.assertEqual(request.META.get('SCRIPT_NAME') + + request.META.get('PATH_INFO'), '/path/')