summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarek Gradzki <mgradzki@cisco.com>2016-12-21 11:00:09 +0100
committerMarek Gradzki <mgradzki@cisco.com>2016-12-21 11:00:09 +0100
commitd5e1e3a563807561caa0b64c414332d85a31f216 (patch)
tree93d321ad158f4b80c784b3606dbf70bf86109213
parent52fabc2b192be911c6de582ea27d145228187a7e (diff)
Fix interface read when alcs are not configured
Change-Id: Id780d9f503901d551f5a9d5693ca0a28cf596a5c Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
-rw-r--r--acl/acl-impl/src/main/java/io/fd/hc2vpp/acl/read/VppAclCustomizer.java14
-rw-r--r--acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/AbstractVppAclCustomizerTest.java79
-rw-r--r--acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/VppEgressAclCustomizerTest.java63
-rw-r--r--acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/VppIngressAclCustomizerTest.java64
4 files changed, 218 insertions, 2 deletions
diff --git a/acl/acl-impl/src/main/java/io/fd/hc2vpp/acl/read/VppAclCustomizer.java b/acl/acl-impl/src/main/java/io/fd/hc2vpp/acl/read/VppAclCustomizer.java
index 90f2b2fe2..948b6408a 100644
--- a/acl/acl-impl/src/main/java/io/fd/hc2vpp/acl/read/VppAclCustomizer.java
+++ b/acl/acl-impl/src/main/java/io/fd/hc2vpp/acl/read/VppAclCustomizer.java
@@ -102,7 +102,13 @@ public class VppAclCustomizer extends FutureJVppAclCustomizer
dump.aclInterfaceListDetails = dump.aclInterfaceListDetails
.stream()
.map(iface -> {
- iface.acls = Arrays.copyOfRange(iface.acls, 0, iface.nInput - 1);
+ if (iface.acls != null && iface.acls.length > 0) {
+ if (iface.nInput <= 0) {
+ iface.acls = new int[0];
+ } else {
+ iface.acls = Arrays.copyOfRange(iface.acls, 0, iface.nInput);
+ }
+ }
return iface;
})
.collect(Collectors.toList());
@@ -116,7 +122,11 @@ public class VppAclCustomizer extends FutureJVppAclCustomizer
dump.aclInterfaceListDetails = dump.aclInterfaceListDetails
.stream()
.map(iface -> {
- iface.acls = Arrays.copyOfRange(iface.acls, iface.nInput, iface.acls.length);
+ if (iface.nInput >= iface.acls.length) {
+ iface.acls = new int[0];
+ } else {
+ iface.acls = Arrays.copyOfRange(iface.acls, iface.nInput, iface.acls.length);
+ }
return iface;
})
.collect(Collectors.toList());
diff --git a/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/AbstractVppAclCustomizerTest.java b/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/AbstractVppAclCustomizerTest.java
new file mode 100644
index 000000000..7bfffb63b
--- /dev/null
+++ b/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/AbstractVppAclCustomizerTest.java
@@ -0,0 +1,79 @@
+/*
+ * Copyright (c) 2016 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.read;
+
+import static org.junit.Assert.assertTrue;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.when;
+
+import io.fd.hc2vpp.common.test.read.ListReaderCustomizerTest;
+import io.fd.hc2vpp.common.translate.util.NamingContext;
+import io.fd.honeycomb.translate.read.ReadFailedException;
+import io.fd.vpp.jvpp.acl.dto.AclInterfaceListDetails;
+import io.fd.vpp.jvpp.acl.dto.AclInterfaceListDetailsReplyDump;
+import io.fd.vpp.jvpp.acl.future.FutureJVppAclFacade;
+import org.junit.Test;
+import org.mockito.Mock;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.InterfacesState;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.interfaces.state.Interface;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.interfaces.state.InterfaceKey;
+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._interface.acl.rev161214._interface.acl.attributes.Acl;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang._interface.acl.rev161214._interface.acl.attributes.acl.Ingress;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang._interface.acl.rev161214.vpp.acls.base.attributes.VppAcls;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang._interface.acl.rev161214.vpp.acls.base.attributes.VppAclsBuilder;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang._interface.acl.rev161214.vpp.acls.base.attributes.VppAclsKey;
+import org.opendaylight.yangtools.concepts.Builder;
+import org.opendaylight.yangtools.yang.binding.DataObject;
+import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
+
+public abstract class AbstractVppAclCustomizerTest extends ListReaderCustomizerTest<VppAcls, VppAclsKey, VppAclsBuilder> {
+
+ protected static final String IF_NAME = "eth1";
+ protected static final int IF_ID = 1;
+ protected static final InstanceIdentifier<VppAcls> IID =
+ InstanceIdentifier.create(InterfacesState.class).child(Interface.class, new InterfaceKey(IF_NAME))
+ .augmentation(VppAclInterfaceStateAugmentation.class).child(Acl.class).child(Ingress.class)
+ .child(VppAcls.class);
+ protected static final String IFC_CTX_NAME = "interface-context";
+ protected static final String ACL_CTX_NAME = "standard-acl-context";
+
+ @Mock
+ protected FutureJVppAclFacade aclApi;
+
+ protected NamingContext interfaceContext = new NamingContext("iface", IFC_CTX_NAME);
+ protected NamingContext standardAclContext = new NamingContext("standardAcl", ACL_CTX_NAME);
+
+ protected AbstractVppAclCustomizerTest(final Class<? extends Builder<? extends DataObject>> parentBuilderClass) {
+ super(VppAcls.class, parentBuilderClass);
+ }
+
+ @Override
+ protected void setUp() throws Exception {
+ defineMapping(mappingContext, IF_NAME, IF_ID, IFC_CTX_NAME);
+ }
+
+ @Test
+ public void testGetAllIdsNoAclConfigured() throws ReadFailedException {
+ AclInterfaceListDetailsReplyDump reply = new AclInterfaceListDetailsReplyDump();
+ final AclInterfaceListDetails details = new AclInterfaceListDetails();
+ details.acls = new int[0];
+ reply.aclInterfaceListDetails.add(details);
+ when(aclApi.aclInterfaceListDump(any())).thenReturn(future(reply));
+ assertTrue(getCustomizer().getAllIds(IID, ctx).isEmpty());
+ }
+} \ No newline at end of file
diff --git a/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/VppEgressAclCustomizerTest.java b/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/VppEgressAclCustomizerTest.java
new file mode 100644
index 000000000..b82a42896
--- /dev/null
+++ b/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/VppEgressAclCustomizerTest.java
@@ -0,0 +1,63 @@
+/*
+ * Copyright (c) 2016 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.read;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertTrue;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.when;
+
+import io.fd.honeycomb.translate.read.ReadFailedException;
+import io.fd.vpp.jvpp.acl.dto.AclInterfaceListDetails;
+import io.fd.vpp.jvpp.acl.dto.AclInterfaceListDetailsReplyDump;
+import org.junit.Test;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang._interface.acl.rev161214._interface.acl.attributes.acl.EgressBuilder;
+
+public class VppEgressAclCustomizerTest extends AbstractVppAclCustomizerTest {
+
+ public VppEgressAclCustomizerTest() {
+ super(EgressBuilder.class);
+ }
+
+ @Override
+ protected VppAclCustomizer initCustomizer() {
+ return new VppAclCustomizer(aclApi, interfaceContext, standardAclContext, false);
+ }
+
+ @Test
+ public void testGetAllIdsNoOutputAclConfigured() throws ReadFailedException {
+ AclInterfaceListDetailsReplyDump reply = new AclInterfaceListDetailsReplyDump();
+ final AclInterfaceListDetails details = new AclInterfaceListDetails();
+ details.acls = new int[1];
+ details.nInput = 1;
+ reply.aclInterfaceListDetails.add(details);
+ when(aclApi.aclInterfaceListDump(any())).thenReturn(future(reply));
+ assertTrue(getCustomizer().getAllIds(IID, ctx).isEmpty());
+ }
+
+ @Test
+ public void testGetAllIds() throws ReadFailedException {
+ AclInterfaceListDetailsReplyDump reply = new AclInterfaceListDetailsReplyDump();
+ final AclInterfaceListDetails details = new AclInterfaceListDetails();
+ defineMapping(mappingContext, "acl3", 3, ACL_CTX_NAME);
+ details.acls = new int[]{1,2,3};
+ details.nInput = 2;
+ reply.aclInterfaceListDetails.add(details);
+ when(aclApi.aclInterfaceListDump(any())).thenReturn(future(reply));
+ assertEquals(details.acls.length - details.nInput, getCustomizer().getAllIds(IID, ctx).size());
+ }
+} \ No newline at end of file
diff --git a/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/VppIngressAclCustomizerTest.java b/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/VppIngressAclCustomizerTest.java
new file mode 100644
index 000000000..773a9fa54
--- /dev/null
+++ b/acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/VppIngressAclCustomizerTest.java
@@ -0,0 +1,64 @@
+/*
+ * Copyright (c) 2016 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.read;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertTrue;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.when;
+
+import io.fd.honeycomb.translate.read.ReadFailedException;
+import io.fd.vpp.jvpp.acl.dto.AclInterfaceListDetails;
+import io.fd.vpp.jvpp.acl.dto.AclInterfaceListDetailsReplyDump;
+import org.junit.Test;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang._interface.acl.rev161214._interface.acl.attributes.acl.IngressBuilder;
+
+public class VppIngressAclCustomizerTest extends AbstractVppAclCustomizerTest {
+
+ public VppIngressAclCustomizerTest() {
+ super(IngressBuilder.class);
+ }
+
+ @Override
+ protected VppAclCustomizer initCustomizer() {
+ return new VppAclCustomizer(aclApi, interfaceContext, standardAclContext, true);
+ }
+
+ @Test
+ public void testGetAllIdsNoInputAclConfigured() throws ReadFailedException {
+ AclInterfaceListDetailsReplyDump reply = new AclInterfaceListDetailsReplyDump();
+ final AclInterfaceListDetails details = new AclInterfaceListDetails();
+ details.acls = new int[1];
+ details.nInput = 0;
+ reply.aclInterfaceListDetails.add(details);
+ when(aclApi.aclInterfaceListDump(any())).thenReturn(future(reply));
+ assertTrue(getCustomizer().getAllIds(IID, ctx).isEmpty());
+ }
+
+ @Test
+ public void testGetAllIds() throws ReadFailedException {
+ AclInterfaceListDetailsReplyDump reply = new AclInterfaceListDetailsReplyDump();
+ final AclInterfaceListDetails details = new AclInterfaceListDetails();
+ defineMapping(mappingContext, "acl1", 1, ACL_CTX_NAME);
+ defineMapping(mappingContext, "acl2", 2, ACL_CTX_NAME);
+ details.acls = new int[]{1,2,3};
+ details.nInput = 2;
+ reply.aclInterfaceListDetails.add(details);
+ when(aclApi.aclInterfaceListDump(any())).thenReturn(future(reply));
+ assertEquals(details.nInput, getCustomizer().getAllIds(IID, ctx).size());
+ }
+} \ No newline at end of file