----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41962/#review114414 -----------------------------------------------------------
src/tests/container_logger_tests.cpp (lines 153 - 154) <https://reviews.apache.org/r/41962/#comment175251> Ugh, using something we passed in as `Owned` above? Have we still not cleaned up this pattern in the code base? - Benjamin Hindman On Jan. 12, 2016, 10:31 p.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41962/ > ----------------------------------------------------------- > > (Updated Jan. 12, 2016, 10:31 p.m.) > > > Review request for mesos, Benjamin Hindman, Gilbert Song, Artem Harutyunyan, > Jie Yu, and Timothy Chen. > > > Bugs: MESOS-4150 > https://issues.apache.org/jira/browse/MESOS-4150 > > > Repository: mesos > > > Description > ------- > > Adds two heavily-mocked tests for the Mesos containerizer and Docker > containerizer. Each checks that `ContainerLogger::recover` is called during > `Containerizer::recover`. > > > Diffs > ----- > > src/tests/container_logger_tests.cpp > c6b2e597517c74a55649287dc5ae5a3115f9a640 > src/tests/mesos.hpp 3d9ebc6c9dc3cd1be02dc3771fbd847386907fac > src/tests/mesos.cpp 365ebe8335c37bfdb983a5424d4c995fa9b76a22 > > Diff: https://reviews.apache.org/r/41962/diff/ > > > Testing > ------- > > make (OSX & Centos7) > > Tests are run in the next review. > > > Thanks, > > Joseph Wu > >
