You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ti...@apache.org on 2015/12/20 23:40:50 UTC

mesos git commit: Added expectation for call to mocked function.

Repository: mesos
Updated Branches:
  refs/heads/master d03d59079 -> 9a46d715d


Added expectation for call to mocked function.

We use a mocked executor here and need to register an expectation on a
call to its mocked `shutdown` method, like is already done at other
sites in this test suite.

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

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


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

Branch: refs/heads/master
Commit: 9a46d715d9d81802726eb5b7d04c9645161ee2f9
Parents: d03d590
Author: Benjamin Bannier <be...@mesosphere.io>
Authored: Sun Dec 20 23:23:30 2015 +0100
Committer: Till Toenshoff <to...@me.com>
Committed: Sun Dec 20 23:23:30 2015 +0100

----------------------------------------------------------------------
 src/tests/slave_tests.cpp | 3 +++
 1 file changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/9a46d715/src/tests/slave_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/slave_tests.cpp b/src/tests/slave_tests.cpp
index 109d31c..90d56b9 100644
--- a/src/tests/slave_tests.cpp
+++ b/src/tests/slave_tests.cpp
@@ -2132,6 +2132,9 @@ TEST_F(SlaveTest, ContainerizerUsageFailure)
   ASSERT_EQ(1, usage.get().executors_size());
   EXPECT_FALSE(usage.get().executors(0).has_statistics());
 
+  EXPECT_CALL(exec, shutdown(_))
+    .Times(AtMost(1));
+
   driver.stop();
   driver.join();