Change in ovirt-engine[master]: core: ImportVmCommandStub ImportValidator

amureini at redhat.com amureini at redhat.com
Thu Oct 13 20:50:21 UTC 2016


Allon Mureinik has submitted this change and it was merged.

Change subject: core: ImportVmCommandStub ImportValidator
......................................................................


core: ImportVmCommandStub ImportValidator

The validator can easily be spied onto the command, so there's no
reason to reimplement the same logic explicitly by saving it and then
checking if it was saved.

Change-Id: I722c75bf29c4b1c7bb7434083dd50c2fdadb7276
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommandTest.java
1 file changed, 2 insertions(+), 14 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Allon Mureinik: Verified; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/64675
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I722c75bf29c4b1c7bb7434083dd50c2fdadb7276
Gerrit-PatchSet: 11
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: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list