Change in ovirt-engine[master]: core: Don't @InjectMocks in MigrateVmCommandTest

Code Review gerrit at ovirt.org
Sun Mar 19 08:52:46 UTC 2017


>From Allon Mureinik <amureini at redhat.com>:

Allon Mureinik has submitted this change and it was merged.

Change subject: core: Don't @InjectMocks in MigrateVmCommandTest
......................................................................


core: Don't @InjectMocks in MigrateVmCommandTest

The MigrateVmCommand instance in MigrateVmCommandTest has nothing
that should be injected, so this annotation is useless at best or
plain wrong at worst.

This patch removes the aforementioned annotation.

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

Approvals:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Verified
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I253ca4dc54cd5d687e82be55949759abed2972fd
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: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list