Manik Surtani wrote:
Nope:
private void callCreateCacheManagers() {
try {
createCacheManagers();
} catch (Throwable th) {
th.printStackTrace();
log.error("Error in test setup: " + th);
}
}
So if there is no reason for this I'll change it to re-throw the
exception.
Cheers
Manik
I was saying that the code is wrong and the exception *should* be
re-thrown, not that it *is* re-thrown.
Is this a sort of public humiliation? :)
On 4 Aug 2009, at 12:14, Mircea Markus wrote:
> Manik Surtani wrote:
>> Mircea,
>>
>> Is there any good reason why exceptions thrown in
>> createCacheManagers() get swallowed (well, logged) but does not
>> cause the tests being run to fail?
>>
>>
> No, the exception should be re-thrown so that the test will fail.
>> Cheers
>> --
>> Manik Surtani
>> manik(a)jboss.org
>> Lead, Infinispan
>> Lead, JBoss Cache
>>
http://www.infinispan.org
>>
http://www.jbosscache.org
>>
>>
>>
>>
>> _______________________________________________
>> infinispan-dev mailing list
>> infinispan-dev(a)lists.jboss.org
>>
https://lists.jboss.org/mailman/listinfo/infinispan-dev
>>
>
--
Manik Surtani
manik(a)jboss.org
Lead, Infinispan
Lead, JBoss Cache
http://www.infinispan.org
http://www.jbosscache.org