commit cf9c6bb8d7239c324805d939899fa2a7cbcc3ac0
author Junxiao Shi <git@mail1.yoursunny.com> Wed Jul 27 02:18:19 2016 +0000
committer Junxiao Shi <git@mail1.yoursunny.com> Wed Jul 27 02:18:19 2016 +0000
tree b769c3565c38c294e82385712d67268ff2a0ea05
parent 19d7a572480c71c1bffefbbcab3e9cb6d2ea6c10

tests: fix "child has exited" error in Face integrated tests

refs #3684

Change-Id: I9f48c481b6d9a19ec9c90a8d220dfbbcbe41418a

tests/integrated/face.cpp

diff --git a/tests/integrated/face.cpp b/tests/integrated/face.cpp
index 68cc91e..3ef7297 100644
--- a/tests/integrated/face.cpp
+++ b/tests/integrated/face.cpp
@@ -31,6 +31,8 @@
 #include "boost-test.hpp"
 #include "key-chain-fixture.hpp"
 
+#include <stdio.h>
+
 namespace ndn {
 namespace tests {
 
@@ -313,17 +315,34 @@
 {
 public:
   void
-  checkPrefix(bool doesExist)
+  checkPrefix(bool shouldExist)
   {
-    int result = std::system("nfd-status -r | grep /Hello/World >/dev/null");
+    // Boost.Test fails if a child process exits with non-zero.
+    // http://stackoverflow.com/q/5325202
+    std::string output = this->executeCommand("nfd-status -r | grep /Hello/World || true");
 
-    if (doesExist) {
-      BOOST_CHECK_EQUAL(result, 0);
+    if (shouldExist) {
+      BOOST_CHECK_NE(output.size(), 0);
     }
     else {
-      BOOST_CHECK_NE(result, 0);
+      BOOST_CHECK_EQUAL(output.size(), 0);
     }
   }
+
+protected:
+  std::string
+  executeCommand(const std::string& cmd)
+  {
+    std::string output;
+    char buf[256];
+    FILE* pipe = popen(cmd.c_str(), "r");
+    BOOST_REQUIRE_MESSAGE(pipe != nullptr, "cannot execute '" << cmd << "'");
+    while (fgets(buf, sizeof(buf), pipe) != nullptr) {
+      output += buf;
+    }
+    pclose(pipe);
+    return output;
+  }
 };
 
 BOOST_FIXTURE_TEST_CASE(RegisterUnregisterPrefix, FacesFixture2)