Allon Mureinik has submitted this change and it was merged. Change subject: core: ImagesHandler.getAllStorageIdsForImageIds ......................................................................
core: ImagesHandler.getAllStorageIdsForImageIds Extracted a method to get all the relevant storage domains IDs from a collection of DiskImages, and added some tests to it. This method will be used in subsequent patches in the effort to decouple ImagesHandler.PerformImagesChecks, but in the meanwhile it's main benefit is the added unit test coverage. Change-Id: If1b4d7231a03ea5a6d991609a65951fe4e279ef5 Signed-off-by: Allon Mureinik <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImagesHandler.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/ImagesHandlerTest.java 2 files changed, 47 insertions(+), 13 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12131 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If1b4d7231a03ea5a6d991609a65951fe4e279ef5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> Gerrit-Reviewer: Alissa Bonas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
