Minor preference: I like to not initialise fields if setUp is going to do it. Doing it twice lets one get away with accidents like the code being deleted from setUp, which then leads to non-deterministic failures when testng decides to run the tests in a different order.