commit e057b41a17ad7cdd8d0e4a31c332a92dfdd611e4 Author: Fedor Korotkov Date: Thu May 28 10:13:50 2015 -0700 Moved creation of per test data to testStarted method. During an upgrade of Pants at Twitter I found out that some of our tests are using org.powermock.modules.junit4.PowerMockRunner and it doesn't call fireTestRunStarted at all. We were getting NPEs because of that. According to the docs testRunStarted and testStarted have pretty much the same meaning except testStarted is invoked just before an execution in the same thread. Testing Done: Build a jar and used it with out tests and they passed. Bugs closed: 1234, 1588 Reviewed at https://rbcommons.com/s/twitter/r/2257/ .../tools/junit/AntJunitXmlReportListener.java | 9 +++------ src/java/org/pantsbuild/tools/junit/ConsoleRunner.java | 17 +++-------------- 2 files changed, 6 insertions(+), 20 deletions(-)