[Erp5-report] r43400 vincent - /erp5/trunk/products/ZMySQLDDA/tests/testDeferredConnection.py

[nobody at svn.erp5.org]

Author: vincent
Date: Thu Feb 17 09:59:01 2011
New Revision: 43400

URL: http://svn.erp5.org?rev=43400&view=rev
Log:
Fix testDeferredConnection failures.

The 2nd test failed because of a side effect of 1st test.
1st test failed because it (purposely) causes a failure when committing
transaction, and this is remembered inside the transaction itself, and it
then always refuses commit. Aborting is the normal execution path after an
error in commit, so do it.

Modified:
    erp5/trunk/products/ZMySQLDDA/tests/testDeferredConnection.py

Modified: erp5/trunk/products/ZMySQLDDA/tests/testDeferredConnection.py
URL: http://svn.erp5.org/erp5/trunk/products/ZMySQLDDA/tests/testDeferredConnection.py?rev=43400&r1=43399&r2=43400&view=diff
==============================================================================
--- erp5/trunk/products/ZMySQLDDA/tests/testDeferredConnection.py [utf8] (original)
+++ erp5/trunk/products/ZMySQLDDA/tests/testDeferredConnection.py [utf8] Thu Feb 17 09:59:01 2011
@@ -151,6 +151,7 @@ class TestDeferredConnection(ERP5TypeTes
       else:
         self.fail()
     finally:
+      transaction.abort()
       delattr(connection, '_query')
       self.unmonkeypatchConnection(connection)