Update of /var/cvs/tests/bridge/org/mmbase/bridge
In directory james.mmbase.org:/tmp/cvs-serv7808

Modified Files:
        TransactionTest.java 
Log Message:
I think the fix for MMB-1680 must result in the the node deleted outside the 
transaction, but commited with the transaction, must result in the node to 
re-exist. Mirrored this in the test-case.


See also: http://cvs.mmbase.org/viewcvs/tests/bridge/org/mmbase/bridge
See also: http://www.mmbase.org/jira/browse/MMB-1680


Index: TransactionTest.java
===================================================================
RCS file: /var/cvs/tests/bridge/org/mmbase/bridge/TransactionTest.java,v
retrieving revision 1.8
retrieving revision 1.9
diff -u -b -r1.8 -r1.9
--- TransactionTest.java        15 Jul 2008 11:44:22 -0000      1.8
+++ TransactionTest.java        15 Jul 2008 11:59:01 -0000      1.9
@@ -11,15 +11,19 @@
 package org.mmbase.bridge;
 
 import org.mmbase.tests.*;
+import org.mmbase.util.logging.Logger;
+import org.mmbase.util.logging.Logging;
+
 
 /**
  * Test class <code>Transaction</code> from the bridge package.
  *
  * @author Michiel Meeuwissen
- * @version $Id: TransactionTest.java,v 1.8 2008/07/15 11:44:22 michiel Exp $
+ * @version $Id: TransactionTest.java,v 1.9 2008/07/15 11:59:01 michiel Exp $
  * @since MMBase-1.8.6
   */
 public class TransactionTest extends BridgeTest {
+    private static final Logger log = 
Logging.getLoggerInstance(TransactionTest.class);
 
     public TransactionTest(String name) {
         super(name);
@@ -214,10 +218,15 @@
             assertFalse(cloud.hasNode(newNode2));
         }
 
-
+        try {
         t.commit();
+        } catch (Exception e) {
+            // should not give exception. MMB-1680
+            log.error(e.getMessage(), e);
+            fail(e.getMessage());
+        }
 
-        assertFalse(cloud.hasNode(newNode2));
+        assertTrue(cloud.hasNode(newNode2));
     }
 
 
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to