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