cbeust/testng — pull request #328 by beverage (2013-01-22 18:51:45+00:00)
Title: Attempt to fix unnecessary execution of @Factory-ctors

(Ignore the build.xml change - it was a fix I had to do to get things working in my local environment that shouldn't of been pushed.)

Here's the change I was suggesting here: https://groups.google.com/forum/?fromgroups=#!topic/testng-dev/3VjYKh6xXZw

I don't think this is an acceptable change now though as it will result in double execution of default constructors. While I think in a perfect world that would be fine (nobody should be doing anything expensive in default constructors anyways - they should have things like that in TestNG annotated configuration methods) it could possibly have horrible side effects for some users who are doing expensive initialization there.

What are your thoughts on this? Alternatively, we could maintain a test class cache from which we could pull objects from once they are initialized - something like that would a bit more complicated though.