fixed #1464 - change so persistence dir is only backed up if node is HA ...

...master or HA disabled; handles the case of HA autodetect selecting master hence the complexity.

also fixes some leaks in test and improves atomic file update support in FileBasedObjectStore.
(incorporates new logging from #1469)
Is testng not reporting this kind of thing for us? If we can't trust messages from `assertTrue` getting reported properly then that impacts a lot of tests!