Mailing-List: contact oak-commits-help@jackrabbit.apache.org; run by ezmlm
Precedence: bulk
Reply-To: oak-dev@jackrabbit.apache.org
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r1502462 -
 /jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MoveTest.java
Date: Fri, 12 Jul 2013 07:50:13 -0000
To: oak-commits@jackrabbit.apache.org
From: alexparvulescu@apache.org
Message-Id: <20130712075013.1A70623888A6@eris.apache.org>

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