[infinispan-dev] Change AbstractInfinispanTest.nullifyInstanceFields method name to zzzNullifyInstanceFields

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. 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