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 an...@apache.org on 2019/02/01 10:16:15 UTC
svn commit: r1852696 -
/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/security/authorization/evaluation/AbstractOakCoreTest.java
Author: angela
Date: Fri Feb 1 10:16:15 2019
New Revision: 1852696
URL: http://svn.apache.org/viewvc?rev=1852696&view=rev
Log:
OAK-8019 : Replace deprecated NodeUtil in AbstractOakCoreTest
Modified:
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/security/authorization/evaluation/AbstractOakCoreTest.java
Modified: jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/security/authorization/evaluation/AbstractOakCoreTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/security/authorization/evaluation/AbstractOakCoreTest.java?rev=1852696&r1=1852695&r2=1852696&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/security/authorization/evaluation/AbstractOakCoreTest.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/security/authorization/evaluation/AbstractOakCoreTest.java Fri Feb 1 10:16:15 2019
@@ -25,7 +25,8 @@ import org.apache.jackrabbit.commons.jac
import org.apache.jackrabbit.oak.AbstractSecurityTest;
import org.apache.jackrabbit.oak.api.ContentSession;
import org.apache.jackrabbit.oak.api.Root;
-import org.apache.jackrabbit.oak.util.NodeUtil;
+import org.apache.jackrabbit.oak.api.Tree;
+import org.apache.jackrabbit.oak.plugins.tree.TreeUtil;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
import org.junit.After;
@@ -40,7 +41,7 @@ import static org.apache.jackrabbit.JcrC
*/
public abstract class AbstractOakCoreTest extends AbstractSecurityTest {
- protected Principal testPrincipal;
+ protected Principal testPrincipal;
private ContentSession testSession;
@Before
@@ -50,18 +51,17 @@ public abstract class AbstractOakCoreTes
testPrincipal = getTestUser().getPrincipal();
- NodeUtil rootNode = new NodeUtil(root.getTree("/"));
- NodeUtil a = rootNode.addChild("a", NT_UNSTRUCTURED);
- a.setString("aProp", "aValue");
+ Tree a = TreeUtil.addChild(root.getTree("/"), "a", NT_UNSTRUCTURED);
+ a.setProperty("aProp", "aValue");
- NodeUtil b = a.addChild("b", NT_UNSTRUCTURED);
- b.setString("bProp", "bValue");
+ Tree b = TreeUtil.addChild(a, "b", NT_UNSTRUCTURED);
+ b.setProperty("bProp", "bValue");
// sibling
- NodeUtil bb = a.addChild("bb", NT_UNSTRUCTURED);
- bb.setString("bbProp", "bbValue");
+ Tree bb = TreeUtil.addChild(a, "bb", NT_UNSTRUCTURED);
+ bb.setProperty("bbProp", "bbValue");
- NodeUtil c = b.addChild("c", NT_UNSTRUCTURED);
- c.setString("cProp", "cValue");
+ Tree c = TreeUtil.addChild(b, "c", NT_UNSTRUCTURED);
+ c.setProperty("cProp", "cValue");
root.commit();
}
@@ -138,10 +138,10 @@ public abstract class AbstractOakCoreTes
@NotNull Principal principal,
boolean isAllow,
@NotNull String... privilegeNames) throws Exception {
- AccessControlManager acMgr = getAccessControlManager(root);
- JackrabbitAccessControlList acl = checkNotNull(AccessControlUtils.getAccessControlList(acMgr, path));
- acl.addEntry(principal, AccessControlUtils.privilegesFromNames(acMgr, privilegeNames), isAllow);
- acMgr.setPolicy(path, acl);
+ AccessControlManager acMgr = getAccessControlManager(root);
+ JackrabbitAccessControlList acl = checkNotNull(AccessControlUtils.getAccessControlList(acMgr, path));
+ acl.addEntry(principal, AccessControlUtils.privilegesFromNames(acMgr, privilegeNames), isAllow);
+ acMgr.setPolicy(path, acl);
root.commit();
}
}