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 22:08:20 +0200 |
| commit | 5cde9a7018021fc3c005e4ebcfa63967bcf56285 (patch) | |
| tree | 7cb27f9673299e419df7c7ecb4c557758f50b97c /tests | |
| parent | e5879811e6854852a2a0d8c850dd2a1795e09e9a (diff) | |
[1.7.x] 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.
Backport of 40bfd856 from master.
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 21f1eb818c..957c7cd033 100644 --- a/tests/admin_views/tests.py +++ b/tests/admin_views/tests.py @@ -3962,11 +3962,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) @@ -4003,12 +3999,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) |
