<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Oct 19, 2009, at 7:22 PM, Manik Surtani wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>Please be careful about this - I'm sure the dependsOnMethods field was <br>set for a reason. Mircea, care to comment?</div></blockquote><blockquote type="cite"><div><font class="Apple-style-span" color="#000000"><br></font></div></blockquote>All fine by me as long as the tests run. Test NG is not very well 'documented' for our use case, so there are several workarounds in use cases, like this one. <br><br><blockquote type="cite"><div>On 19 Oct 2009, at 16:27, Vladimir Blagojevic wrote:<br><br><blockquote type="cite">On 09-10-19 10:17 AM, Galder Zamarreno wrote:<br></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">On 10/19/2009 03:46 PM, Vladimir Blagojevic wrote:<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">You gotta be kidding! Are you sure that your method is annotated <br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">with<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">@AfterClass and not @AfterTest. Is this in svn? I'd like to try it <br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">myself!<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">I'm not kidding and I'm sure :)<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">I've just committed it, r969. Do an svn updated, revert to the<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">AbstractInfinispanTest.nullifyInstanceFields and run<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">org.infinispan.jndi.BindingTest and you'll see an NPE being thrown in<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">the preDestroy() method.<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite">destroyAfterClass in SingleCacheManagerTest with its dependsOnMethods<br></blockquote><blockquote type="cite">annotation field is throwing the order off. If you remove<br></blockquote><blockquote type="cite">dependsOnMethods field and remove preDestroy method in<br></blockquote><blockquote type="cite">SingleCacheManagerTest everything works as expected. Try it yourself.<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">These dependsOnMethods annotation fields complicate things for no<br></blockquote><blockquote type="cite">reason. If we do not use them the order of invocation of @AfterClass<br></blockquote><blockquote type="cite">methods is always from the leaf all the way up to ancestor.<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">_______________________________________________<br></blockquote><blockquote type="cite">infinispan-dev mailing list<br></blockquote><blockquote type="cite"><a href="mailto:infinispan-dev@lists.jboss.org">infinispan-dev@lists.jboss.org</a><br></blockquote><blockquote type="cite"><a href="https://lists.jboss.org/mailman/listinfo/infinispan-dev">https://lists.jboss.org/mailman/listinfo/infinispan-dev</a><br></blockquote><br>--<br>Manik Surtani<br><a href="mailto:manik@jboss.org">manik@jboss.org</a><br>Lead, Infinispan<br>Lead, JBoss Cache<br>http://www.infinispan.org<br>http://www.jbosscache.org<br><br><br><br><br>_______________________________________________<br>infinispan-dev mailing list<br>infinispan-dev@lists.jboss.org<br>https://lists.jboss.org/mailman/listinfo/infinispan-dev<br></div></blockquote></div><br></body></html>