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