X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=applications%2Fforwardingrules-manager%2Fsrc%2Ftest%2Fjava%2Ftest%2Fmock%2FMeterListenerTest.java;h=7f6e40866f297a5b436b3c83bf5d843677a05a78;hb=137e4d7d86e8f402f3d52fd0fa162792f9ff60eb;hp=bab64fc88575015472ad3c50cd87936797447373;hpb=77a7d6170ea741d9109dee08e981af9143bc0141;p=openflowplugin.git diff --git a/applications/forwardingrules-manager/src/test/java/test/mock/MeterListenerTest.java b/applications/forwardingrules-manager/src/test/java/test/mock/MeterListenerTest.java index bab64fc885..7f6e40866f 100644 --- a/applications/forwardingrules-manager/src/test/java/test/mock/MeterListenerTest.java +++ b/applications/forwardingrules-manager/src/test/java/test/mock/MeterListenerTest.java @@ -19,10 +19,12 @@ import org.mockito.Mockito; import org.mockito.runners.MockitoJUnitRunner; import org.opendaylight.controller.md.sal.binding.api.WriteTransaction; import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType; +import org.opendaylight.controller.sal.binding.api.NotificationProviderService; import org.opendaylight.controller.sal.binding.api.RpcProviderRegistry; import org.opendaylight.mdsal.singleton.common.api.ClusterSingletonServiceProvider; import org.opendaylight.openflowplugin.applications.frm.impl.DeviceMastershipManager; import org.opendaylight.openflowplugin.applications.frm.impl.ForwardingRulesManagerImpl; +import org.opendaylight.openflowplugin.applications.reconciliation.ReconciliationManager; import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.FlowCapableNode; import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.meters.Meter; import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.meters.MeterBuilder; @@ -46,13 +48,17 @@ import test.mock.util.SalMeterServiceMock; @RunWith(MockitoJUnitRunner.class) public class MeterListenerTest extends FRMTest { private ForwardingRulesManagerImpl forwardingRulesManager; - private final static NodeId NODE_ID = new NodeId("testnode:1"); - private final static NodeKey s1Key = new NodeKey(NODE_ID); + private static final NodeId NODE_ID = new NodeId("testnode:1"); + private static final NodeKey NODE_KEY = new NodeKey(NODE_ID); RpcProviderRegistry rpcProviderRegistryMock = new RpcProviderRegistryMock(); @Mock ClusterSingletonServiceProvider clusterSingletonService; @Mock DeviceMastershipManager deviceMastershipManager; + @Mock + private NotificationProviderService notificationService; + @Mock + private ReconciliationManager reconciliationManager; @Before public void setUp() { @@ -60,7 +66,11 @@ public class MeterListenerTest extends FRMTest { getDataBroker(), rpcProviderRegistryMock, getConfig(), - clusterSingletonService); + clusterSingletonService, + notificationService, + getConfigurationService(), + reconciliationManager); + forwardingRulesManager.start(); // TODO consider tests rewrite (added because of complicated access) forwardingRulesManager.setDeviceMastershipManager(deviceMastershipManager); @@ -69,12 +79,12 @@ public class MeterListenerTest extends FRMTest { @Test public void addTwoMetersTest() throws Exception { - addFlowCapableNode(s1Key); + addFlowCapableNode(NODE_KEY); MeterKey meterKey = new MeterKey(new MeterId((long) 2000)); - InstanceIdentifier meterII = InstanceIdentifier.create(Nodes.class).child(Node.class, s1Key) + InstanceIdentifier meterII = InstanceIdentifier.create(Nodes.class).child(Node.class, NODE_KEY) .augmentation(FlowCapableNode.class).child(Meter.class, meterKey); - Meter meter = new MeterBuilder().setKey(meterKey).setMeterName("meter_one").build(); + Meter meter = new MeterBuilder().withKey(meterKey).setMeterName("meter_one").build(); WriteTransaction writeTx = getDataBroker().newWriteOnlyTransaction(); writeTx.put(LogicalDatastoreType.CONFIGURATION, meterII, meter); @@ -85,9 +95,9 @@ public class MeterListenerTest extends FRMTest { assertEquals("DOM-0", addMeterCalls.get(0).getTransactionUri().getValue()); meterKey = new MeterKey(new MeterId((long) 2001)); - meterII = InstanceIdentifier.create(Nodes.class).child(Node.class, s1Key) + meterII = InstanceIdentifier.create(Nodes.class).child(Node.class, NODE_KEY) .augmentation(FlowCapableNode.class).child(Meter.class, meterKey); - meter = new MeterBuilder().setKey(meterKey).setMeterName("meter_two").setBarrier(true).build(); + meter = new MeterBuilder().withKey(meterKey).setMeterName("meter_two").setBarrier(true).build(); writeTx = getDataBroker().newWriteOnlyTransaction(); writeTx.put(LogicalDatastoreType.CONFIGURATION, meterII, meter); assertCommit(writeTx.submit()); @@ -100,12 +110,12 @@ public class MeterListenerTest extends FRMTest { @Test public void updateMeterTest() throws Exception { - addFlowCapableNode(s1Key); + addFlowCapableNode(NODE_KEY); MeterKey meterKey = new MeterKey(new MeterId((long) 2000)); - InstanceIdentifier meterII = InstanceIdentifier.create(Nodes.class).child(Node.class, s1Key) + InstanceIdentifier meterII = InstanceIdentifier.create(Nodes.class).child(Node.class, NODE_KEY) .augmentation(FlowCapableNode.class).child(Meter.class, meterKey); - Meter meter = new MeterBuilder().setKey(meterKey).setMeterName("meter_one").setBarrier(false).build(); + Meter meter = new MeterBuilder().withKey(meterKey).setMeterName("meter_one").setBarrier(false).build(); WriteTransaction writeTx = getDataBroker().newWriteOnlyTransaction(); writeTx.put(LogicalDatastoreType.CONFIGURATION, meterII, meter); @@ -115,7 +125,7 @@ public class MeterListenerTest extends FRMTest { assertEquals(1, addMeterCalls.size()); assertEquals("DOM-0", addMeterCalls.get(0).getTransactionUri().getValue()); - meter = new MeterBuilder().setKey(meterKey).setMeterName("meter_two").setBarrier(true).build(); + meter = new MeterBuilder().withKey(meterKey).setMeterName("meter_two").setBarrier(true).build(); writeTx = getDataBroker().newWriteOnlyTransaction(); writeTx.put(LogicalDatastoreType.CONFIGURATION, meterII, meter); assertCommit(writeTx.submit()); @@ -128,12 +138,12 @@ public class MeterListenerTest extends FRMTest { @Test public void removeMeterTest() throws Exception { - addFlowCapableNode(s1Key); + addFlowCapableNode(NODE_KEY); MeterKey meterKey = new MeterKey(new MeterId((long) 2000)); - InstanceIdentifier meterII = InstanceIdentifier.create(Nodes.class).child(Node.class, s1Key) + InstanceIdentifier meterII = InstanceIdentifier.create(Nodes.class).child(Node.class, NODE_KEY) .augmentation(FlowCapableNode.class).child(Meter.class, meterKey); - Meter meter = new MeterBuilder().setKey(meterKey).setMeterName("meter_one").build(); + Meter meter = new MeterBuilder().withKey(meterKey).setMeterName("meter_one").build(); WriteTransaction writeTx = getDataBroker().newWriteOnlyTransaction(); writeTx.put(LogicalDatastoreType.CONFIGURATION, meterII, meter); @@ -153,15 +163,14 @@ public class MeterListenerTest extends FRMTest { assertEquals(meterII, removeMeterCalls.get(0).getMeterRef().getValue()); } - @Test public void staleMeterCreationTest() throws Exception { - addFlowCapableNode(s1Key); + addFlowCapableNode(NODE_KEY); StaleMeterKey meterKey = new StaleMeterKey(new MeterId((long) 2000)); - InstanceIdentifier meterII = InstanceIdentifier.create(Nodes.class).child(Node.class, s1Key) + InstanceIdentifier meterII = InstanceIdentifier.create(Nodes.class).child(Node.class, NODE_KEY) .augmentation(FlowCapableNode.class).child(StaleMeter.class, meterKey); - StaleMeter meter = new StaleMeterBuilder().setKey(meterKey).setMeterName("stale_meter_one").build(); + StaleMeter meter = new StaleMeterBuilder().withKey(meterKey).setMeterName("stale_meter_one").build(); WriteTransaction writeTx = getDataBroker().newWriteOnlyTransaction(); writeTx.put(LogicalDatastoreType.CONFIGURATION, meterII, meter); @@ -172,5 +181,4 @@ public class MeterListenerTest extends FRMTest { public void tearDown() throws Exception { forwardingRulesManager.close(); } - }