Moti Asayag has submitted this change and it was merged.
Change subject: engine: fix mocking in QueriesCommandBaseTest
......................................................................
engine: fix mocking in QueriesCommandBaseTest
Add proper mocking for QueriesCommandBase usage of SessionDataContainer.
That fixes NPEs in the test.
Change-Id: I1378780f6ef666000e837a135fbafecc82f661cd
Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/QueriesCommandBase.java
M
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/QueriesCommandBaseTest.java
2 files changed, 27 insertions(+), 13 deletions(-)
Approvals:
Martin Mucha: Looks good to me, but someone else must approve
Moti Asayag: Looks good to me, approved
Yevgeny Zaspitsky: Verified
Objections:
oVirt Jenkins CI Server: Fails
--
To view, visit
https://gerrit.ovirt.org/39603
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1378780f6ef666000e837a135fbafecc82f661cd
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server