Michael Kublin has submitted this change and it was merged. Change subject: core: Improve AutoRecoveryManagerTest ...................................................................... core: Improve AutoRecoveryManagerTest Before this patch, the test ran the OnTime() method and checked that no exception was thrown. This patch extends it functionality and used Mockito's verification capabilities to assert that the correct recovery actions are called. Change-Id: I5410ad3b5aa383ed0145b0b03ec1b86ef63fb011 Signed-off-by: Allon Mureinik <amureini@redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AutoRecoveryManagerTest.java 1 file changed, 27 insertions(+), 4 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6864 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5410ad3b5aa383ed0145b0b03ec1b86ef63fb011 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Ayal Baron <abaron@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin@redhat.com>