-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65679/
-----------------------------------------------------------
(Updated April 5, 2018, 3:35 p.m.)
Review request for mesos, Chun-Hung Hsiao and Greg Mann.
Changes
-------
Added additional fix for `RemoveExecutorUponFailedLaunch`
Bugs: MESOS-8611
https://issues.apache.org/jira/browse/MESOS-8611
Repository: mesos
Description
-------
Directly invoking unmock calls in the test process can potentially
cause races with the real mock slave process. It is more robust to
dispatch the unmock calls to the real mock slave process.
Also added several mock expectations to avoid "uninteresting mock
call" test warnings.
Diffs (updated)
-----
src/tests/slave_tests.cpp 0d7d9726ee39e4c588ea5eb8a54a73fbe1249353
Diff: https://reviews.apache.org/r/65679/diff/10/
Changes: https://reviews.apache.org/r/65679/diff/9-10/
Testing
-------
`./bin/mesos-tests.sh --gtest_filter=*SlaveTest* --gtest_repeat=-1
--gtest_break_on_failure` runs forever :)
Thanks,
Meng Zhu
