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@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@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com>