- 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 6918996a027b02b586a7ba9ca7e7ca3a832cf7ca..a51992a0da8701fe1bc835d31c19821981bd1297 100644 (file)
--- 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)])