From 86fd920f6761cbf93a7e5a9eb3c634e0e168a038 Mon Sep 17 00:00:00 2001 From: Aymeric Augustin Date: Thu, 7 Mar 2013 17:20:50 +0100 Subject: [PATCH] Removed a test that no longer makes any sense. Since unmanaged == autocommit, there's nothing to commit or roll back. --- tests/middleware/tests.py | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/tests/middleware/tests.py b/tests/middleware/tests.py index 7e26037967..74b8ab9601 100644 --- a/tests/middleware/tests.py +++ b/tests/middleware/tests.py @@ -703,19 +703,6 @@ class TransactionMiddlewareTest(IgnorePendingDeprecationWarningsMixin, Transacti self.assertFalse(transaction.is_dirty()) self.assertEqual(Band.objects.count(), 1) - # TODO: update this test to account for database-level autocommit. - # Currently it fails under PostgreSQL because connections are never - # marked dirty in non-managed mode. - @expectedFailure - def test_unmanaged_response(self): - transaction.enter_transaction_management(False) - self.assertEqual(Band.objects.count(), 0) - TransactionMiddleware().process_response(self.request, self.response) - self.assertFalse(transaction.is_managed()) - # The transaction middleware doesn't commit/rollback if management - # has been disabled. - self.assertTrue(transaction.is_dirty()) - def test_exception(self): transaction.enter_transaction_management() Band.objects.create(name='The Beatles')