Change in ovirt-engine[master]: engine: Fix junit failing on GlusterSyncJob instance

sabose at redhat.com sabose at redhat.com
Mon Jun 29 15:27:31 UTC 2015


Sahina Bose has submitted this change and it was merged.

Change subject: engine: Fix junit failing on GlusterSyncJob instance
......................................................................


engine: Fix junit failing on GlusterSyncJob instance

Junits are failing with unable to find instance for
GlusterSyncJob due to DI changes introduced.

Added mocks for Injector and corrected the
DI for AuditLogDirector

Change-Id: I4bf2e25c440663b87b7fe06e727e57c3500e6702
Signed-off-by: Sahina Bose <sabose at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJob.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJobTest.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/RefreshGlusterVolumeDetailsCommandTest.java
3 files changed, 23 insertions(+), 4 deletions(-)

Approvals:
  Martin Peřina: Verified; Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Sahina Bose: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4bf2e25c440663b87b7fe06e727e57c3500e6702
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list