From: Mike Bayer <mike_mp@zzzcomputing.com>
Date: Fri, 20 Nov 2015 23:19:33 +0000 (-0500)
Subject: - try to get this to skip for oracle
X-Git-Tag: rel_1_0_10~42
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=65d6a7ef2c091284c1a8f240a93bb6d07539075c;p=thirdparty%2Fsqlalchemy%2Fsqlalchemy.git

- try to get this to skip for oracle

(cherry picked from commit ef9a4cb60b4e7fe305367c5223e8bb2cbf2b3b0f)
---

diff --git a/test/orm/test_transaction.py b/test/orm/test_transaction.py
index 6918996a02..a51992a0da 100644
--- a/test/orm/test_transaction.py
+++ b/test/orm/test_transaction.py
@@ -657,6 +657,7 @@ class SessionTransactionTest(FixtureTest):
         assert session.transaction is not None, \
             'autocommit=False should start a new transaction'
 
+    @testing.skip_if("oracle", "oracle doesn't support release of savepoint")
     @testing.requires.savepoints
     def test_report_primary_error_when_rollback_fails(self):
         User, users = self.classes.User, self.tables.users
@@ -671,7 +672,9 @@ class SessionTransactionTest(FixtureTest):
                 connection()._Connection__transaction._savepoint
 
             # force the savepoint to disappear
-            session.execute("RELEASE SAVEPOINT %s" % savepoint)
+            session.connection().dialect.do_release_savepoint(
+                session.connection(), savepoint
+            )
 
             # now do a broken flush
             session.add_all([User(id=1), User(id=1)])