You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by vi...@apache.org on 2014/06/26 05:55:33 UTC

git commit: Fixed DRFAllocatorTest.SameShareAllocations test.

Repository: mesos
Updated Branches:
  refs/heads/master 03284a323 -> 24602dfb7


Fixed DRFAllocatorTest.SameShareAllocations test.

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


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

Branch: refs/heads/master
Commit: 24602dfb7ec3b7dafd5cb2d7a60508eb30b5829d
Parents: 03284a3
Author: Vinod Kone <vi...@gmail.com>
Authored: Wed Jun 25 18:39:09 2014 -0700
Committer: Vinod Kone <vi...@gmail.com>
Committed: Wed Jun 25 18:41:08 2014 -0700

----------------------------------------------------------------------
 src/tests/allocator_tests.cpp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/24602dfb/src/tests/allocator_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/allocator_tests.cpp b/src/tests/allocator_tests.cpp
index a968ea3..42cb345 100644
--- a/src/tests/allocator_tests.cpp
+++ b/src/tests/allocator_tests.cpp
@@ -368,7 +368,11 @@ TEST_F(DRFAllocatorTest, SameShareAllocations)
   MesosSchedulerDriver driver2(
       &sched2, frameworkInfo2, master.get(), DEFAULT_CREDENTIAL);
 
-  EXPECT_CALL(allocator, frameworkAdded(_, _, _));
+  // We need to retire this expectation on the first match because
+  // framework1 can match this expectation first in which case
+  // framework2 should be able to match the expectation above.
+  EXPECT_CALL(allocator, frameworkAdded(_, _, _))
+    .RetiresOnSaturation();
 
   Future<Nothing> registered2;
   EXPECT_CALL(sched2, registered(_, _, _))