[jira] Resolved: (JCR-1948) Let the AbstractISMLockingTest tests fail properly

["Martijn Hendriks (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/JCR-1948?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Martijn Hendriks resolved JCR-1948.
-----------------------------------

    Resolution: Fixed

> Let the AbstractISMLockingTest tests fail properly
> --------------------------------------------------
>
>                 Key: JCR-1948
>                 URL: https://issues.apache.org/jira/browse/JCR-1948
>             Project: Jackrabbit Content Repository
>          Issue Type: Improvement
>          Components: test
>            Reporter: Martijn Hendriks
>         Attachments: AbstractISMLockingTest.patch
>
>
> The tests in the AbstractISMLockingTest class call junit.framework.Assert.fail() on threads that are not managed by the JUnit framework. Therefore, such calls to fail are not interpreted as test failures, but are merely logged to the console and the build succeeds. This is easy to see with a stub implementation of the ISMLocking type which returns non-null references from the two acquire methods and the downgrade method: many of the following stacktraces appear, but the build succeeds.
> Exception in thread "Thread-1" junit.framework.AssertionFailedError: acquireWriteLock must block
> 	at junit.framework.Assert.fail(Assert.java:47)
> 	at org.apache.jackrabbit.core.state.AbstractISMLockingTest.checkBlocking(AbstractISMLockingTest.java:214)
> 	at org.apache.jackrabbit.core.state.AbstractISMLockingTest$1.run(AbstractISMLockingTest.java:88)
> 	at java.lang.Thread.run(Thread.java:613)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.