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

Bertrand Delacretaz commented on SLING-9901:
--------------------------------------------

I have created 
[https://github.com/apache/sling-org-apache-sling-jcr-resource/pull/10] with 
the {{JcrSystemUserValidatorTest}} modified to use Oak for testing, via Sling 
Mocks.

I have not added any new tests so far but [~angela] that should allow you to 
add the new tests that you have in mind?

Note that I had to increase the {{org.apache.sling.scripting.api}} version to 
2.2.0 (from 2.1.0) which changes the bundle resolution requirements, but the 
bundle has {{                        
org.apache.sling.scripting.api.*;resolution:=optional}} so I suppose that's not 
a problem. 

> JcrSystemUserValidatorTest contains not tests for system users
> --------------------------------------------------------------
>
>                 Key: SLING-9901
>                 URL: https://issues.apache.org/jira/browse/SLING-9901
>             Project: Sling
>          Issue Type: Improvement
>          Components: JCR
>            Reporter: Angela Schreiber
>            Priority: Minor
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> _org.apache.sling.jcr.resource_ comes with a {{JcrSystemUserValidator}} that 
> adds validation to service-user-mapping and if enable prevents regular users 
> or groups from being mapped.
> however, {{JcrSystemUserValidatorTest}} doesn't contain a single test case 
> verifying if the validation is actually successful for system users (or that 
> disabled system users would not be valid).
> i suspect that this is due to the fact that 
> {{org.apache.sling.commons.testing.jcr.RepositoryTestBase}} creates a 
> Jackrabbit repository and has never been adjusted to allow for an Oak 
> repository to be used as test repository. In other words: an attempt to 
> create a system user will fail.
> [~rombert], is there an alternative to the {{RepositoryTestBase}} that would 
> operate on a Oak repository?



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to