From: Timotej Kubas <tkubas@cisco.com>
Date: Fri, 27 Feb 2015 12:57:10 +0000 (+0100)
Subject: BUG-2382: OutputActionBuilder.setOutputNodeConnector
X-Git-Tag: release/lithium~651^2
X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=commitdiff_plain;h=1c1d7c715b8fb8197ca4e72272cf741f6ed0cf53;hp=0ae38ade4f6f6bd17d0d9364272699e5664263ec;p=openflowplugin.git

BUG-2382: OutputActionBuilder.setOutputNodeConnector

 - use of OutputActionBuilder.setOutputNodeConnector in openflowplugin changed
   to proper form

Change-Id: I930aba68293a7498d0b2be61bf9a2c155637427a
Signed-off-by: Timotej Kubas <tkubas@cisco.com>
---

diff --git a/legacy/sal-compatibility/src/main/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/NodeMapping.java b/legacy/sal-compatibility/src/main/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/NodeMapping.java
index 6002427e12..b58d46a0b9 100644
--- a/legacy/sal-compatibility/src/main/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/NodeMapping.java
+++ b/legacy/sal-compatibility/src/main/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/NodeMapping.java
@@ -10,6 +10,7 @@ package org.opendaylight.openflowplugin.legacy.sal.compatibility;
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Objects;
 import com.google.common.base.Preconditions;
+
 import org.opendaylight.controller.sal.common.util.Arguments;
 import org.opendaylight.controller.sal.core.AdvertisedBandwidth;
 import org.opendaylight.controller.sal.core.Bandwidth;
@@ -205,7 +206,7 @@ public final class NodeMapping {
             return org.opendaylight.controller.sal.core.NodeConnector.SPECIALNODECONNECTORID;
         }
 
-        String nodeConnectorIdStripped = ALL_CHARS_TO_COLON.matcher(nodeConnectorId.getValue()).replaceFirst("");
+        String nodeConnectorIdStripped = stripToColon(nodeConnectorId.getValue());
 
         if (NUMBERS_ONLY.matcher(nodeConnectorIdStripped).matches()) {
             Short nodeConnectorIdVal = null;
@@ -219,6 +220,10 @@ public final class NodeMapping {
         return nodeConnectorIdStripped;
     }
 
+    public static String stripToColon(final String elementId) {
+        return ALL_CHARS_TO_COLON.matcher(elementId).replaceFirst("");
+    }
+
     public static NodeId toAdNodeId(final NodeConnectorId nodeConnectorId) {
         NodeId nodeId = null;
         if (nodeConnectorId != null) {
diff --git a/legacy/sal-compatibility/src/main/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/ToSalConversionsUtils.java b/legacy/sal-compatibility/src/main/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/ToSalConversionsUtils.java
index 8af6c7713f..b7ec91596c 100644
--- a/legacy/sal-compatibility/src/main/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/ToSalConversionsUtils.java
+++ b/legacy/sal-compatibility/src/main/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/ToSalConversionsUtils.java
@@ -216,8 +216,9 @@ public class ToSalConversionsUtils {
                     }else{
                         type = type + ":";
                     }
-                    Uri fullNodeConnector = new Uri(type+node.getID()+":"+nodeConnector.getValue());
-                        targetAction.add(new Output(fromNodeConnectorRef(fullNodeConnector, node)));
+                    Uri fullNodeConnector = new Uri(type+node.getID()+":"
+                            +NodeMapping.stripToColon(nodeConnector.getValue()));
+                    targetAction.add(new Output(fromNodeConnectorRef(fullNodeConnector, node)));
                 }
             } else if (sourceAction instanceof PopMplsActionCase) {
                 // TODO: define maping
diff --git a/legacy/sal-compatibility/src/test/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/test/NodeMappingTest.java b/legacy/sal-compatibility/src/test/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/test/NodeMappingTest.java
index 3a5ce2c006..babf62f861 100644
--- a/legacy/sal-compatibility/src/test/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/test/NodeMappingTest.java
+++ b/legacy/sal-compatibility/src/test/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/test/NodeMappingTest.java
@@ -208,7 +208,20 @@ public class NodeMappingTest {
         NodeKey nodeKey = NodeMapping.toNodeKey(aDNode);
         Assert.assertEquals("openflow:42", nodeKey.getId().getValue());
     }
-
+    
+    /**
+     * Test method for
+     * {@link org.opendaylight.openflowplugin.legacy.sal.compatibility.NodeMapping#stripToColon(NodeConnectorId)}
+     * .
+     * @throws ConstructionException
+     */
+    @Test
+    public void testStripToColon() {
+        Assert.assertEquals("1", NodeMapping.stripToColon("openflow:42:1"));
+        Assert.assertEquals("1", NodeMapping.stripToColon("openflow:1"));
+        Assert.assertEquals("1", NodeMapping.stripToColon("1"));
+    }
+ 
     /**
      * @param nodeId
      * @param portId
diff --git a/legacy/sal-compatibility/src/test/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/test/TestToSalConversionsUtils.java b/legacy/sal-compatibility/src/test/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/test/TestToSalConversionsUtils.java
index 5f2bc94403..3ea3dbf2c9 100644
--- a/legacy/sal-compatibility/src/test/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/test/TestToSalConversionsUtils.java
+++ b/legacy/sal-compatibility/src/test/java/org/opendaylight/openflowplugin/legacy/sal/compatibility/test/TestToSalConversionsUtils.java
@@ -207,7 +207,7 @@ public class TestToSalConversionsUtils {
         List<Action> odActions = new ArrayList<>();
 
         OutputActionBuilder outputActionBuilder = new OutputActionBuilder();
-        outputActionBuilder.setOutputNodeConnector(new Uri("CONTROLLER"));
+        outputActionBuilder.setOutputNodeConnector(new Uri("openflow:42:CONTROLLER"));
         OutputActionCaseBuilder outputActionCaseBuilder = new OutputActionCaseBuilder();
         outputActionCaseBuilder.setOutputAction(outputActionBuilder.build());
         odActions.add(new ActionBuilder().setAction(outputActionCaseBuilder.build()).build());
@@ -600,7 +600,7 @@ public class TestToSalConversionsUtils {
 
     private void prepareActionOutput(OutputActionCaseBuilder wrapper) {
         OutputActionBuilder outputActionBuilder = new OutputActionBuilder();
-        outputActionBuilder.setOutputNodeConnector(new Uri("1"));
+        outputActionBuilder.setOutputNodeConnector(new Uri("openflow:1:1"));
         wrapper.setOutputAction(outputActionBuilder.build());
     }