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 <vvolansk@redhat.com> --- 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 <vvolansk@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk@redhat.com> Gerrit-Reviewer: automation@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server