Policy exclusions & parallel netconf transactions
[groupbasedpolicy.git] / renderers / vpp / src / test / java / org / opendaylight / groupbasedpolicy / renderer / vpp / policy / ForwardingManagerTest.java
index 04a8de7a8139fcd572370caa317f29bdfcf066a0..fd48e23cfa260fecba06551624d258970b991016 100644 (file)
@@ -1,3 +1,10 @@
+/*
+ * Copyright (c) 2016 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.groupbasedpolicy.renderer.vpp.policy;
 
 import java.util.Arrays;
@@ -15,7 +22,15 @@ import org.opendaylight.controller.md.sal.binding.api.WriteTransaction;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.groupbasedpolicy.renderer.vpp.DtoFactory;
 import org.opendaylight.groupbasedpolicy.renderer.vpp.api.BridgeDomainManager;
+import org.opendaylight.groupbasedpolicy.renderer.vpp.dhcp.DhcpRelayHandler;
 import org.opendaylight.groupbasedpolicy.renderer.vpp.iface.InterfaceManager;
+import org.opendaylight.groupbasedpolicy.renderer.vpp.lisp.LispStateManager;
+import org.opendaylight.groupbasedpolicy.renderer.vpp.lisp.flat.overlay.FlatOverlayManager;
+import org.opendaylight.groupbasedpolicy.renderer.vpp.lisp.loopback.LoopbackManager;
+import org.opendaylight.groupbasedpolicy.renderer.vpp.nat.NatManager;
+import org.opendaylight.groupbasedpolicy.renderer.vpp.policy.acl.AccessListWrapper;
+import org.opendaylight.groupbasedpolicy.renderer.vpp.policy.acl.AclManager;
+import org.opendaylight.groupbasedpolicy.renderer.vpp.routing.RoutingManager;
 import org.opendaylight.groupbasedpolicy.renderer.vpp.util.KeyFactory;
 import org.opendaylight.groupbasedpolicy.test.CustomDataBrokerTest;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.base_endpoint.rev160427.has.absolute.location.AbsoluteLocation;
@@ -28,10 +43,10 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.renderer.r
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.vpp_renderer.rev160425.Config;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.vpp_renderer.rev160425.NetworkTypeBase;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.vpp_renderer.rev160425.VlanNetwork;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.vpp_renderer.rev160425.config.BridgeDomain;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.vpp_renderer.rev160425.config.BridgeDomainBuilder;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.vpp_renderer.rev160425.config.GbpBridgeDomain;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.vpp_renderer.rev160425.config.GbpBridgeDomainBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.l2.types.rev130827.VlanId;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.v3po.rev150105.VxlanVni;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.v3po.rev170607.VxlanVni;
 import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.NodeId;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 
@@ -46,36 +61,56 @@ public class ForwardingManagerTest extends CustomDataBrokerTest {
     private static final String BD_1 = "bd1";
     private static final NodeId NODE_1 = new NodeId("node1");
     private static final VlanId VLAN_1 = new VlanId(1);
+    private static final boolean IS_BVI = false;
     @Mock
     private InterfaceManager ifaceManager;
     @Mock
+    private AclManager aclManager;
+    @Mock
     private BridgeDomainManager bdManager;
+    @Mock
+    private NatManager natManager;
+    @Mock
+    private RoutingManager routingManager;
+    @Mock
+    private LispStateManager lispStateManager;
+    @Mock
+    private FlatOverlayManager flatOverlayManager;
+    @Mock
+    private LoopbackManager loopbackManager;
+    @Mock
+    private DhcpRelayHandler dhcpRelayHandler;
 
     private ForwardingManager fwdManager;
 
     @Before
     public void init() {
-        fwdManager = new ForwardingManager(ifaceManager, bdManager, getDataBroker());
+        fwdManager =
+            new ForwardingManager(ifaceManager, aclManager, natManager, routingManager, bdManager, lispStateManager,
+                    loopbackManager, flatOverlayManager, dhcpRelayHandler, getDataBroker());
+        Mockito.when(aclManager.resolveAclsOnInterface(Mockito.any(RendererEndpointKey.class),
+                Mockito.any(PolicyContext.class)))
+            .thenReturn(Futures.immediateFuture(null));
     }
 
     @Override
     public Collection<Class<?>> getClassesFromModules() {
-        return Arrays.asList(BridgeDomain.class);
+        return Arrays.asList(GbpBridgeDomain.class);
     }
 
     @Test
     public void testReadBridgeDomainConfig() throws Exception {
-        BridgeDomain bd = new BridgeDomainBuilder().setId(BD_1).setType(NetworkTypeBase.class).build();
-        InstanceIdentifier<BridgeDomain> bdIid =
-                InstanceIdentifier.builder(Config.class).child(BridgeDomain.class, bd.getKey()).build();
-        Optional<BridgeDomain> bdOptional = fwdManager.readBridgeDomainConfig(bd.getId());
+        GbpBridgeDomain bd = new GbpBridgeDomainBuilder().setId(BD_1).setType(NetworkTypeBase.class).build();
+        InstanceIdentifier<GbpBridgeDomain> bdIid =
+                InstanceIdentifier.builder(Config.class).child(GbpBridgeDomain.class, bd.getKey()).build();
+        Optional<GbpBridgeDomain> bdOptional = fwdManager.readGbpBridgeDomainConfig(bd.getId());
         Assert.assertFalse(bdOptional.isPresent());
 
         WriteTransaction wTx = getDataBroker().newWriteOnlyTransaction();
         wTx.put(LogicalDatastoreType.CONFIGURATION, bdIid, bd);
         wTx.submit().get();
 
-        bdOptional = fwdManager.readBridgeDomainConfig(bd.getId());
+        bdOptional = fwdManager.readGbpBridgeDomainConfig(bd.getId());
         Assert.assertTrue(bdOptional.isPresent());
         Assert.assertEquals(bd, bdOptional.get());
     }
@@ -97,9 +132,10 @@ public class ForwardingManagerTest extends CustomDataBrokerTest {
         Mockito.when(bdManager.createVlanBridgeDomainOnVppNode(Mockito.eq(BD_1), Mockito.any(VlanId.class),
                 Mockito.eq(NODE_1)))
             .thenReturn(Futures.immediateFuture(null));
-        BridgeDomain bd = new BridgeDomainBuilder().setId(BD_1).setType(VlanNetwork.class).setVlan(VLAN_1).build();
-        InstanceIdentifier<BridgeDomain> bdIid =
-                InstanceIdentifier.builder(Config.class).child(BridgeDomain.class, bd.getKey()).build();
+        GbpBridgeDomain bd =
+            new GbpBridgeDomainBuilder().setId(BD_1).setType(VlanNetwork.class).setVlan(VLAN_1).build();
+        InstanceIdentifier<GbpBridgeDomain> bdIid =
+                InstanceIdentifier.builder(Config.class).child(GbpBridgeDomain.class, bd.getKey()).build();
         WriteTransaction wTx = getDataBroker().newWriteOnlyTransaction();
         wTx.put(LogicalDatastoreType.CONFIGURATION, bdIid, bd);
         wTx.submit().get();
@@ -114,8 +150,6 @@ public class ForwardingManagerTest extends CustomDataBrokerTest {
     public void testRemoveBridgeDomainOnNodes() throws Exception {
         Mockito.when(bdManager.removeBridgeDomainFromVppNode(Mockito.eq(BD_1), Mockito.eq(NODE_1)))
             .thenReturn(Futures.immediateFuture(null));
-        SetMultimap<String, NodeId> vppNodesByBd = ImmutableSetMultimap.of(BD_1, NODE_1);
-
         bdManager.removeBridgeDomainFromVppNode(BD_1, NODE_1);
         Mockito.verify(bdManager).removeBridgeDomainFromVppNode(Matchers.eq(BD_1), Matchers.eq(NODE_1));
     }
@@ -140,12 +174,11 @@ public class ForwardingManagerTest extends CustomDataBrokerTest {
         AddressEndpointWithLocation firstAddrEpWithLoc =
                 policyCtx.getAddrEpByKey().get(KeyFactory.addressEndpointKey(firstRendererEp.getKey()));
         Mockito.when(ifaceManager.addBridgeDomainToInterface(Mockito.eq(DtoFactory.L2FD_CTX.getValue()),
-                Mockito.eq(firstAddrEpWithLoc)))
+                Mockito.eq(firstAddrEpWithLoc), Mockito.anyListOf(AccessListWrapper.class),Mockito.eq(IS_BVI)))
             .thenReturn(Futures.immediateFuture(null));
-
         fwdManager.createForwardingForEndpoint(firstRendererEp.getKey(), policyCtx);
         Mockito.verify(ifaceManager).addBridgeDomainToInterface(Matchers.eq(DtoFactory.L2FD_CTX.getValue()),
-                Matchers.eq(firstAddrEpWithLoc));
+                Matchers.eq(firstAddrEpWithLoc), Mockito.anyListOf(AccessListWrapper.class), Mockito.eq(IS_BVI));
     }
 
     @Test
@@ -153,12 +186,14 @@ public class ForwardingManagerTest extends CustomDataBrokerTest {
         String clientIp = "1.1.1.1";
         String clientIfaceName = "client1";
         String bdNameOnVpp = "bdRed";
-        AbsoluteLocation clientLocation = DtoFactory.absoluteLocation(DtoFactory.VPP_NODE_1_IID, bdNameOnVpp, clientIfaceName);
+        AbsoluteLocation clientLocation =
+            DtoFactory.absoluteLocation(DtoFactory.VPP_NODE_1_IID, bdNameOnVpp, clientIfaceName);
         AddressEndpointWithLocation clientEp =
                 DtoFactory.createEndpoint(clientIp, DtoFactory.L2FD_CTX.getValue(), clientLocation);
         String webIp = "2.2.2.2";
         String webIfaceName = "web1";
-        AbsoluteLocation webLocation = DtoFactory.absoluteLocation(DtoFactory.VPP_NODE_1_IID, bdNameOnVpp, webIfaceName);
+        AbsoluteLocation webLocation =
+            DtoFactory.absoluteLocation(DtoFactory.VPP_NODE_1_IID, bdNameOnVpp, webIfaceName);
         AddressEndpointWithLocation webEp =
                 DtoFactory.createEndpoint(webIp, DtoFactory.L2FD_CTX.getValue(), webLocation);
         Configuration configuration = DtoFactory.createConfiguration(Arrays.asList(clientEp), Arrays.asList(webEp));