> On May 29, 2014, 5:13 p.m., Vinod Kone wrote: > > src/tests/slave_tests.cpp, lines 753-768 > > <https://reviews.apache.org/r/21968/diff/6/?file=597869#file597869line753> > > > > Use the LaunchTasks action to simplify this. See allocator_tests and > > gc_tests for examples.
Thanks! I copied that from the gc_tests. > On May 29, 2014, 5:13 p.m., Vinod Kone wrote: > > src/tests/slave_tests.cpp, line 815 > > <https://reviews.apache.org/r/21968/diff/6/?file=597869#file597869line815> > > > > I dont think you need this abstraction. Just advance the clock and make > > sure there is no registration message sent. This is where I feel confused, because I found that I could not catch a ReregisterSlaveMessage, instead, I could only catch the SlaveReregisteredMessage. - Yifan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21968/#review44265 ----------------------------------------------------------- On May 29, 2014, 10:40 p.m., Yifan Gu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21968/ > ----------------------------------------------------------- > > (Updated May 29, 2014, 10:40 p.m.) > > > Review request for mesos, Adam B and Niklas Nielsen. > > > Bugs: MESOS-878 > https://issues.apache.org/jira/browse/MESOS-878 > > > Repository: mesos-git > > > Description > ------- > > Added a sentence to check the slave's state in doReliableRegistration() to > make sure that the function returns when the slave's state is TERMINATING. > Also, to write the test, I need to make sure there's no additional > doReliableRegistration() calls, so I added NO_FUTURE_PROTOBUFS() and > NO_FUTURE_MESSAGES() to make sure there's no more SlaveReregisteredMessages. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/gmock.hpp daba7e3 > src/slave/slave.cpp 7fbedb1 > src/tests/mesos.hpp 1b0f358 > src/tests/slave_tests.cpp 80fe3cf > > Diff: https://reviews.apache.org/r/21968/diff/ > > > Testing > ------- > > Slave_tests.cpp:TerminatingSlaveDoesNotReregister. > > > Thanks, > > Yifan Gu > >
