-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43613/
-----------------------------------------------------------
(Updated March 2, 2016, 1:45 p.m.)
Review request for mesos, Benjamin Hindman, Bernd Mathiske, Artem Harutyunyan,
and Michael Park.
Changes
-------
Re-introduce the `Try<>` in the factory method's return types. This will
enforce correct gtest error propagation.
Bugs: MESOS-4633 and MESOS-4634
https://issues.apache.org/jira/browse/MESOS-4633
https://issues.apache.org/jira/browse/MESOS-4634
Repository: mesos
Description
-------
Major rewrite of the `tests/cluster` helpers. This strongly ties the scope of
test objects to the test body.
Changes the `Cluster` class into two RAII objects (`Master` and `Slave`). The
`Slave` object performs cleanup originally found in `cluster::Slave::stop`.
`cluster::Master::start` and `cluster::Slave::start` were changed to factory
methods.
Diffs (updated)
-----
src/tests/cluster.hpp 99a785ab0d4ee1a1e745202d2551de58a7631a85
src/tests/cluster.cpp 084fb1ce37a315c561c4587c4761c870f54c8625
Diff: https://reviews.apache.org/r/43613/diff/
Testing
-------
Tests are run at the end of this review chain.
Thanks,
Joseph Wu
