You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bm...@apache.org on 2014/01/17 22:07:09 UTC

git commit: Fixed the flaky GroupTest.RetryableErrors test.

Updated Branches:
  refs/heads/master ac8e9bbd3 -> 5b4a3a80e


Fixed the flaky GroupTest.RetryableErrors test.

From: Jiang Yan Xu <ya...@jxu.me>
Review: https://reviews.apache.org/r/16824


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

Branch: refs/heads/master
Commit: 5b4a3a80ed4618b4ddf4186bf4657150402ebe43
Parents: ac8e9bb
Author: Benjamin Mahler <bm...@twitter.com>
Authored: Fri Jan 17 12:59:01 2014 -0800
Committer: Benjamin Mahler <bm...@twitter.com>
Committed: Fri Jan 17 12:59:01 2014 -0800

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


http://git-wip-us.apache.org/repos/asf/mesos/blob/5b4a3a80/src/tests/group_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/group_tests.cpp b/src/tests/group_tests.cpp
index 957256e..ac1942b 100644
--- a/src/tests/group_tests.cpp
+++ b/src/tests/group_tests.cpp
@@ -366,7 +366,9 @@ TEST_F(GroupTest, RetryableErrors)
   session = group.session();
   AWAIT_READY(session);
   ASSERT_SOME(session.get());
+  connected = FUTURE_DISPATCH(_, &GroupProcess::connected);
   server->expireSession(session.get().get());
 
   AWAIT_READY(cancellation);
+  AWAIT_READY(connected);
 }