diff --git a/src/main/java/net/floodlightcontroller/core/types/NodePortTuple.java b/src/main/java/net/floodlightcontroller/core/types/NodePortTuple.java
index c289368b7e99a59899ad97a6aba105fe88462307..828c9bc5cb533911afd20b3a00df304fe21b7f14 100644
--- a/src/main/java/net/floodlightcontroller/core/types/NodePortTuple.java
+++ b/src/main/java/net/floodlightcontroller/core/types/NodePortTuple.java
@@ -32,8 +32,8 @@ import org.projectfloodlight.openflow.types.OFPort;
  */
 
 public class NodePortTuple implements Comparable<NodePortTuple> {
-    protected DatapathId nodeId; // switch DPID
-    protected OFPort portId; // switch port id
+    private DatapathId nodeId; // switch DPID
+    private OFPort portId; // switch port id
 
     /**
      * Creates a NodePortTuple
diff --git a/src/test/java/net/floodlightcontroller/topology/TopologyInstanceTest.java b/src/test/java/net/floodlightcontroller/topology/TopologyInstanceTest.java
index 4b542db1d8be6f6d66852cac6e8dbc2f4a78b10a..7488a8aecef496d8d7ee78adadfcd59cd803ec77 100644
--- a/src/test/java/net/floodlightcontroller/topology/TopologyInstanceTest.java
+++ b/src/test/java/net/floodlightcontroller/topology/TopologyInstanceTest.java
@@ -135,7 +135,7 @@ public class TopologyInstanceTest {
                 expected.add(npt);
             }
             TopologyInstance ti = topologyManager.getCurrentInstance(tunnelsEnabled);
-            Set<NodePortTuple> computed = ti.getBroadcastNodePortsInCluster(npt.nodeId);
+            Set<NodePortTuple> computed = ti.getBroadcastNodePortsInCluster(npt.getNodeId());
             log.info("computed: {}", computed);
             if (computed != null)
                 assertTrue(computed.equals(expected));