commit a52bf7d09b48ffe1bff9527e9b5c381800bb80a1
author andrewsbrown <andrew.brown@intel.com> Mon Apr 06 13:51:53 2015 -0700
committer andrewsbrown <andrew.brown@intel.com> Mon Apr 06 13:51:53 2015 -0700
tree a9ed276b4477d1fe5e414510ec9f6e870295d281
parent aada3939b1e36361627ace3984c0005401ef7157

Ensure prefix registration connects the underlying transport

src/main/java/com/intel/jndn/mock/MockFace.java

diff --git a/src/main/java/com/intel/jndn/mock/MockFace.java b/src/main/java/com/intel/jndn/mock/MockFace.java
index e1a62de..6a9a11a 100644
--- a/src/main/java/com/intel/jndn/mock/MockFace.java
+++ b/src/main/java/com/intel/jndn/mock/MockFace.java
@@ -497,6 +497,10 @@
   @Override
   public long registerPrefix(Name prefix, OnInterest onInterest, OnRegisterFailed onRegisterFailed,
           ForwardingFlags flags, WireFormat wireFormat) throws IOException, net.named_data.jndn.security.SecurityException {
+    // since we don't send an Interest, ensure the transport is connected
+    if (!getTransport().getIsConnected())
+      getTransport().connect(node_.getConnectionInfo(), node_);
+    
     lastRegisteredId++;
     handlerMap.put(lastRegisteredId, new MockOnInterestHandler(prefix, onInterest, flags));
     return lastRegisteredId;

src/test/java/com/intel/jndn/mock/MockFaceTest.java

diff --git a/src/test/java/com/intel/jndn/mock/MockFaceTest.java b/src/test/java/com/intel/jndn/mock/MockFaceTest.java
index 8c51a6e..6bbbdda 100644
--- a/src/test/java/com/intel/jndn/mock/MockFaceTest.java
+++ b/src/test/java/com/intel/jndn/mock/MockFaceTest.java
@@ -21,6 +21,7 @@
 import net.named_data.jndn.OnData;
 import net.named_data.jndn.OnInterest;
 import net.named_data.jndn.encoding.EncodingException;
+import net.named_data.jndn.security.SecurityException;
 import net.named_data.jndn.transport.Transport;
 import net.named_data.jndn.util.Blob;
 import org.junit.Test;
@@ -138,4 +139,18 @@
       return count;
     }
   }
+
+  /**
+   * Ensure registering a prefix connects the underlying transport
+   *
+   * @throws IOException
+   * @throws SecurityException
+   */
+  @Test
+  public void testRegistrationConnectsTransport() throws IOException, SecurityException {
+    MockFace face = new MockFace();
+    assertFalse(face.getTransport().getIsConnected());
+    face.registerPrefix(new Name("/fake/prefix"), null, null);
+    assertTrue(face.getTransport().getIsConnected());
+  }
 }