[MMBASE SCM] r40795 - mmbase/branches/MMBase-1_9/tests/bridge/org/mmbase/bridge

[michiel]

Author: michiel
Date: 2010-02-02 14:59:49 +0100 (Tue, 02 Feb 2010)
New Revision: 40795

Modified:
   
mmbase/branches/MMBase-1_9/tests/bridge/org/mmbase/bridge/TransactionTest.java
Log:
added a few assertions

Modified: 
mmbase/branches/MMBase-1_9/tests/bridge/org/mmbase/bridge/TransactionTest.java
===================================================================
--- 
mmbase/branches/MMBase-1_9/tests/bridge/org/mmbase/bridge/TransactionTest.java  
    2010-02-02 13:47:26 UTC (rev 40794)
+++ 
mmbase/branches/MMBase-1_9/tests/bridge/org/mmbase/bridge/TransactionTest.java  
    2010-02-02 13:59:49 UTC (rev 40795)
@@ -136,29 +136,48 @@
     //Test for http://www.mmbase.org/jira/browse/MMB-1621
     public void testGetValue() {
         Cloud cloud = getCloud();
+
         String value = cloud.getNode(newNode).getStringValue("title");
+        assertFalse("zzzzz".equals(value));
 
         Transaction t = cloud.getTransaction("bar4");
+        System.out.println("Transaction now " + t);
         Node node = t.getNode(newNode);
         assertEquals(1, t.getNodes().size());
         node.setStringValue("title", "zzzzz");
+        assertEquals("zzzzz", node.getStringValue("title"));
         node.commit(); // committing inside transaction
 
+
+        assertEquals("zzzzz", node.getStringValue("title"));
         assertEquals(value, cloud.getNode(newNode).getStringValue("title"));
 
         t.commit();
         assertEquals("zzzzz", cloud.getNode(newNode).getStringValue("title"));
 
     }
+
     public void testReuseTransaction() {
         Cloud cloud = getCloud();
-        Transaction t = cloud.getTransaction("bar4");
-        assertEquals(0, t.getNodes().size());
-        Node node = t.getNode(newNode);
-        assertEquals(1, t.getNodes().size());
-        node.setStringValue("title", "wwwwww");
-        node.commit();
-        t.cancel();
+        {
+            Node node = cloud.getNode(newNode);
+            node.setStringValue("title", "zzyyxx");
+            node.commit();
+            assertEquals("zzyyxx", 
cloud.getNode(newNode).getStringValue("title"));
+        }
+        {
+            Transaction t = cloud.getTransaction("bar4");
+            assertEquals(0, t.getNodes().size());
+            Node node = t.getNode(newNode);
+            assertEquals("zzyyxx", node.getStringValue("title"));
+            assertEquals(1, t.getNodes().size());
+            node.setStringValue("title", "wwwwww");
+            node.commit();
+            assertEquals("wwwwww", node.getStringValue("title"));
+            t.cancel();
+            assertEquals("zzyyxx", 
cloud.getNode(newNode).getStringValue("title"));
+        }
+        assertEquals("zzyyxx", cloud.getNode(newNode).getStringValue("title"));
 
     }
 
@@ -654,5 +673,4 @@
     }
 
 
-
 }

_______________________________________________
Cvs mailing list
Cvs@lists.mmbase.org
http://lists.mmbase.org/mailman/listinfo/cvs