Bug 5596 Cleaning part 1
[openflowplugin.git] / openflowplugin-impl / src / test / java / org / opendaylight / openflowplugin / impl / statistics / StatisticsManagerImplTest.java
index 61728409e26b737a31dbd3bfcc48aab74558174f..68ed88b8d2d9032ff734957cf9954845e2e8341f 100644 (file)
@@ -1,36 +1,94 @@
+/**
+ * Copyright (c) 2015 Cisco Systems, Inc. and others.  All rights reserved.
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the Eclipse Public License v1.0 which accompanies this distribution,
+ * and is available at http://www.eclipse.org/legal/epl-v10.html
+ */
 package org.opendaylight.openflowplugin.impl.statistics;
 
-import static org.mockito.Matchers.eq;
-import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
+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 junit.framework.TestCase;
+import java.util.Collections;
+import java.util.List;
+import java.util.Map;
+import java.util.Optional;
+import java.util.concurrent.ExecutionException;
+import java.util.concurrent.Future;
+import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
+import org.mockito.ArgumentCaptor;
+import org.mockito.Captor;
+import org.mockito.Matchers;
 import org.mockito.Mock;
+import org.mockito.Mockito;
+import org.mockito.invocation.InvocationOnMock;
 import org.mockito.runners.MockitoJUnitRunner;
+import org.mockito.stubbing.Answer;
+import org.opendaylight.controller.md.sal.binding.api.DataBroker;
+import org.opendaylight.controller.sal.binding.api.BindingAwareBroker;
 import org.opendaylight.controller.sal.binding.api.RpcProviderRegistry;
 import org.opendaylight.openflowjava.protocol.api.connection.ConnectionAdapter;
+import org.opendaylight.openflowjava.protocol.api.connection.OutboundQueue;
 import org.opendaylight.openflowplugin.api.OFConstants;
 import org.opendaylight.openflowplugin.api.openflow.connection.ConnectionContext;
 import org.opendaylight.openflowplugin.api.openflow.device.DeviceContext;
+import org.opendaylight.openflowplugin.api.openflow.device.DeviceInfo;
+import org.opendaylight.openflowplugin.api.openflow.device.DeviceManager;
 import org.opendaylight.openflowplugin.api.openflow.device.DeviceState;
+import org.opendaylight.openflowplugin.api.openflow.device.RequestContext;
 import org.opendaylight.openflowplugin.api.openflow.device.RequestContextStack;
 import org.opendaylight.openflowplugin.api.openflow.device.handlers.DeviceInitializationPhaseHandler;
+import org.opendaylight.openflowplugin.api.openflow.device.handlers.DeviceTerminationPhaseHandler;
+import org.opendaylight.openflowplugin.api.openflow.device.handlers.MultiMsgCollector;
+import org.opendaylight.openflowplugin.api.openflow.lifecycle.LifecycleConductor;
+import org.opendaylight.openflowplugin.api.openflow.lifecycle.LifecycleService;
+import org.opendaylight.openflowplugin.api.openflow.registry.ItemLifeCycleRegistry;
+import org.opendaylight.openflowplugin.api.openflow.rpc.ItemLifeCycleSource;
+import org.opendaylight.openflowplugin.api.openflow.rpc.listener.ItemLifecycleListener;
+import org.opendaylight.openflowplugin.api.openflow.statistics.StatisticsContext;
 import org.opendaylight.openflowplugin.api.openflow.statistics.ofpspecific.MessageSpy;
 import org.opendaylight.openflowplugin.impl.registry.flow.DeviceFlowRegistryImpl;
+import org.opendaylight.openflowplugin.openflow.md.core.sal.convertor.ConvertorManager;
+import org.opendaylight.openflowplugin.openflow.md.core.sal.convertor.ConvertorManagerFactory;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeId;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.Nodes;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.Node;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.NodeKey;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.FeaturesReply;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.GetFeaturesOutput;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.MultipartReply;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.MultipartRequestInput;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.OfHeader;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.openflowplugin.sm.control.rev150812.ChangeStatisticsWorkModeInputBuilder;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.openflowplugin.sm.control.rev150812.GetStatisticsWorkModeOutput;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.openflowplugin.sm.control.rev150812.StatisticsManagerControlService;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.openflowplugin.sm.control.rev150812.StatisticsWorkMode;
+import org.opendaylight.yangtools.yang.binding.KeyedInstanceIdentifier;
+import org.opendaylight.yangtools.yang.common.RpcResult;
+import org.opendaylight.yangtools.yang.common.RpcResultBuilder;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 
 @RunWith(MockitoJUnitRunner.class)
-public class StatisticsManagerImplTest extends TestCase {
+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
     RequestContextStack mockedRequestContextStack;
@@ -39,6 +97,8 @@ public class StatisticsManagerImplTest extends TestCase {
     @Mock
     FeaturesReply mockedFeatures;
     @Mock
+    GetFeaturesOutput mockedFeaturesOutput;
+    @Mock
     ConnectionAdapter mockedConnectionAdapter;
     @Mock
     MessageSpy mockedMessagSpy;
@@ -47,33 +107,292 @@ public class StatisticsManagerImplTest extends TestCase {
     @Mock
     DeviceState mockedDeviceState;
     @Mock
+    DeviceInfo mockedDeviceInfo;
+    @Mock
     DeviceInitializationPhaseHandler mockedDevicePhaseHandler;
     @Mock
+    DeviceTerminationPhaseHandler mockedTerminationPhaseHandler;
+    @Mock
     private RpcProviderRegistry rpcProviderRegistry;
+    @Mock
+    private HashedWheelTimer hashedWheelTimer;
+    @Mock
+    private OutboundQueue outboundQueue;
+    @Mock
+    private MultiMsgCollector multiMagCollector;
+    @Mock
+    private ItemLifeCycleRegistry itemLifeCycleRegistry;
+    @Captor
+    private ArgumentCaptor<ItemLifecycleListener> itemLifeCycleListenerCapt;
+    @Mock
+    private BindingAwareBroker.RpcRegistration<StatisticsManagerControlService> serviceControlRegistration;
+    @Mock
+    private DeviceManager deviceManager;
+    @Mock
+    private LifecycleConductor conductor;
+    @Mock
+    private GetFeaturesOutput featuresOutput;
+    @Mock
+    private DeviceInitializationPhaseHandler deviceInitializationPhaseHandler;
+    @Mock
+    private DeviceInfo deviceInfo;
+    @Mock
+    private DataBroker dataBroker;
+    @Mock
+    private LifecycleService lifecycleService;
+
+    private RequestContext<List<MultipartReply>> currentRequestContext;
+    private StatisticsManagerImpl statisticsManager;
+
 
     @Before
     public void initialization() {
-        when(mockedFeatures.getDatapathId()).thenReturn(DUMMY_DATAPATH_ID);
-        when(mockedFeatures.getVersion()).thenReturn(DUMMY_VERSION);
+        final KeyedInstanceIdentifier<Node, NodeKey> nodePath = KeyedInstanceIdentifier
+                .create(Nodes.class)
+                .child(Node.class, new NodeKey(new NodeId("openflow:10")));
 
         when(mockedPrimConnectionContext.getFeatures()).thenReturn(mockedFeatures);
         when(mockedPrimConnectionContext.getConnectionAdapter()).thenReturn(mockedConnectionAdapter);
         when(mockedPrimConnectionContext.getConnectionState()).thenReturn(ConnectionContext.CONNECTION_STATE.WORKING);
+        when(mockedPrimConnectionContext.getNodeId()).thenReturn(NODE_ID);
+        when(mockedPrimConnectionContext.getOutboundQueueProvider()).thenReturn(outboundQueue);
 
+        when(mockedDeviceState.isFlowStatisticsAvailable()).thenReturn(Boolean.TRUE);
+        when(mockedDeviceState.isGroupAvailable()).thenReturn(Boolean.TRUE);
+        when(mockedDeviceState.isMetersAvailable()).thenReturn(Boolean.TRUE);
+        when(mockedDeviceState.isPortStatisticsAvailable()).thenReturn(Boolean.TRUE);
+        when(mockedDeviceState.isQueueStatisticsAvailable()).thenReturn(Boolean.TRUE);
+        when(mockedDeviceState.isTableStatisticsAvailable()).thenReturn(Boolean.TRUE);
+        when(mockedDeviceInfo.getNodeInstanceIdentifier()).thenReturn(nodePath);
+        when(mockedDeviceInfo.getDatapathId()).thenReturn(BigInteger.TEN);
+        when(mockedDeviceInfo.getNodeId()).thenReturn(new NodeId("ofp-unit-dummy-node-id"));
+
+        when(mockedDeviceContext.getDeviceInfo()).thenReturn(mockedDeviceInfo);
         when(mockedDeviceContext.getPrimaryConnectionContext()).thenReturn(mockedPrimConnectionContext);
         when(mockedDeviceContext.getMessageSpy()).thenReturn(mockedMessagSpy);
-        when(mockedDeviceContext.getDeviceFlowRegistry()).thenReturn(new DeviceFlowRegistryImpl());
+        when(mockedDeviceContext.getDeviceFlowRegistry()).thenReturn(new DeviceFlowRegistryImpl(dataBroker, nodePath));
         when(mockedDeviceContext.getDeviceState()).thenReturn(mockedDeviceState);
-        when(mockedDeviceContext.getTimer()).thenReturn(mock(HashedWheelTimer.class));
+        when(mockedDeviceContext.getMultiMsgCollector(
+                Matchers.<RequestContext<List<MultipartReply>>>any())).thenAnswer(
+                new Answer<MultiMsgCollector>() {
+                    @Override
+                    public MultiMsgCollector answer(InvocationOnMock invocation) throws Throwable {
+                        currentRequestContext = (RequestContext<List<MultipartReply>>) invocation.getArguments()[0];
+                        return multiMagCollector;
+                    }
+                }
+        );
+        when(mockedDeviceContext.getItemLifeCycleSourceRegistry()).thenReturn(itemLifeCycleRegistry);
+        when(rpcProviderRegistry.addRpcImplementation(
+                Matchers.eq(StatisticsManagerControlService.class),
+                Matchers.<StatisticsManagerControlService>any())).thenReturn(serviceControlRegistration);
+
+        final ConvertorManager convertorManager = ConvertorManagerFactory.createDefaultManager();
+        statisticsManager = new StatisticsManagerImpl(rpcProviderRegistry, false, conductor, convertorManager);
+        statisticsManager.setDeviceInitializationPhaseHandler(deviceInitializationPhaseHandler);
+        when(conductor.getDeviceContext(deviceInfo)).thenReturn(mockedDeviceContext);
     }
 
     @Test
     public void testOnDeviceContextLevelUp() throws Exception {
-        final StatisticsManagerImpl statisticsManager = new StatisticsManagerImpl(rpcProviderRegistry);
+        Mockito.doAnswer(new Answer<Void>() {
+            @Override
+            public Void answer(InvocationOnMock invocation) throws Throwable {
+                final FutureCallback<OfHeader> callback = (FutureCallback<OfHeader>) invocation.getArguments()[2];
+                LOG.debug("committing entry: {}", ((MultipartRequestInput) invocation.getArguments()[1]).getType());
+                callback.onSuccess(null);
+                currentRequestContext.setResult(RpcResultBuilder.<List<MultipartReply>>success().build());
+                return null;
+            }
+        }).when(outboundQueue)
+                .commitEntry(Matchers.anyLong(), Matchers.<OfHeader>any(), Matchers.<FutureCallback<OfHeader>>any());
+
         statisticsManager.setDeviceInitializationPhaseHandler(mockedDevicePhaseHandler);
-        statisticsManager.onDeviceContextLevelUp(mockedDeviceContext);
-        verify(mockedDeviceState).setDeviceSynchronized(eq(true));
+        statisticsManager.onDeviceContextLevelUp(deviceInfo, lifecycleService);
+        verify(mockedDevicePhaseHandler).onDeviceContextLevelUp(deviceInfo, lifecycleService);
+    }
+
+    @Test
+    public void testOnDeviceContextClosed() throws Exception {
+        final StatisticsContext statisticContext = Mockito.mock(StatisticsContext.class);
+        final Map<DeviceInfo, StatisticsContext> contextsMap = getContextsMap(statisticsManager);
+
+        contextsMap.put(deviceInfo, statisticContext);
+        Assert.assertEquals(1, contextsMap.size());
+
+        statisticsManager.setDeviceTerminationPhaseHandler(mockedTerminationPhaseHandler);
+        statisticsManager.onDeviceContextLevelDown(deviceInfo);
+        verify(statisticContext).close();
+        verify(mockedTerminationPhaseHandler).onDeviceContextLevelDown(deviceInfo);
+        Assert.assertEquals(0, contextsMap.size());
+    }
+
+    private static Map<DeviceInfo, StatisticsContext> getContextsMap(final StatisticsManagerImpl statisticsManager)
+            throws NoSuchFieldException, IllegalAccessException {
+        // HACK: contexts map for testing shall be accessed in some more civilized way
+        final Field contextsField = StatisticsManagerImpl.class.getDeclaredField("contexts");
+        Assert.assertNotNull(contextsField);
+        contextsField.setAccessible(true);
+        return (Map<DeviceInfo, StatisticsContext>) contextsField.get(statisticsManager);
+    }
+
+    @Test
+    public void testGetStatisticsWorkMode() throws Exception {
+        final Future<RpcResult<GetStatisticsWorkModeOutput>> workMode = statisticsManager.getStatisticsWorkMode();
+        Assert.assertTrue(workMode.isDone());
+        Assert.assertTrue(workMode.get().isSuccessful());
+        Assert.assertNotNull(workMode.get().getResult());
+        Assert.assertEquals(StatisticsWorkMode.COLLECTALL, workMode.get().getResult().getMode());
+    }
+
+    /**
+     * switching to {@link StatisticsWorkMode#FULLYDISABLED}; no pollTimeout and no lifecycleRegistry
+     *
+     * @throws Exception
+     */
+    @Test
+    public void testChangeStatisticsWorkMode1() throws Exception {
+        final StatisticsContext statisticContext = Mockito.mock(StatisticsContext.class);
+        when(statisticContext.getPollTimeout()).thenReturn(
+                Optional.<Timeout>empty());
+        when(itemLifeCycleRegistry.getLifeCycleSources()).thenReturn(
+                Collections.<ItemLifeCycleSource>emptyList());
+
+        getContextsMap(statisticsManager).put(deviceInfo, statisticContext);
+
+        final ChangeStatisticsWorkModeInputBuilder changeStatisticsWorkModeInputBld =
+                new ChangeStatisticsWorkModeInputBuilder()
+                        .setMode(StatisticsWorkMode.FULLYDISABLED);
+
+        final Future<RpcResult<Void>> workMode = statisticsManager
+                .changeStatisticsWorkMode(changeStatisticsWorkModeInputBld.build());
+
+        checkWorkModeChangeOutcome(workMode);
+        Mockito.verify(itemLifeCycleRegistry).getLifeCycleSources();
+        Mockito.verify(statisticContext).getPollTimeout();
+    }
+
+    private static void checkWorkModeChangeOutcome(Future<RpcResult<Void>> workMode) throws InterruptedException, ExecutionException {
+        Assert.assertTrue(workMode.isDone());
+        Assert.assertTrue(workMode.get().isSuccessful());
+    }
+
+
+    /**
+     * switching to {@link StatisticsWorkMode#FULLYDISABLED}; with pollTimeout and lifecycleRegistry
+     *
+     * @throws Exception
+     */
+    @Test
+    public void testChangeStatisticsWorkMode2() throws Exception {
+        final Timeout pollTimeout = Mockito.mock(Timeout.class);
+        final ItemLifeCycleSource itemLifecycleSource = Mockito.mock(ItemLifeCycleSource.class);
+        final StatisticsContext statisticContext = Mockito.mock(StatisticsContext.class);
+        when(statisticContext.getPollTimeout()).thenReturn(
+                Optional.of(pollTimeout));
+        when(itemLifeCycleRegistry.getLifeCycleSources()).thenReturn(
+                Collections.singletonList(itemLifecycleSource));
+
+        getContextsMap(statisticsManager).put(deviceInfo, statisticContext);
+
+        final ChangeStatisticsWorkModeInputBuilder changeStatisticsWorkModeInputBld =
+                new ChangeStatisticsWorkModeInputBuilder()
+                        .setMode(StatisticsWorkMode.FULLYDISABLED);
+
+        Future<RpcResult<Void>> workMode = statisticsManager.changeStatisticsWorkMode(changeStatisticsWorkModeInputBld.build());
+        checkWorkModeChangeOutcome(workMode);
+
+        Mockito.verify(itemLifeCycleRegistry).getLifeCycleSources();
+        Mockito.verify(statisticContext).getPollTimeout();
+        Mockito.verify(pollTimeout).cancel();
+        Mockito.verify(itemLifecycleSource).setItemLifecycleListener(Matchers.<ItemLifecycleListener>any());
+    }
+
+    /**
+     * switching to {@link StatisticsWorkMode#FULLYDISABLED} and back
+     * to {@link StatisticsWorkMode#COLLECTALL}; with lifecycleRegistry and pollTimeout
+     *
+     * @throws Exception
+     */
+    @Test
+    public void testChangeStatisticsWorkMode3() throws Exception {
+        final Timeout pollTimeout = Mockito.mock(Timeout.class);
+        final ItemLifeCycleSource itemLifecycleSource = Mockito.mock(ItemLifeCycleSource.class);
+        Mockito.doNothing().when(itemLifecycleSource)
+                .setItemLifecycleListener(itemLifeCycleListenerCapt.capture());
+
+        final StatisticsContext statisticContext = Mockito.mock(StatisticsContext.class);
+        when(statisticContext.getPollTimeout()).thenReturn(
+                Optional.of(pollTimeout));
+        when(statisticContext.getItemLifeCycleListener()).thenReturn(
+                Mockito.mock(ItemLifecycleListener.class));
+        when(itemLifeCycleRegistry.getLifeCycleSources()).thenReturn(
+                Collections.singletonList(itemLifecycleSource));
+
+        getContextsMap(statisticsManager).put(deviceInfo, statisticContext);
+
+        final ChangeStatisticsWorkModeInputBuilder changeStatisticsWorkModeInputBld =
+                new ChangeStatisticsWorkModeInputBuilder()
+                        .setMode(StatisticsWorkMode.FULLYDISABLED);
+
+        Future<RpcResult<Void>> workMode;
+        workMode = statisticsManager.changeStatisticsWorkMode(
+                changeStatisticsWorkModeInputBld.build());
+        checkWorkModeChangeOutcome(workMode);
+
+        changeStatisticsWorkModeInputBld.setMode(StatisticsWorkMode.COLLECTALL);
+        workMode = statisticsManager.changeStatisticsWorkMode(
+                changeStatisticsWorkModeInputBld.build());
+        checkWorkModeChangeOutcome(workMode);
+
+        Mockito.verify(itemLifeCycleRegistry, Mockito.times(2)).getLifeCycleSources();
+        Mockito.verify(statisticContext).getPollTimeout();
+        Mockito.verify(pollTimeout).cancel();
+
+        final List<ItemLifecycleListener> itemLifeCycleListenerValues = itemLifeCycleListenerCapt.getAllValues();
+        Assert.assertEquals(2, itemLifeCycleListenerValues.size());
+        Assert.assertNotNull(itemLifeCycleListenerValues.get(0));
+        Assert.assertNull(itemLifeCycleListenerValues.get(1));
+    }
+
+    @Test
+    public void testClose() throws Exception {
+        statisticsManager.close();
+        Mockito.verify(serviceControlRegistration).close();
+    }
+
+    @Test
+    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);
+        Assert.assertEquals(6000L, StatisticsManagerImpl.getCurrentTimerDelay());
+    }
+
+    @Test
+    public void testPollStatistics() throws Exception {
+        final StatisticsContext statisticsContext = Mockito.mock(StatisticsContext.class);
+        final TimeCounter mockTimerCounter = Mockito.mock(TimeCounter.class);
+
+        statisticsManager.pollStatistics(mockedDeviceContext.getDeviceState(), statisticsContext, mockTimerCounter, mockedDeviceInfo);
+        verify(mockedDeviceContext).getDeviceState();
+
+        statisticsManager.pollStatistics(mockedDeviceContext.getDeviceState(), statisticsContext, mockTimerCounter, mockedDeviceInfo);
+
+        when(mockedDeviceContext.getDeviceState().isStatisticsPollingEnabled()).thenReturn(true);
+        statisticsManager.pollStatistics(mockedDeviceContext.getDeviceState(), statisticsContext, mockTimerCounter, mockedDeviceInfo);
+
+        when(statisticsContext.gatherDynamicData()).thenReturn(Futures.immediateCheckedFuture(Boolean.TRUE));
+        when(statisticsContext.isSchedulingEnabled()).thenReturn(Boolean.TRUE);
+        statisticsManager.pollStatistics(mockedDeviceContext.getDeviceState(), statisticsContext, mockTimerCounter, mockedDeviceInfo);
+        Mockito.verify(mockTimerCounter).markStart();
+        Mockito.verify(mockTimerCounter).addTimeMark();
 
-        statisticsManager.onDeviceContextClosed(mockedDeviceContext);
+        when(statisticsContext.gatherDynamicData()).thenReturn(Futures.immediateFailedFuture(new Throwable("error msg")));
+        statisticsManager.pollStatistics(mockedDeviceContext.getDeviceState(), statisticsContext, mockTimerCounter, mockedDeviceInfo);
+        Mockito.verify(mockTimerCounter,times(2)).addTimeMark();
     }
-}
\ No newline at end of file
+}