commit ca53ac6b003ef1d0bd630fd3074fe3ef38967c07
author Steve DiBenedetto <dibenede@cs.colostate.edu> Thu Mar 27 19:58:40 2014 -0600
committer Alex Afanasyev <alexander.afanasyev@ucla.edu> Sun Apr 06 10:54:20 2014 -0700
tree a6d4663091e999bf33d3bcb8abb8e209734b91a1
parent b07788af059a6b4128049fe97ddf3fcb6126a5b0

face, mgmt: improve face creation failure and bad URI handling

Prevent creation of faces to endpoints owned by NFD instance

Prevent creation of UDP broadcast faces (255.255.255.255 and
those belonging to machine's interfaces) in response to
face creation command

Sanity check configuration file port numbers in face manager

refs: #1414, #1427

Change-Id: Ia3f0a9337f3d97c34388773eab05bc39ad6dd804

tests/face/tcp.cpp

diff --git a/tests/face/tcp.cpp b/tests/face/tcp.cpp
index 6bbe4ba..b71de48 100644
--- a/tests/face/tcp.cpp
+++ b/tests/face/tcp.cpp
@@ -161,13 +161,12 @@
   std::list< shared_ptr<Face> > faces;
 };
 
-
 BOOST_FIXTURE_TEST_CASE(EndToEnd4, EndToEndFixture)
 {
-  TcpFactory factory;
+  TcpFactory factory1;
 
-  shared_ptr<TcpChannel> channel1 = factory.createChannel("127.0.0.1", "20070");
-  factory.createChannel("127.0.0.1", "20071");
+  shared_ptr<TcpChannel> channel1 = factory1.createChannel("127.0.0.1", "20070");
+  factory1.createChannel("127.0.0.1", "20071");
 
   BOOST_CHECK_EQUAL(channel1->isListening(), false);
 
@@ -176,9 +175,14 @@
 
   BOOST_CHECK_EQUAL(channel1->isListening(), true);
 
-  factory.createFace(FaceUri("tcp://127.0.0.1:20070"),
-                     bind(&EndToEndFixture::channel2_onFaceCreated, this, _1),
-                     bind(&EndToEndFixture::channel2_onConnectFailed, this, _1));
+  TcpFactory factory2;
+
+  shared_ptr<TcpChannel> channel2 = factory2.createChannel("127.0.0.2", "20070");
+  factory2.createChannel("127.0.0.2", "20071");
+
+  factory2.createFace(FaceUri("tcp://127.0.0.1:20070"),
+                      bind(&EndToEndFixture::channel2_onFaceCreated, this, _1),
+                      bind(&EndToEndFixture::channel2_onConnectFailed, this, _1));
 
   BOOST_CHECK_MESSAGE(limitedIo.run(2, time::seconds(10)) == LimitedIo::EXCEED_OPS,
                       "TcpChannel error: cannot connect or cannot accept connection");
@@ -253,17 +257,21 @@
 
 BOOST_FIXTURE_TEST_CASE(EndToEnd6, EndToEndFixture)
 {
-  TcpFactory factory;
+  TcpFactory factory1;
 
-  shared_ptr<TcpChannel> channel1 = factory.createChannel("::1", "20070");
-  shared_ptr<TcpChannel> channel2 = factory.createChannel("::1", "20071");
+  shared_ptr<TcpChannel> channel1 = factory1.createChannel("::1", "20070");
+  shared_ptr<TcpChannel> channel2 = factory1.createChannel("::1", "20071");
 
   channel1->listen(bind(&EndToEndFixture::channel1_onFaceCreated,   this, _1),
                    bind(&EndToEndFixture::channel1_onConnectFailed, this, _1));
 
-  factory.createFace(FaceUri("tcp://[::1]:20070"),
-                     bind(&EndToEndFixture::channel2_onFaceCreated, this, _1),
-                     bind(&EndToEndFixture::channel2_onConnectFailed, this, _1));
+  TcpFactory factory2;
+
+  factory2.createChannel("::2", "20070");
+
+  factory2.createFace(FaceUri("tcp://[::1]:20070"),
+                      bind(&EndToEndFixture::channel2_onFaceCreated, this, _1),
+                      bind(&EndToEndFixture::channel2_onConnectFailed, this, _1));
 
   BOOST_CHECK_MESSAGE(limitedIo.run(2, time::seconds(10)) == LimitedIo::EXCEED_OPS,
                       "TcpChannel error: cannot connect or cannot accept connection");

tests/mgmt/face-manager.cpp

diff --git a/tests/mgmt/face-manager.cpp b/tests/mgmt/face-manager.cpp
index 4d6830c..ebae6f7 100644
--- a/tests/mgmt/face-manager.cpp
+++ b/tests/mgmt/face-manager.cpp
@@ -1465,7 +1465,7 @@
 
   getFace()->onReceiveData +=
     bind(&FaceFixture::validateControlResponse, this, _1,
-         command->getName(), 400, "Failed to create face");
+         command->getName(), 408, "unit-test-reason");
 
   onConnectFailed(command->getName(), "unit-test-reason");