diff options
| author | Aymeric Augustin <aymeric.augustin@m4x.org> | 2014-06-08 22:42:12 +0200 |
|---|---|---|
| committer | Aymeric Augustin <aymeric.augustin@m4x.org> | 2014-06-09 21:38:10 +0200 |
| commit | 40bfd8561d016f6632c95d03e46a720571ad1a8a (patch) | |
| tree | a56e3a87281529f13b8c6a19d180ab9e9e54f052 /tests | |
| parent | 127218b92bb8482fcb8b02457ed9797486ed3a22 (diff) | |
Fixed #20420 -- Normalized query counts on Oracle.
This is achieved by inserting a fake entry in connection.queries when
not releasing a savepoint (since Oracle doesn't support that operation.)
Also removed the can_release_savepoints feature that was recently added,
but is superseded by this solution.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/admin_views/tests.py | 13 |
1 files changed, 2 insertions, 11 deletions
diff --git a/tests/admin_views/tests.py b/tests/admin_views/tests.py index 4b08956936..82cabc1cfb 100644 --- a/tests/admin_views/tests.py +++ b/tests/admin_views/tests.py @@ -3960,11 +3960,7 @@ class UserAdminTest(TestCase): # Don't depend on a warm cache, see #17377. ContentType.objects.clear_cache() - expected_queries = 10 - if not connection.features.can_release_savepoints: - expected_queries -= 1 - - with self.assertNumQueries(expected_queries): + with self.assertNumQueries(10): response = self.client.get('/test_admin/admin/auth/user/%s/' % u.pk) self.assertEqual(response.status_code, 200) @@ -4001,12 +3997,7 @@ class GroupAdminTest(TestCase): def test_group_permission_performance(self): g = Group.objects.create(name="test_group") - - expected_queries = 8 - if not connection.features.can_release_savepoints: - expected_queries -= 1 - - with self.assertNumQueries(expected_queries): + with self.assertNumQueries(8): response = self.client.get('/test_admin/admin/auth/group/%s/' % g.pk) self.assertEqual(response.status_code, 200) |
