Commit 692203f9 by Ben Konrath

Check for 200 status when unknown filter requested.

This changes the test from the failing checking for status 400. See
discussion here:

https://github.com/tomchristie/django-rest-framework/pull/169#issuecomment-9240480
parent 1e9ece0f
...@@ -147,14 +147,13 @@ class IntegrationTestFiltering(TestCase): ...@@ -147,14 +147,13 @@ class IntegrationTestFiltering(TestCase):
request = factory.get('/') request = factory.get('/')
self.assertRaises(AssertionError, view, request) self.assertRaises(AssertionError, view, request)
# TODO Return 400 filter paramater requested that hasn't been configured. def test_unknown_filter(self):
def test_bad_request(self):
""" """
GET requests with filters that aren't configured should return 400. GET requests with filters that aren't configured should return 200.
""" """
view = FilterFieldsRootView.as_view() view = FilterFieldsRootView.as_view()
search_integer = 10 search_integer = 10
request = factory.get('/?integer=%s' % search_integer) request = factory.get('/?integer=%s' % search_integer)
response = view(request).render() response = view(request).render()
self.assertEquals(response.status_code, status.HTTP_400_BAD_REQUEST) self.assertEquals(response.status_code, status.HTTP_200_OK)
\ No newline at end of file \ No newline at end of file
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment