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();