Change in ovirt-engine[master]: core: UnPowerMock MoveOrCopyDiskCommandTest

lhornyak at redhat.com lhornyak at redhat.com
Fri May 25 11:01:26 UTC 2012


Laszlo Hornyak has submitted this change and it was merged.

Change subject: core: UnPowerMock MoveOrCopyDiskCommandTest
......................................................................


core: UnPowerMock MoveOrCopyDiskCommandTest

Removed PowerMock dependency from MoveOrCopyDiskCommandTest.

In order to facilitate this change several protected methods were
created in the MoveOrCopyDiskCommand, in order to allow spying out the
dependency on StorageDomainChecker and ImagesHandler.

Change-Id: I75fa03b0bcb0229a662a72fd01902f6883ef8983
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommandTest.java
2 files changed, 48 insertions(+), 43 deletions(-)

Approvals:
  Laszlo Hornyak: Verified; Looks good to me, approved
  Allon Mureinik: Verified


--
To view, visit http://gerrit.ovirt.org/4771
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I75fa03b0bcb0229a662a72fd01902f6883ef8983
Gerrit-PatchSet: 2
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: Laszlo Hornyak <lhornyak at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list