[ http://issues.apache.org/jira/browse/JCR-540?page=all ]

Marcel Reutegger updated JCR-540:
---------------------------------

    Issue Type: Improvement  (was: Bug)

This is done on purpose. Throwing a NotExecutableException will indicate to the 
TCK that the test case
did not fail even though it did not run. Hence the test was successful.

If all Level 2 tests on a Level 1 repository throw NotExecutableExceptions the 
TCK would finally declare the Repository Level 2 compliant which is not the 
case.

Failing Level 2 test cases of a Level 1 repository do *not* have an effect on 
its Level 1 compliance.

> AbstractJCRTest fails on level 1 repositories
> ---------------------------------------------
>
>                 Key: JCR-540
>                 URL: http://issues.apache.org/jira/browse/JCR-540
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: JCR TCK
>            Reporter: Julian Reschke
>            Priority: Minor
>         Attachments: JCR-540.diff.txt
>
>
> If a test case indicates that it's not read-only, 
> org.apache.jackrabbit.test.AbstractJCRTest tries to cleanup the test root in 
> the setUp method. This will cause the test case to fail, because a Level 1 
> repository will throw an UnsupportedOperationException here.
> Proposal: before trying the cleanup, check for L2 functionality and throw a 
> NotExecutableException otherwise:
>             if (! isSupported(Repository.LEVEL_2_SUPPORTED)) {
>               cleanUp();
>               String msg = "Test case requires level 2 functionality";
>               throw new NotExecutableException(msg);
>             }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to