ISPN-4631 Make sure test servers are shut down before/after tests

https://issues.jboss.org/browse/ISPN-4631
I think this should be in an `@AfterClass` method, because `stopJoiningNodes` won't run if one of the previous tests fails. Then you shouldn't need the other `controller.kill()` calls in `startNodes`.