X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=openflowplugin-impl%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fopenflowplugin%2Fimpl%2Fstatistics%2FStatisticsManagerImplTest.java;h=77a3eb04e3cbf2795844245954f6e1e960b73805;hb=0ac8877c3539d82e55ffe49dc0a2c8bcb0c82b19;hp=ec9270f705e081f00c79c59dfd81b1b654f9f444;hpb=a7a8c6529cfe80c9280e0336b6372138fe307b68;p=openflowplugin.git diff --git a/openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/statistics/StatisticsManagerImplTest.java b/openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/statistics/StatisticsManagerImplTest.java index ec9270f705..77a3eb04e3 100644 --- a/openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/statistics/StatisticsManagerImplTest.java +++ b/openflowplugin-impl/src/test/java/org/opendaylight/openflowplugin/impl/statistics/StatisticsManagerImplTest.java @@ -7,14 +7,10 @@ */ package org.opendaylight.openflowplugin.impl.statistics; -import static org.mockito.Mockito.times; -import static org.mockito.Mockito.verify; -import static org.mockito.Mockito.when; +import static org.mockito.Mockito.*; import com.google.common.util.concurrent.FutureCallback; import com.google.common.util.concurrent.Futures; -import io.netty.util.HashedWheelTimer; -import io.netty.util.Timeout; import java.lang.reflect.Field; import java.math.BigInteger; import java.util.Collections; @@ -78,6 +74,8 @@ import org.opendaylight.yangtools.yang.common.RpcResult; import org.opendaylight.yangtools.yang.common.RpcResultBuilder; import org.slf4j.Logger; import org.slf4j.LoggerFactory; +import io.netty.util.HashedWheelTimer; +import io.netty.util.Timeout; @RunWith(MockitoJUnitRunner.class) @@ -85,8 +83,6 @@ public class StatisticsManagerImplTest { private static final Logger LOG = LoggerFactory.getLogger(StatisticsManagerImplTest.class); - private static final BigInteger DUMMY_DATAPATH_ID = new BigInteger("444"); - private static final Short DUMMY_VERSION = OFConstants.OFP_VERSION_1_3; public static final NodeId NODE_ID = new NodeId("ofp-unit-dummy-node-id"); @Mock @@ -167,7 +163,7 @@ public class StatisticsManagerImplTest { when(mockedDeviceContext.getDeviceInfo()).thenReturn(mockedDeviceInfo); when(mockedDeviceContext.getPrimaryConnectionContext()).thenReturn(mockedPrimConnectionContext); when(mockedDeviceContext.getMessageSpy()).thenReturn(mockedMessagSpy); - when(mockedDeviceContext.getDeviceFlowRegistry()).thenReturn(new DeviceFlowRegistryImpl(dataBroker, nodePath)); + when(mockedDeviceContext.getDeviceFlowRegistry()).thenReturn(new DeviceFlowRegistryImpl(OFConstants.OFP_VERSION_1_3, dataBroker, nodePath)); when(mockedDeviceContext.getDeviceState()).thenReturn(mockedDeviceState); when(mockedDeviceContext.getMultiMsgCollector( Matchers.>>any())).thenAnswer( @@ -185,7 +181,10 @@ public class StatisticsManagerImplTest { Matchers.any())).thenReturn(serviceControlRegistration); final ConvertorManager convertorManager = ConvertorManagerFactory.createDefaultManager(); - statisticsManager = new StatisticsManagerImpl(rpcProviderRegistry, false, new HashedWheelTimer(), convertorManager); + final long basicTimerDelay = 3000L; + final long maximumTimerDelay = 900000L; + statisticsManager = new StatisticsManagerImpl(rpcProviderRegistry, false, new HashedWheelTimer(), + convertorManager, basicTimerDelay, maximumTimerDelay); statisticsManager.setDeviceInitializationPhaseHandler(deviceInitializationPhaseHandler); } @@ -223,7 +222,7 @@ public class StatisticsManagerImplTest { statisticsManager.onDeviceContextLevelDown(deviceInfo); verify(statisticContext).close(); verify(mockedTerminationPhaseHandler).onDeviceContextLevelDown(deviceInfo); - Assert.assertEquals(0, contextsMap.size()); + Assert.assertEquals(1, contextsMap.size()); } private static Map getContextsMap(final StatisticsManagerImpl statisticsManager) @@ -257,7 +256,8 @@ public class StatisticsManagerImplTest { when(itemLifeCycleRegistry.getLifeCycleSources()).thenReturn( Collections.emptyList()); - when(statisticContext.getLifecycleService()).thenReturn(lifecycleService); + when(statisticContext.gainDeviceContext()).thenReturn(mockedDeviceContext); + when(statisticContext.gainDeviceState()).thenReturn(mockedDeviceState); when(lifecycleService.getDeviceContext()).thenReturn(mockedDeviceContext); getContextsMap(statisticsManager).put(deviceInfo, statisticContext); @@ -297,7 +297,8 @@ public class StatisticsManagerImplTest { getContextsMap(statisticsManager).put(deviceInfo, statisticContext); - when(statisticContext.getLifecycleService()).thenReturn(lifecycleService); + when(statisticContext.gainDeviceContext()).thenReturn(mockedDeviceContext); + when(statisticContext.gainDeviceState()).thenReturn(mockedDeviceState); when(lifecycleService.getDeviceContext()).thenReturn(mockedDeviceContext); final ChangeStatisticsWorkModeInputBuilder changeStatisticsWorkModeInputBld = @@ -336,7 +337,8 @@ public class StatisticsManagerImplTest { getContextsMap(statisticsManager).put(deviceInfo, statisticContext); - when(statisticContext.getLifecycleService()).thenReturn(lifecycleService); + when(statisticContext.gainDeviceContext()).thenReturn(mockedDeviceContext); + when(statisticContext.gainDeviceState()).thenReturn(mockedDeviceState); when(lifecycleService.getDeviceContext()).thenReturn(mockedDeviceContext); final ChangeStatisticsWorkModeInputBuilder changeStatisticsWorkModeInputBld = @@ -373,7 +375,6 @@ public class StatisticsManagerImplTest { public void testCalculateTimerDelay() throws Exception { final TimeCounter timeCounter = Mockito.mock(TimeCounter.class); when(timeCounter.getAverageTimeBetweenMarks()).thenReturn(2000L, (Long)4000L); - statisticsManager.calculateTimerDelay(timeCounter); Assert.assertEquals(3000L, StatisticsManagerImpl.getCurrentTimerDelay()); statisticsManager.calculateTimerDelay(timeCounter);