From 74997f95def1ed655f24bdd526aed1f8912fe646 Mon Sep 17 00:00:00 2001 From: Karen Tracey Date: Wed, 22 Oct 2008 23:14:48 +0000 Subject: [1.0.X] Fixed #9252 -- Moved the try/except protecting against incorrect lookup params to where the error is now raised, and added a test for this case. Backport of [9245] from trunk. git-svn-id: http://code.djangoproject.com/svn/django/branches/releases/1.0.X@9246 bcc190cf-cafb-0310-a4f2-bffc1f526a37 --- tests/regressiontests/admin_views/tests.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) (limited to 'tests/regressiontests') diff --git a/tests/regressiontests/admin_views/tests.py b/tests/regressiontests/admin_views/tests.py index bc9d1d49ae..f335c093de 100644 --- a/tests/regressiontests/admin_views/tests.py +++ b/tests/regressiontests/admin_views/tests.py @@ -160,7 +160,14 @@ class AdminViewBasicTest(TestCase): 'Blue' in response.content, "Changelist filter not correctly limited by limit_choices_to." ) - + + def testIncorrectLookupParameters(self): + """Ensure incorrect lookup parameters are handled gracefully.""" + response = self.client.get('/test_admin/admin/admin_views/thing/', {'notarealfield': '5'}) + self.assertRedirects(response, '/test_admin/admin/admin_views/thing/?e=1') + response = self.client.get('/test_admin/admin/admin_views/thing/', {'color__id__exact': 'StringNotInteger!'}) + self.assertRedirects(response, '/test_admin/admin/admin_views/thing/?e=1') + def get_perm(Model, perm): """Return the permission object, for the Model""" ct = ContentType.objects.get_for_model(Model) -- cgit v1.3