You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@openjpa.apache.org by kw...@apache.org on 2012/03/30 22:20:04 UTC

svn commit: r1307595 - /openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestSavepoints.java

Author: kwsutter
Date: Fri Mar 30 20:20:04 2012
New Revision: 1307595

URL: http://svn.apache.org/viewvc?rev=1307595&view=rev
Log:
OPENJPA-2164.  Change the Savepoints testcase to be more consistent with the expectations of savepoints.

Modified:
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestSavepoints.java

Modified: openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestSavepoints.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestSavepoints.java?rev=1307595&r1=1307594&r2=1307595&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestSavepoints.java (original)
+++ openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestSavepoints.java Fri Mar 30 20:20:04 2012
@@ -147,10 +147,10 @@ public class TestSavepoints extends Base
         pm.setSavepoint("s1");
         pc = pm.find(ModRuntimeTest1.class, oid);
         assertTrue(pm.isTransactional(pc));
+        pc.setStringField("test");
         pm.setSavepoint("s2");
         pc.setStringField("bar");
         pm.rollbackToSavepoint("s2");
-        assertEquals("orig", pc.getStringField());
         assertTrue(pm.isTransactional(pc));
 
         rollbackTx(pm);