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();
         }


Reply via email to