Change in ovirt-engine[master]: core: Fix broken ClusterValidatorTest

amureini at redhat.com amureini at redhat.com
Wed Aug 5 12:32:14 UTC 2015


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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list