diff options
| author | Kevin Stone | 2013-03-28 18:48:48 -0700 |
|---|---|---|
| committer | Kevin Stone | 2013-03-28 18:48:48 -0700 |
| commit | 4531ded061831a9cf402c6c5d84e42f31bc025ad (patch) | |
| tree | 281cc675ae1bc1e1ff47e113cee3cb3a4aa773ca | |
| parent | d4df617f8c1980c1d5f1b91a6b9928185c4c4dce (diff) | |
| download | django-rest-framework-4531ded061831a9cf402c6c5d84e42f31bc025ad.tar.bz2 | |
Removed pagination regression special case for Django<1.4. Having DjangoFilterBackend return an actual query set fixes this issue.
Signed-off-by: Kevin Stone <kevinastone@gmail.com>
| -rw-r--r-- | rest_framework/tests/pagination.py | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/rest_framework/tests/pagination.py b/rest_framework/tests/pagination.py index d2c9b051..6b8ef02f 100644 --- a/rest_framework/tests/pagination.py +++ b/rest_framework/tests/pagination.py @@ -129,16 +129,6 @@ class IntegrationTestPaginationAndFiltering(TestCase): view = FilterFieldsRootView.as_view() EXPECTED_NUM_QUERIES = 2 - if django.VERSION < (1, 4): - # On Django 1.3 we need to use django-filter 0.5.4 - # - # The filter objects there don't expose a `.count()` method, - # which means we only make a single query *but* it's a single - # query across *all* of the queryset, instead of a COUNT and then - # a SELECT with a LIMIT. - # - # Although this is fewer queries, it's actually a regression. - EXPECTED_NUM_QUERIES = 1 request = factory.get('/?decimal=15.20') with self.assertNumQueries(EXPECTED_NUM_QUERIES): |