Allon Mureinik has uploaded a new change for review. Change subject: core: UnPowerMock VdsUpdateRunTimeInfoTest ......................................................................
core: UnPowerMock VdsUpdateRunTimeInfoTest Remover PowerMock depedency from VdsUpdateRunTimeInfoTest. In order to acheive this improvement, some refactoring was required in EjbUtils and VdsUpdateRunTimeInfo intself. EjbUtils is mocked by setting the inner strategy object and returning it back to the original obejct after the test is done. If this method is deemed acceptable, perhaps it would be a good idea to extrat it to its own @Rule. Change-Id: I5519a02c197b0ef09a9cabd37e6fa90dbecc0897 Signed-off-by: Allon Mureinik <[email protected]> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ejb/EjbUtils.java M backend/manager/modules/vdsbroker/pom.xml M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsUpdateRunTimeInfoTest.java 4 files changed, 116 insertions(+), 70 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/94/5194/1 -- To view, visit http://gerrit.ovirt.org/5194 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I5519a02c197b0ef09a9cabd37e6fa90dbecc0897 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
