diff options
| author | Ben Konrath | 2012-10-09 08:22:00 +0200 |
|---|---|---|
| committer | Ben Konrath | 2012-10-11 12:01:07 +0200 |
| commit | 692203f933b77cc3b18e15434002169b642fbd84 (patch) | |
| tree | 5c5b1e7c6e989d645b92bbcf9fcbe64631a27565 | |
| parent | 1e9ece0f9353515265da9b6266dc4b39775a0257 (diff) | |
| download | django-rest-framework-692203f933b77cc3b18e15434002169b642fbd84.tar.bz2 | |
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
| -rw-r--r-- | rest_framework/tests/filterset.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/rest_framework/tests/filterset.py b/rest_framework/tests/filterset.py index 8c857f3f..b21abacb 100644 --- a/rest_framework/tests/filterset.py +++ b/rest_framework/tests/filterset.py @@ -147,14 +147,13 @@ class IntegrationTestFiltering(TestCase): request = factory.get('/') self.assertRaises(AssertionError, view, request) - # TODO Return 400 filter paramater requested that hasn't been configured. - def test_bad_request(self): + def test_unknown_filter(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() search_integer = 10 request = factory.get('/?integer=%s' % search_integer) response = view(request).render() - self.assertEquals(response.status_code, status.HTTP_400_BAD_REQUEST)
\ No newline at end of file + self.assertEquals(response.status_code, status.HTTP_200_OK)
\ No newline at end of file |