Mocking out the LeadControllerManager in test
* Mocking out the LeadControllerManager in test
We should not need the real one, so it is better to mock the class altogether
* Addressing review comment
@@ -1321,8 +1321,7 @@ public class PinotLLCRealtimeSegmentManagerTest {
protected FakePinotLLCRealtimeSegmentManager(PinotHelixResourceManager pinotHelixResourceManager,
List<String> existingLLCSegments, ControllerMetrics controllerMetrics) {
- super(pinotHelixResourceManager, CONTROLLER_CONF, controllerMetrics,
- new LeadControllerManager(pinotHelixResourceManager.getHelixZkManager(), controllerMetrics));
+ super(pinotHelixResourceManager, CONTROLLER_CONF, controllerMetrics, mock(LeadControllerManager.class));
try {
TableConfigCache mockCache = mock(TableConfigCache.class);
