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 ju...@apache.org on 2013/02/14 01:52:40 UTC

svn commit: r1446005 - /jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/segment/SegmentSizeTest.java

Author: jukka
Date: Thu Feb 14 00:52:40 2013
New Revision: 1446005

URL: http://svn.apache.org/r1446005
Log:
OAK-593: Segment-based MK

Fix minor issue in the size test

Modified:
    jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/segment/SegmentSizeTest.java

Modified: jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/segment/SegmentSizeTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/segment/SegmentSizeTest.java?rev=1446005&r1=1446004&r2=1446005&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/segment/SegmentSizeTest.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/segment/SegmentSizeTest.java Thu Feb 14 00:52:40 2013
@@ -113,7 +113,7 @@ public class SegmentSizeTest {
         NodeBuilder deny = builder.child("deny");
         deny.setProperty("jcr:primaryType", "rep:DenyACE", Type.NAME);
         deny.setProperty("rep:principalName", "everyone");
-        builder.setProperty(PropertyStates.createProperty(
+        deny.setProperty(PropertyStates.createProperty(
                 "rep:privileges", ImmutableList.of("jcr:read"), Type.NAMES));
         assertEquals(134, getSize(builder));
         assertEquals(28, getAmortizedSize(builder));
@@ -132,16 +132,16 @@ public class SegmentSizeTest {
         deny0.setProperty("rep:glob", "*/activities/*");
         builder.setProperty(PropertyStates.createProperty(
                 "rep:privileges", ImmutableList.of("jcr:read"), Type.NAMES));
-        assertEquals(331, getSize(builder));
-        assertEquals(104, getAmortizedSize(builder));
+        assertEquals(348, getSize(builder));
+        assertEquals(116, getAmortizedSize(builder));
 
         NodeBuilder allow0 = builder.child("allow0");
         allow0.setProperty("jcr:primaryType", "rep:GrantACE");
         allow0.setProperty("rep:principalName", "user-administrators");
         allow0.setProperty(PropertyStates.createProperty(
                 "rep:privileges", ImmutableList.of("jcr:all"), Type.NAMES));
-        assertEquals(394, getSize(builder));
-        assertEquals(140, getAmortizedSize(builder));
+        assertEquals(411, getSize(builder));
+        assertEquals(152, getAmortizedSize(builder));
     }
 
     private int getSize(NodeBuilder builder) {