Allon Mureinik has submitted this change and it was merged. Change subject: core: Removed redundant storage allocation checks ......................................................................
core: Removed redundant storage allocation checks AddVmCommandTest contains storage allocation checks. StorageDomainValidator already has the necessary tests, so there's no need for this redundancy. Therefore removing storage tests from AddVmCommandTest. Change-Id: Ia0e28f6cc8bd2834d6fe6ce6288a20946a69e9b3 Related-to: https://bugzilla.redhat.com/show_bug.cgi?id=917682 Related-to: https://bugzilla.redhat.com/show_bug.cgi?id=1053742 Signed-off-by: Vered Volansky <[email protected]> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmCommandTest.java 1 file changed, 0 insertions(+), 78 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved Arik Hadas: Looks good to me, but someone else must approve Vered Volansky: Verified -- To view, visit http://gerrit.ovirt.org/26732 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia0e28f6cc8bd2834d6fe6ce6288a20946a69e9b3 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
