From d5e1e3a563807561caa0b64c414332d85a31f216 Mon Sep 17 00:00:00 2001 From: Marek Gradzki Date: Wed, 21 Dec 2016 11:00:09 +0100 Subject: Fix interface read when alcs are not configured Change-Id: Id780d9f503901d551f5a9d5693ca0a28cf596a5c Signed-off-by: Marek Gradzki --- .../io/fd/hc2vpp/acl/read/VppAclCustomizer.java | 14 +++- .../acl/read/AbstractVppAclCustomizerTest.java | 79 ++++++++++++++++++++++ .../acl/read/VppEgressAclCustomizerTest.java | 63 +++++++++++++++++ .../acl/read/VppIngressAclCustomizerTest.java | 64 ++++++++++++++++++ 4 files changed, 218 insertions(+), 2 deletions(-) create mode 100644 acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/AbstractVppAclCustomizerTest.java create mode 100644 acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/VppEgressAclCustomizerTest.java create mode 100644 acl/acl-impl/src/test/java/io/fd/hc2vpp/acl/read/VppIngressAclCustomizerTest.java 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 { + + protected static final String IF_NAME = "eth1"; + protected static final int IF_ID = 1; + protected static final InstanceIdentifier 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> 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 -- cgit 1.2.3-korg