Change in ovirt-engine[master]: core: Improve AutoRecoveryManagerTest

mkublin at redhat.com mkublin at redhat.com
Sun Aug 5 13:47:06 UTC 2012


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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>



More information about the Engine-commits mailing list