You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by me...@apache.org on 2016/02/24 09:01:31 UTC

mesos git commit: Made sure that spawned processes terminated before leaving tests.

Repository: mesos
Updated Branches:
  refs/heads/master eaa9f14a7 -> 0786e6636


Made sure that spawned processes terminated before leaving tests.

Review: https://reviews.apache.org/r/43893/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/0786e663
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/0786e663
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/0786e663

Branch: refs/heads/master
Commit: 0786e6636576e12f5168d0923cfbe000403eabea
Parents: eaa9f14
Author: Benjamin Bannier <be...@mesosphere.io>
Authored: Wed Feb 24 00:00:54 2016 -0800
Committer: Adam B <ad...@mesosphere.io>
Committed: Wed Feb 24 00:00:54 2016 -0800

----------------------------------------------------------------------
 src/tests/container_logger_tests.cpp | 2 ++
 1 file changed, 2 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/0786e663/src/tests/container_logger_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/container_logger_tests.cpp b/src/tests/container_logger_tests.cpp
index 6e2b8c1..00f4129 100644
--- a/src/tests/container_logger_tests.cpp
+++ b/src/tests/container_logger_tests.cpp
@@ -115,6 +115,7 @@ TEST_F(ContainerLoggerTest, MesosContainerizerRecover)
   // try to reap the PID.
   Try<Subprocess> s = subprocess("exit 0");
   ASSERT_SOME(s);
+  AWAIT(s->status());
 
   // Construct a mock `SlaveState`.
   ExecutorID executorId;
@@ -196,6 +197,7 @@ TEST_F(ContainerLoggerTest, ROOT_DOCKER_ContainerizerRecover)
   // try to reap the PID.
   Try<Subprocess> s = subprocess("exit 0");
   ASSERT_SOME(s);
+  AWAIT(s->status());
 
   // Construct a mock `SlaveState`.
   ExecutorID executorId;