You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by al...@apache.org on 2013/07/12 09:50:13 UTC

svn commit: r1502462 - /jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MoveTest.java

Author: alexparvulescu
Date: Fri Jul 12 07:50:12 2013
New Revision: 1502462

URL: http://svn.apache.org/r1502462
Log:
OAK-898 Session#move with a fresh session doesn't work properly
 - tweaked the test

Modified:
    jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MoveTest.java

Modified: jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MoveTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MoveTest.java?rev=1502462&r1=1502461&r2=1502462&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MoveTest.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MoveTest.java Fri Jul 12 07:50:12 2013
@@ -72,7 +72,8 @@ public class MoveTest extends AbstractJC
     /**
      * Simulate a 'rename' call using 2 sessions:
      * - 1st create a node that has a '.tmp' extension 
-     * - 2nd remove the '.tmp' by issuing a Session#move call on a fresh session
+     * - 2nd remove the '.tmp' by issuing a Session#move call on a new session
+     * - 3rd verify the move by issuing a #getNode call on the destination path using a new session
      */
     @Test
     @Ignore("OAK-898")
@@ -87,7 +88,13 @@ public class MoveTest extends AbstractJC
         Session ts = getHelper().getSuperuserSession();
         try {
             ts.move(node1.getPath(), destPath);
-            assertEquals(destPath, node1.getPath());
+        } finally {
+            ts.logout();
+        }
+
+        ts = getHelper().getSuperuserSession();
+        try {
+            assertNotNull(ts.getNode(destPath));
         } finally {
             ts.logout();
         }