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