Allon Mureinik has uploaded a new change for review. Change subject: core: UnPowerMock and clean pools with vm commands ......................................................................
core: UnPowerMock and clean pools with vm commands Removed PowerMock depedency and replaced it with directly spying the relevant methods in the tests of commands relevant to manipulating pools which contain VMs. Some refactoring in the commands themselves and their base classes was required in order to achieve this functionality - mainly extracting DAOs and other methods which access external resources to their own helper methods. Change-Id: I43cdf43682d3ab2a3e75ca965490982888b93763 Signed-off-by: Allon Mureinik <amure...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmPoolWithVmsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmPoolCommandBase.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmPoolWithVmsCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommandTestAbstract.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateVmPoolWithVmsCommandTest.java 6 files changed, 151 insertions(+), 191 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/17/5517/1 -- To view, visit http://gerrit.ovirt.org/5517 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I43cdf43682d3ab2a3e75ca965490982888b93763 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amure...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
