HC2VPP-118 - reference check for acl's

Adds reference cecking before delete for Acl and MacIpAcl

Change-Id: I7acc92784498928059a96f88ba89604fc70bc075
Signed-off-by: Jan Srnicek <jsrnicek@cisco.com>

4 files changed, 198 insertions, 0 deletions

diff --git a/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/write/AclReferenceCheckTest.java b/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/write/AclReferenceCheckTest.java
new file mode 100644
index 000000000..61783af57
--- /dev/null
+++ b/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/write/AclReferenceCheckTest.java
@@ -0,0 +1,86 @@
+/*
+ * Copyright (c) 2017 Cisco and/or its affiliates.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at:
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package io.fd.hc2vpp.acl.write;
+
+import static io.fd.hc2vpp.acl.write.VppAclCustomizer.AclReferenceCheck.checkAclReferenced;
+import static java.util.stream.Collectors.toSet;
+import static org.hamcrest.MatcherAssert.assertThat;
+import static org.hamcrest.Matchers.containsInAnyOrder;
+import static org.hamcrest.Matchers.hasSize;
+import static org.mockito.Mockito.when;
+import static org.mockito.MockitoAnnotations.initMocks;
+
+import com.google.common.base.Optional;
+import io.fd.hc2vpp.acl.AclTestSchemaContext;
+import io.fd.honeycomb.test.tools.HoneycombTestRunner;
+import io.fd.honeycomb.test.tools.annotations.InjectTestData;
+import io.fd.honeycomb.translate.write.WriteContext;
+import java.util.List;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.Mock;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.access.control.list.rev160708.access.lists.AclBuilder;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.Interfaces;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.interfaces.Interface;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.vpp.acl.rev161214.VppAcl;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.vpp.acl.rev161214.VppMacipAcl;
+import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
+
+
+@RunWith(HoneycombTestRunner.class)
+public class AclReferenceCheckTest implements AclTestSchemaContext {
+
+    @InjectTestData(id = "/ietf-interfaces:interfaces", resourcePath = "/reference/acl-references.json")
+    private Interfaces interfaces;
+
+    @Mock
+    private WriteContext writeContext;
+
+    @Before
+    public void init(){
+        initMocks(this);
+        when(writeContext.readAfter(InstanceIdentifier.create(Interfaces.class))).thenReturn(Optional.of(interfaces));
+    }
+
+    @Test
+    public void testReferencedVppAclFirst() {
+        final List<Interface> referenced = checkAclReferenced(writeContext, new AclBuilder()
+                .setAclName("acl1").setAclType(VppAcl.class).build());
+        assertThat(referenced, hasSize(3));
+        assertThat(referenced.stream().map(Interface::getName).collect(toSet()),
+                containsInAnyOrder("eth0", "eth1", "eth2"));
+    }
+
+    @Test
+    public void testReferencedVppAclSecond() {
+        final List<Interface> referenced = checkAclReferenced(writeContext, new AclBuilder()
+                .setAclName("acl2").setAclType(VppAcl.class).build());
+        assertThat(referenced, hasSize(1));
+        assertThat(referenced.stream().map(Interface::getName).collect(toSet()),
+                containsInAnyOrder("eth1"));
+    }
+
+    @Test
+    public void testReferencedMacipAcl() {
+        final List<Interface> referenced = checkAclReferenced(writeContext, new AclBuilder()
+                .setAclName("acl4").setAclType(VppMacipAcl.class).build());
+        assertThat(referenced, hasSize(1));
+        assertThat(referenced.stream().map(Interface::getName).collect(toSet()),
+                containsInAnyOrder("eth2"));
+    }
+}
\ No newline at end of file
diff --git a/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/write/VppAclCustomizerTest.java b/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/write/VppAclCustomizerTest.java
index 197b68626..daf2c0922 100644
--- a/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/write/VppAclCustomizerTest.java
+++ b/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/write/VppAclCustomizerTest.java
@@ -18,11 +18,14 @@ package io.fd.hc2vpp.acl.write;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
 import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
+import com.google.common.base.Optional;
 import io.fd.hc2vpp.acl.AclTestSchemaContext;
 import io.fd.hc2vpp.acl.util.AclContextManager;
 import io.fd.hc2vpp.common.test.write.WriterCustomizerTest;
@@ -40,6 +43,7 @@ import io.fd.vpp.jvpp.acl.types.AclRule;
 import io.fd.vpp.jvpp.acl.types.MacipAclRule;
 import java.nio.charset.StandardCharsets;
 import java.util.Arrays;
+import java.util.Collections;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.ArgumentCaptor;
@@ -48,6 +52,11 @@ import org.mockito.Mock;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.access.control.list.rev160708.AccessLists;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.access.control.list.rev160708.access.lists.Acl;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.access.control.list.rev160708.access.lists.AclKey;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.Interfaces;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.InterfacesBuilder;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.interfaces.Interface;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang._interface.acl.rev161214.VppAclInterfaceAugmentation;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang._interface.acl.rev161214.VppAclInterfaceStateAugmentation;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.vpp.acl.rev161214.VppAcl;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 
@@ -223,6 +232,7 @@ public class VppAclCustomizerTest extends WriterCustomizerTest implements AclTes
     @Test
     public void deleteCurrentAttributesIcmpIpv4(@InjectTestData(resourcePath = "/acl/standard/standard-acl-icmp.json")
                                                         AccessLists standardAcls) throws Exception {
+        when(writeContext.readAfter(InstanceIdentifier.create(Interfaces.class))).thenReturn(Optional.absent());
         final int aclIndex = 4;
         when(standardAclContext.getAclIndex("standard-acl", mappingContext)).thenReturn(aclIndex);
         aclCustomizer.deleteCurrentAttributes(validId, standardAcls.getAcl().get(0), writeContext);
@@ -235,6 +245,7 @@ public class VppAclCustomizerTest extends WriterCustomizerTest implements AclTes
     public void deleteCurrentAttributesIcmpIpv6(
             @InjectTestData(resourcePath = "/acl/standard/standard-acl-icmp-v6.json")
                     AccessLists standardAcls) throws Exception {
+        when(writeContext.readAfter(InstanceIdentifier.create(Interfaces.class))).thenReturn(Optional.absent());
         final int aclIndex = 4;
         when(standardAclContext.getAclIndex("standard-acl", mappingContext)).thenReturn(aclIndex);
         aclCustomizer.deleteCurrentAttributes(validId, standardAcls.getAcl().get(0), writeContext);
@@ -246,6 +257,7 @@ public class VppAclCustomizerTest extends WriterCustomizerTest implements AclTes
     @Test
     public void deleteCurrentAttributesTcp(@InjectTestData(resourcePath = "/acl/standard/standard-acl-tcp.json")
                                                    AccessLists standardAcls) throws Exception {
+        when(writeContext.readAfter(InstanceIdentifier.create(Interfaces.class))).thenReturn(Optional.absent());
         final int aclIndex = 4;
         when(standardAclContext.getAclIndex("standard-acl", mappingContext)).thenReturn(aclIndex);
         aclCustomizer.deleteCurrentAttributes(validId, standardAcls.getAcl().get(0), writeContext);
@@ -257,6 +269,7 @@ public class VppAclCustomizerTest extends WriterCustomizerTest implements AclTes
     @Test
     public void deleteCurrentAttributesUdp(@InjectTestData(resourcePath = "/acl/standard/standard-acl-udp.json")
                                                    AccessLists standardAcls) throws Exception {
+        when(writeContext.readAfter(InstanceIdentifier.create(Interfaces.class))).thenReturn(Optional.absent());
         final int aclIndex = 4;
         when(standardAclContext.getAclIndex("standard-acl", mappingContext)).thenReturn(aclIndex);
         aclCustomizer.deleteCurrentAttributes(validId, standardAcls.getAcl().get(0), writeContext);
@@ -265,6 +278,27 @@ public class VppAclCustomizerTest extends WriterCustomizerTest implements AclTes
         assertEquals(aclIndex, aclDelRequestCaptor.getValue().aclIndex);
     }
 
+    @Test
+    public void deleteCurrentAttributesUdpReferenced(
+            @InjectTestData(resourcePath = "/acl/standard/standard-acl-udp.json")
+                    AccessLists standardAcls,
+            @InjectTestData(resourcePath = "/acl/standard/interface-ref-acl-udp.json")
+                    Interfaces references) throws Exception {
+        // TODO - HONEYCOMB-349 - change after resolving to specific node injection
+        when(writeContext.readAfter(InstanceIdentifier.create(Interfaces.class))).thenReturn(
+                Optional.of(new InterfacesBuilder().setInterface(references.getInterface()).build()));
+
+        final int aclIndex = 4;
+        when(standardAclContext.getAclIndex("standard-acl", mappingContext)).thenReturn(aclIndex);
+        try {
+            aclCustomizer.deleteCurrentAttributes(validId, standardAcls.getAcl().get(0), writeContext);
+        } catch (IllegalStateException e) {
+            verify(aclApi, never()).aclDel(any(AclDel.class));
+            return;
+        }
+        fail("IllegalStateException should have been thrown");
+    }
+
     private void verifyUdpRequest(final int aclIndex) {
         final AclAddReplace request = aclAddReplaceRequestCaptor.getValue();
         assertEquals(aclIndex, request.aclIndex);
diff --git a/acl/acl-impl/src/test/resources/acl/standard/interface-ref-acl-udp.json b/acl/acl-impl/src/test/resources/acl/standard/interface-ref-acl-udp.json
new file mode 100644
index 000000000..ae987a798
--- /dev/null
+++ b/acl/acl-impl/src/test/resources/acl/standard/interface-ref-acl-udp.json
@@ -0,0 +1,21 @@
+{
+  "interfaces":{
+    "interface": [{
+      "name": "eth2",
+      "acl": {
+        "ingress": {
+          "vpp-acls": [
+            {
+              "type": "vpp-acl:vpp-acl",
+              "name": "standard-acl"
+            }
+          ],
+          "vpp-macip-acl": {
+            "name": "acl4",
+            "type": "vpp-acl:vpp-macip-acl"
+          }
+        }
+      }
+    }]
+  }
+}
\ No newline at end of file
diff --git a/acl/acl-impl/src/test/resources/reference/acl-references.json b/acl/acl-impl/src/test/resources/reference/acl-references.json
new file mode 100644
index 000000000..d60eeca45
--- /dev/null
+++ b/acl/acl-impl/src/test/resources/reference/acl-references.json
@@ -0,0 +1,57 @@
+{
+  "interfaces": {
+    "interface": [
+      {
+        "name": "eth0",
+        "acl": {
+          "ingress": {
+            "vpp-acls": [
+              {
+                "type": "vpp-acl",
+                "name": "acl1"
+              }
+            ]
+          }
+        }
+      },
+      {
+        "name": "eth1",
+        "acl": {
+          "egress": {
+            "vpp-acls": [
+              {
+                "type": "vpp-acl",
+                "name": "acl1"
+              }
+            ]
+          },
+          "ingress": {
+            "vpp-acls": [
+              {
+                "type": "vpp-acl",
+                "name": "acl2"
+              }
+            ]
+          }
+        }
+      },
+      {
+        "name": "eth2",
+        "acl": {
+          "ingress": {
+            "vpp-acls": [
+              {
+                "type": "vpp-acl",
+                "name": "acl1"
+              }
+            ],
+            "vpp-macip-acl": {
+              "name": "acl4",
+              "type": "vpp-macip-acl"
+            }
+          }
+        }
+      }
+    ]
+  }
+}
\ No newline at end of file