mgmt: respond Nack when query is invalid

refs #1993

Change-Id: I21e398cd7b2e562b7dc759a89f6e668960e72c41
diff --git a/tests/daemon/mgmt/face-manager.cpp b/tests/daemon/mgmt/face-manager.cpp
index 966844a..2b2a15e 100644
--- a/tests/daemon/mgmt/face-manager.cpp
+++ b/tests/daemon/mgmt/face-manager.cpp
@@ -1813,23 +1813,23 @@
   BOOST_REQUIRE(m_finished);
 }
 
-//BOOST_FIXTURE_TEST_CASE(TestInvalidQueryFilter, FaceQueryListFixture)
-//{
-//  Name queryName("/localhost/nfd/faces/query");
-//  ndn::nfd::FaceStatus queryFilter;
-//  queryName.append(queryFilter.wireEncode());
-//
-//  shared_ptr<Interest> query(make_shared<Interest>(queryName));
-//
-//  shared_ptr<DummyLocalFace> face(make_shared<DummyLocalFace>());
-//  add(face);
-//
-//  m_face->onReceiveData +=
-//    bind(&FaceQueryStatusPublisherFixture::decodeNackBlock, this, _1);
-//
-//  m_manager.listQueriedFaces(*query);
-//  BOOST_REQUIRE(m_finished);
-//}
+BOOST_FIXTURE_TEST_CASE(TestInvalidQueryFilter, FaceQueryListFixture)
+{
+  Name queryName("/localhost/nfd/faces/query");
+  ndn::nfd::FaceStatus queryFilter;
+  queryName.append(queryFilter.wireEncode());
+
+  shared_ptr<Interest> query(make_shared<Interest>(queryName));
+
+  shared_ptr<DummyLocalFace> face(make_shared<DummyLocalFace>());
+  add(face);
+
+  m_face->onReceiveData +=
+    bind(&FaceQueryStatusPublisherFixture::decodeNackBlock, this, _1);
+
+  m_manager.listQueriedFaces(*query);
+  BOOST_REQUIRE(m_finished);
+}
 
 BOOST_AUTO_TEST_SUITE_END()
 
diff --git a/tests/daemon/mgmt/face-query-status-publisher-common.hpp b/tests/daemon/mgmt/face-query-status-publisher-common.hpp
index a0e4070..a274c52 100644
--- a/tests/daemon/mgmt/face-query-status-publisher-common.hpp
+++ b/tests/daemon/mgmt/face-query-status-publisher-common.hpp
@@ -133,13 +133,13 @@
     m_finished = true;
   }
 
-  //void
-  //decodeNackBlock(const Data& data)
-  //{
-  //  BOOST_REQUIRE_EQUAL(data.getContentType(), ndn::tlv::ContentType_Nack);
+  void
+  decodeNackBlock(const Data& data)
+  {
+    BOOST_REQUIRE_EQUAL(data.getContentType(), tlv::ContentType_Nack);
 
-  //  m_finished = true;
-  //}
+    m_finished = true;
+  }
 
 protected:
   Forwarder m_forwarder;