From: Flavio Fernandes Date: Fri, 17 Apr 2015 18:40:11 +0000 (+0000) Subject: Merge "Fix for TerminationPointCreateCommand to insert a termination point, if not... X-Git-Tag: release/beryllium-sr2~652 X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=commitdiff_plain;h=79573aa5cbb3291ae9a6a61ac9fb11eafdedd617;hp=56fa87914d2dd9918ce88e853056c7101b3240c8;p=netvirt.git Merge "Fix for TerminationPointCreateCommand to insert a termination point, if not present already in operational store" --- diff --git a/openstack/net-virt-providers/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/services/InboundNatServiceTest.java b/openstack/net-virt-providers/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/services/InboundNatServiceTest.java new file mode 100644 index 0000000000..d377b8e5a0 --- /dev/null +++ b/openstack/net-virt-providers/src/test/java/org/opendaylight/ovsdb/openstack/netvirt/providers/openflow13/services/InboundNatServiceTest.java @@ -0,0 +1,111 @@ +/* + * Copyright (c) 2015 Inocybe 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.ovsdb.openstack.netvirt.providers.openflow13.services; + +import static org.junit.Assert.assertEquals; +import static org.mockito.Matchers.any; +import static org.mockito.Matchers.anyBoolean; +import static org.mockito.Mockito.mock; +import static org.mockito.Mockito.times; +import static org.mockito.Mockito.verify; +import static org.mockito.Mockito.when; + +import java.net.InetAddress; + +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; +import org.mockito.InjectMocks; +import org.mockito.Mock; +import org.mockito.runners.MockitoJUnitRunner; +import org.opendaylight.controller.md.sal.binding.api.DataBroker; +import org.opendaylight.controller.md.sal.binding.api.ReadWriteTransaction; +import org.opendaylight.controller.md.sal.binding.api.WriteTransaction; +import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType; +import org.opendaylight.controller.md.sal.common.api.data.TransactionCommitFailedException; +import org.opendaylight.ovsdb.openstack.netvirt.api.Action; +import org.opendaylight.ovsdb.openstack.netvirt.providers.openflow13.MdsalConsumer; +import org.opendaylight.ovsdb.openstack.netvirt.providers.openflow13.PipelineOrchestrator; +import org.opendaylight.ovsdb.openstack.netvirt.providers.openflow13.Service; +import org.opendaylight.ovsdb.plugin.api.Status; +import org.opendaylight.ovsdb.plugin.api.StatusCode; +import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.Node; +import org.opendaylight.yangtools.yang.binding.InstanceIdentifier; + +import com.google.common.util.concurrent.CheckedFuture; + +/** + * Unit test for {@link InboundNatService} + */ +@RunWith(MockitoJUnitRunner.class) +public class InboundNatServiceTest { + + @InjectMocks private InboundNatService inboundNatService = new InboundNatService(Service.ARP_RESPONDER); + + @Mock private MdsalConsumer mdsalConsumer; + @Mock private PipelineOrchestrator orchestrator; + + @Mock private ReadWriteTransaction readWriteTransaction; + @Mock private WriteTransaction writeTransaction; + @Mock private CheckedFuture commitFuture; + + private static final String HOST_ADDRESS = "127.0.0.1"; + private static final String HOST_ADDRESS_PREFIX = "127.0.0.1/32"; + + @Before + public void setUp() { + when(readWriteTransaction.submit()).thenReturn(commitFuture); + when(writeTransaction.submit()).thenReturn(commitFuture); + + DataBroker dataBroker = mock(DataBroker.class); + when(dataBroker.newReadWriteTransaction()).thenReturn(readWriteTransaction); + when(dataBroker.newWriteOnlyTransaction()).thenReturn(writeTransaction); + + when(mdsalConsumer.getDataBroker()).thenReturn(dataBroker); + + when(orchestrator.getNextServiceInPipeline(any(Service.class))).thenReturn(Service.ARP_RESPONDER); + } + + /** + * Test method {@link InboundNatService#programIpRewriteRule(Node, Long, String, InetAddress, InetAddress, Action)} + */ + @Test + public void testProgramIpRewriteRule() throws Exception { + InetAddress rewriteAddress = mock(InetAddress.class); + when(rewriteAddress.getHostAddress()).thenReturn(HOST_ADDRESS); + InetAddress matchAddress = mock(InetAddress.class); + when(matchAddress.getHostAddress()).thenReturn(HOST_ADDRESS); + + assertEquals("Error, did not return the expected StatusCode", new Status(StatusCode.SUCCESS), inboundNatService.programIpRewriteRule(mock(Node.class), Long.valueOf(123), "2", matchAddress, rewriteAddress, Action.ADD)); + verify(readWriteTransaction, times(2)).put(any(LogicalDatastoreType.class), any(InstanceIdentifier.class), any(Node.class), anyBoolean()); + verify(readWriteTransaction, times(1)).submit(); + verify(commitFuture, times(1)).get(); + + assertEquals("Error, did not return the expected StatusCode", new Status(StatusCode.SUCCESS), inboundNatService.programIpRewriteRule(mock(Node.class), Long.valueOf(123), "2", matchAddress, rewriteAddress, Action.DELETE)); + verify(writeTransaction, times(1)).delete(any(LogicalDatastoreType.class), any(InstanceIdentifier.class)); + verify(readWriteTransaction, times(1)).submit(); + verify(commitFuture, times(2)).get(); // 1 + 1 above + } + + /** + * Test method {@link InboundNatService#programIpRewriteExclusion(Node, Long, String, String, Action)} + */ + @Test + public void testProgramIpRewriteExclusion() throws Exception { + assertEquals("Error, did not return the expected StatusCode", new Status(StatusCode.SUCCESS), inboundNatService.programIpRewriteExclusion(mock(Node.class), Long.valueOf(123), "2", HOST_ADDRESS_PREFIX, Action.ADD)); + verify(readWriteTransaction, times(2)).put(any(LogicalDatastoreType.class), any(InstanceIdentifier.class), any(Node.class), anyBoolean()); + verify(readWriteTransaction, times(1)).submit(); + verify(commitFuture, times(1)).get(); + + assertEquals("Error, did not return the expected StatusCode", new Status(StatusCode.SUCCESS), inboundNatService.programIpRewriteExclusion(mock(Node.class), Long.valueOf(123), "2", HOST_ADDRESS_PREFIX, Action.DELETE)); + verify(writeTransaction, times(1)).delete(any(LogicalDatastoreType.class), any(InstanceIdentifier.class)); + verify(readWriteTransaction, times(1)).submit(); + verify(commitFuture, times(2)).get(); // 1 + 1 above + } +}