From 4fcc1e4ad8d153f41132b171c231b6d5d4086c28 Mon Sep 17 00:00:00 2001 From: Florian Apolloner Date: Sun, 22 Sep 2013 22:58:46 +0200 Subject: [PATCH] Stopped a test from executing queries at the module level. Currently module level queries are executed against the real database (specified in NAME) instead of the test database; since it is to late to fix this for 1.6, we at least ensures stable builds. Refs #21443. --- tests/transactions_regress/tests.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/tests/transactions_regress/tests.py b/tests/transactions_regress/tests.py index 2376bcd277..e3453d329d 100644 --- a/tests/transactions_regress/tests.py +++ b/tests/transactions_regress/tests.py @@ -1,6 +1,6 @@ from __future__ import unicode_literals -from unittest import skipIf, skipUnless +from unittest import skipIf, skipUnless, SkipTest from django.db import (connection, connections, transaction, DEFAULT_DB_ALIAS, DatabaseError, IntegrityError) @@ -367,11 +367,13 @@ class SavepointTest(IgnoreDeprecationWarningsMixin, TransactionTestCase): @skipIf(connection.vendor == 'sqlite', "SQLite doesn't support savepoints in managed mode") - @skipIf(connection.vendor == 'mysql' and - connection.features._mysql_storage_engine == 'MyISAM', - "MyISAM MySQL storage engine doesn't support savepoints") @skipUnlessDBFeature('uses_savepoints') def test_savepoint_rollback(self): + # _mysql_storage_engine issues a query and as such can't be applied in + # a skipIf decorator since that would execute the query on module load. + if (connection.vendor == 'mysql' and + connection.features._mysql_storage_engine == 'MyISAM'): + raise SkipTest("MyISAM MySQL storage engine doesn't support savepoints") @commit_manually def work(): mod = Mod.objects.create(fld=1)