Change in ovirt-engine[master]: core: RefreshGlusterVolumeDetailsCommandTest syncJob

amureini at redhat.com amureini at redhat.com
Wed Sep 28 22:55:28 UTC 2016


Allon Mureinik has submitted this change and it was merged.

Change subject: core: RefreshGlusterVolumeDetailsCommandTest syncJob
......................................................................


core: RefreshGlusterVolumeDetailsCommandTest syncJob

The member syncJob is mocked and then spied, overriding the mocking.

This patch removes the useless spying (and the injection of the
AuditLogDirector required because of it), as this test is for
RefreshGlusterVolumeDetailsCommand, not the GlusterSyncJob.

Change-Id: Icb08c703ff78f383d10f9a0b2539da3858a2af44
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/RefreshGlusterVolumeDetailsCommandTest.java
1 file changed, 0 insertions(+), 3 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Allon Mureinik: Verified; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/64609
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icb08c703ff78f383d10f9a0b2539da3858a2af44
Gerrit-PatchSet: 3
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: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list