[ 
https://issues.apache.org/jira/browse/SLING-3224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16140043#comment-16140043
 ] 

Robert Munteanu commented on SLING-3224:
----------------------------------------

* fixed bug with minor refactoring + extensive tests in 
[r1806048|https://svn.apache.org/r1806048] 
* remove failing test in [r1806049|https://svn.apache.org/1806049] after 
validating that the test passes with the SNAPSHOT version of the bundle

I've removed the test since it was testing a very specific scenario now covered 
in a unit test placed in the bundle. I've left the over tests to make sure we 
have some integration coverage.

> GetAclTest integration test fails due to incorrect privilege expansion in 
> AbstractGetAclServlet
> -----------------------------------------------------------------------------------------------
>
>                 Key: SLING-3224
>                 URL: https://issues.apache.org/jira/browse/SLING-3224
>             Project: Sling
>          Issue Type: Bug
>          Components: JCR
>            Reporter: Bertrand Delacretaz
>            Assignee: Robert Munteanu
>              Labels: sling-IT
>             Fix For: JCR Jackrabbit Access Manager 3.0.2
>
>
> Failed tests:   testEffectiveAclMergeForUser_SubsetOfPrivilegesDeniedOnChild:
> Expected privilege jcr:modifyProperties to be NOT INCLUDED in supplied list: 
> [rep:userManagement, jcr:nodeTypeManagement, jcr:modifyProperties, 
> jcr:namespaceManagement, rep:privilegeManagement, jcr:workspaceManagement, 
> rep:readProperties, rep:alterProperties, jcr:nodeTypeDefinitionManagement, 
> jcr:lockManagement, jcr:read, jcr:lifecycleManagement, jcr:removeNode, 
> jcr:modifyAccessControl, jcr:removeChildNodes, jcr:versionManagement, 
> rep:addProperties, rep:removeProperties, rep:readNodes, 
> jcr:readAccessControl, jcr:addChildNodes, jcr:retentionManagement])



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to