I like to always include `(alwaysRun=true)`. Otherwise when someone marks one of the tests as `groups="Integration"` it will work in Eclipse but fail in mvn because setUp won't be called for it.
But this is unconnected to your pull request.