GBP ofoverlay.sf test improvements
[groupbasedpolicy.git] / renderers / ofoverlay / src / test / java / org / opendaylight / groupbasedpolicy / renderer / ofoverlay / sf / ChainActionTest.java
index 9fde7121efe5f03ad5f0558296cd9f3c5e5aea0e..a2ae75bd942dc5c186c94c05b337a01de47173d7 100755 (executable)
@@ -8,11 +8,12 @@
 
 package org.opendaylight.groupbasedpolicy.renderer.ofoverlay.sf;
 
+import static org.junit.Assert.assertEquals;
 import static org.mockito.Matchers.any;
+import static org.mockito.Matchers.eq;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
-import java.util.Arrays;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.HashSet;
@@ -20,6 +21,8 @@ import java.util.List;
 import java.util.Map;
 import java.util.Set;
 
+import com.google.common.base.Optional;
+import com.google.common.util.concurrent.CheckedFuture;
 import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
@@ -35,13 +38,19 @@ import org.opendaylight.groupbasedpolicy.renderer.ofoverlay.OfWriter;
 import org.opendaylight.groupbasedpolicy.renderer.ofoverlay.endpoint.EndpointManager;
 import org.opendaylight.groupbasedpolicy.renderer.ofoverlay.flow.OrdinalFactory.EndpointFwdCtxOrdinals;
 import org.opendaylight.groupbasedpolicy.renderer.ofoverlay.mapper.policyenforcer.NetworkElements;
+import org.opendaylight.yang.gen.v1.urn.cisco.params.xml.ns.yang.sfc.common.rev151017.RspName;
 import org.opendaylight.yang.gen.v1.urn.cisco.params.xml.ns.yang.sfc.common.rev151017.SfcName;
 import org.opendaylight.yang.gen.v1.urn.cisco.params.xml.ns.yang.sfc.common.rev151017.SfpName;
 import org.opendaylight.yang.gen.v1.urn.cisco.params.xml.ns.yang.sfc.rsp.rev140701.rendered.service.paths.RenderedServicePath;
+import org.opendaylight.yang.gen.v1.urn.cisco.params.xml.ns.yang.sfc.rsp.rev140701.rendered.service.paths.RenderedServicePathBuilder;
 import org.opendaylight.yang.gen.v1.urn.cisco.params.xml.ns.yang.sfc.sfp.rev140701.service.function.paths.ServiceFunctionPath;
+import org.opendaylight.yang.gen.v1.urn.cisco.params.xml.ns.yang.sfc.sfp.rev140701.service.function.paths.ServiceFunctionPathBuilder;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev100924.MacAddress;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.action.types.rev131112.action.list.ActionBuilder;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.common.rev140421.L2BridgeDomainId;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.common.rev140421.TenantId;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.endpoint.rev140421.endpoints.Endpoint;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.endpoint.rev140421.endpoints.EndpointBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.policy.rev140421.HasDirection.Direction;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.policy.rev140421.tenants.tenant.policy.subject.feature.instances.ActionInstance;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeId;
@@ -50,9 +59,6 @@ import org.powermock.api.mockito.PowerMockito;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.junit4.PowerMockRunner;
 
-import com.google.common.base.Optional;
-import com.google.common.util.concurrent.CheckedFuture;
-
 @RunWith(PowerMockRunner.class)
 @PrepareForTest(ChainAction.class)
 public class ChainActionTest {
@@ -72,16 +78,19 @@ public class ChainActionTest {
     private Endpoint endpoint;
     private EndpointManager endpointManager;
     private EgKey egKey;
-    private TenantId tenant = new TenantId("e09a2308-6ffa-40af-92a2-69f54b2cf3e4");
+    private TenantId tenantId = new TenantId("e09a2308-6ffa-40af-92a2-69f54b2cf3e4");
 
     @SuppressWarnings("unchecked")
     @Before
-    public void initialise() throws Exception {
+    public void init() throws Exception {
         chainAction = new ChainAction();
 
-        sfcPath = mock(ServiceFunctionPath.class);
-        when(sfcPath.getName()).thenReturn(new SfpName("sfcPathName"));
-        when(sfcPath.isSymmetric()).thenReturn(true);
+        sfcPath = new ServiceFunctionPathBuilder().setName(new SfpName("sfcPathName")).setSymmetric(true).build();
+        rsp = new RenderedServicePathBuilder().setName(new RspName("rspName")).build();
+        endpoint = new EndpointBuilder().setL2Context(new L2BridgeDomainId("L2context"))
+            .setMacAddress(new MacAddress("01:23:45:67:89:ab"))
+            .setTenant(tenantId)
+            .build();
 
         ctx = mock(OfContext.class);
         dataBroker = mock(DataBroker.class);
@@ -92,7 +101,6 @@ public class ChainActionTest {
         when(rTx.read(any(LogicalDatastoreType.class), any(InstanceIdentifier.class))).thenReturn(checkedFuture);
         optRsp = mock(Optional.class);
         when(checkedFuture.checkedGet()).thenReturn(optRsp);
-        rsp = mock(RenderedServicePath.class);
         when(optRsp.isPresent()).thenReturn(true).thenReturn(false);
         when(optRsp.get()).thenReturn(rsp);
 
@@ -102,36 +110,35 @@ public class ChainActionTest {
         endpointFwdCtxOrdinals = mock(EndpointFwdCtxOrdinals.class);
         when(netElements.getSrcEpOrdinals()).thenReturn(endpointFwdCtxOrdinals);
         when(netElements.getDstEpOrdinals()).thenReturn(endpointFwdCtxOrdinals);
-        endpoint = mock(Endpoint.class);
         when(netElements.getSrcEp()).thenReturn(endpoint);
-        when(netElements.getSrcEp().getTenant()).thenReturn(tenant);
 
         endpointManager = mock(EndpointManager.class);
         when(ctx.getEndpointManager()).thenReturn(endpointManager);
         egKey = mock(EgKey.class);
-        Set<EgKey> keysForEndpoint = new HashSet<EgKey>();
+        Set<EgKey> keysForEndpoint = new HashSet<>();
         keysForEndpoint.add(egKey);
         when(endpointManager.getEgKeysForEndpoint(any(Endpoint.class))).thenReturn(keysForEndpoint);
     }
 
     @Test
-    public void staticTest() {
-        Assert.assertNotNull(chainAction.getId());
-        Assert.assertNotNull(chainAction.getActionDef());
+    public void testGetters() {
+        assertEquals(ChainActionDefinition.ID, chainAction.getId());
+        assertEquals(ChainActionDefinition.DEFINITION, chainAction.getActionDef());
     }
 
     @Test
-    public void updateActionTestDirectionOutOpendaylightSfcNull() {
+    public void testUpdateAction_DirectionOut_OpendaylightSfcNull() {
         ActionBuilder actionBuilder = mock(ActionBuilder.class);
-        List<ActionBuilder> actions = Arrays.asList(actionBuilder);
-        Map<String, Object> params = new HashMap<String, Object>();
+        List<ActionBuilder> actions = Collections.singletonList(actionBuilder);
+        Map<String, Object> params = new HashMap<>();
         String chainName = "chainName";
         params.put(ChainActionDefinition.SFC_CHAIN_NAME, chainName);
-        Integer order = Integer.valueOf(0);
+        Integer order = 0;
         OfWriter ofWriter = mock(OfWriter.class);
 
         PowerMockito.mockStatic(ChainAction.class);
-        when(ChainAction.getSfcPath(new SfcName(chainName))).thenReturn(sfcPath);
+        SfcName sfcName = new SfcName(chainName);
+        when(ChainAction.getSfcPath(eq(sfcName))).thenReturn(sfcPath);
         chainAction.setResolvedSymmetricChains(Collections.singletonList(chainName));
 
         List<ActionBuilder> result =
@@ -140,10 +147,10 @@ public class ChainActionTest {
     }
 
     @Test
-    public void updateActionTestParamsNull() {
+    public void testUpdateAction_ParamsNull() {
         ActionBuilder actionBuilder = mock(ActionBuilder.class);
-        List<ActionBuilder> actions = Arrays.asList(actionBuilder);
-        Integer order = Integer.valueOf(0);
+        List<ActionBuilder> actions = Collections.singletonList(actionBuilder);
+        Integer order = 0;
         OfWriter ofWriter = mock(OfWriter.class);
 
         List<ActionBuilder> result =
@@ -152,12 +159,12 @@ public class ChainActionTest {
     }
 
     @Test
-    public void updateActionTestChainNameNull() {
+    public void testUpdateAction_ChainNameNull() {
         ActionBuilder actionBuilder = mock(ActionBuilder.class);
-        List<ActionBuilder> actions = Arrays.asList(actionBuilder);
-        Map<String, Object> params = new HashMap<String, Object>();
+        List<ActionBuilder> actions = Collections.singletonList(actionBuilder);
+        Map<String, Object> params = new HashMap<>();
         params.put(ChainActionDefinition.SFC_CHAIN_NAME, null);
-        Integer order = Integer.valueOf(0);
+        Integer order = 0;
         NetworkElements netElements = mock(NetworkElements.class);
         OfWriter ofWriter = mock(OfWriter.class);
 
@@ -168,18 +175,19 @@ public class ChainActionTest {
     }
 
     @Test
-    public void updateActionTest() {
+    public void testUpdateAction() {
         ActionBuilder actionBuilder = mock(ActionBuilder.class);
-        List<ActionBuilder> actions = Arrays.asList(actionBuilder);
-        Map<String, Object> params = new HashMap<String, Object>();
+        List<ActionBuilder> actions = Collections.singletonList(actionBuilder);
+        Map<String, Object> params = new HashMap<>();
         String chainName = "chainName";
         params.put(ChainActionDefinition.SFC_CHAIN_NAME, chainName);
-        Integer order = Integer.valueOf(0);
+        Integer order = 0;
         OfWriter ofWriter = mock(OfWriter.class);
 
+        ServiceFunctionPath sfcPathNameNull = new ServiceFunctionPathBuilder().setName(null).setSymmetric(true).build();
+
         PowerMockito.mockStatic(ChainAction.class);
-        when(ChainAction.getSfcPath(new SfcName(chainName))).thenReturn(sfcPath);
-        when(sfcPath.getName()).thenReturn(null);
+        when(ChainAction.getSfcPath(new SfcName(chainName))).thenReturn(sfcPathNameNull);
 
         List<ActionBuilder> result =
                 chainAction.updateAction(actions, params, order, netElements, ofWriter, ctx, Direction.Out);
@@ -187,8 +195,8 @@ public class ChainActionTest {
     }
 
     @Test
-    public void isValidTestParameterValueNull() {
-         ActionInstance actionInstance = mock(ActionInstance.class);
-         Assert.assertFalse(chainAction.validate(actionInstance).isValid());
+    public void testIsValid_ParameterValueNull() {
+        ActionInstance actionInstance = mock(ActionInstance.class);
+        Assert.assertFalse(chainAction.validate(actionInstance).isValid());
     }
 }