Allon Mureinik has submitted this change and it was merged.
Change subject: core: Fix broken ClusterValidatorTest
......................................................................
core: Fix broken ClusterValidatorTest
Patch 29d8a1c17d83ae4b79698e012e8e68d16733fa2a intoruced injecting
CpuFlagsManagerHandler in ClusterValidator, but failed to mock it
properly in the test, thus breaking it.
This patch introduces an InjectorRule to the test to fix it.
Change-Id: Ib55e805bb60094d037190db0c5907b41a10b4cfe
Signed-off-by: Allon Mureinik <amureini(a)redhat.com>
---
M
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/ClusterValidatorTest.java
1 file changed, 5 insertions(+), 0 deletions(-)
Approvals:
Allon Mureinik: Verified; Passed CI tests
Omer Frenkel: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/44414
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib55e805bb60094d037190db0c5907b41a10b4cfe
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org