[infinispan-dev] Change AbstractInfinispanTest.nullifyInstanceFields method name to zzzNullifyInstanceFields
Galder Zamarreno
galder at redhat.com
Mon Oct 19 07:55:32 EDT 2009
On 10/19/2009 01:52 PM, Galder Zamarreno wrote:
> Hi,
>
> I'm trying to modify SingleCacheManagerTest so that in the same way that
> there's a preCreate() callback, there can be a preDestroy() callback
> too. This is so that for example, in the
> https://jira.jboss.org/jira/browse/ISPN-153 test, I can stop the naming
> server created at in preCreate().
>
> The problem though is that by the time either
> SingleCacheManagerTest.destroyAfterClass is called,
> AbstractInfinispanTest.nullifyInstanceFields has already run and I get NPEs.
Sorry, here meant: before SingleCacheManagerTest.preDestroy is called,
AbstractInfinispanTest.nullifyInstanceFields has already run.
>
> However, I see other cases such as
> BdbjeCacheStoreFunctionalIntegrationTest where clearTempDir is called
> before AbstractInfinispanTest.nullifyInstanceFields.
>
> I just run a test without the preDestroy() in SingleCacheManagerTest so
> that there was only nullifyInstanceFields and destroyAfterClass
> @AfterClass methods to called and destroyAfterClass was called first.
> So, there does appear to be some kind alphabetical order in which testng
> calls its @AfterClass methods.
>
> So, I've just tried to change
> AbstractInfinispanTest.nullifyInstanceFields method name to
> AbstractInfinispanTest.zzzNullifyInstanceFields and now the preDestroy()
> gets called first.
>
> Any objections to this change?
>
> The other solution would to have some kind of dependsOnMethods in
> AbstractInfinispanTest but that would force specific naming on
> subclasses. Changing the name in AbstractInfinispanTest seems like a
> much easier option.
--
Galder Zamarreño
Sr. Software Engineer
Infinispan, JBoss Cache
More information about the infinispan-dev
mailing list