Change in ovirt-engine[master]: core: Fix CreateAllSnapshotsFromVmCommandTest

oourfali at redhat.com oourfali at redhat.com
Tue Aug 20 08:07:21 UTC 2013


Oved Ourfali has submitted this change and it was merged.

Change subject: core: Fix CreateAllSnapshotsFromVmCommandTest
......................................................................


core: Fix CreateAllSnapshotsFromVmCommandTest

Fix test-break caused by commit
665c1c9e243eaa6ff16cbcdc88828b3645009f8a.

The aforementioned patch moves the verification if a VM is (not)
external from VmHandler to the VM entity itself, and was thus no
longer mocked by the test, causing the failure.

This patch adds such mocking, to allow the test to pass.

Change-Id: I298d3bd0c05f3175371806dec60c7dccb798d165
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommandTest.java
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Allon Mureinik: Verified
  Oved Ourfali: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/18302
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I298d3bd0c05f3175371806dec60c7dccb798d165
Gerrit-PatchSet: 1
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: Oved Ourfali <oourfali at redhat.com>



More information about the Engine-commits mailing list