----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9996/#review18099 -----------------------------------------------------------
Ship it! src/tests/allocator_tests.cpp <https://reviews.apache.org/r/9996/#comment38194> pull this down to line #408, to be closer to where it is used. - Vinod Kone On March 18, 2013, 9:40 p.m., Thomas Marshall wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9996/ > ----------------------------------------------------------- > > (Updated March 18, 2013, 9:40 p.m.) > > > Review request for mesos, Benjamin Hindman, Vinod Kone, and Ben Mahler. > > > Description > ------- > > In the ResourcesUnused test, a framework receives an offer and then launches > a task with it that uses less than the entire offer, to check that the unused > resources are being returned and reoffered correctly. Since we don't care > that the task is actually launched by the executor, just that the master > processes the task launching, the test was written under the assumption that > it would exit before the executor was registered and received the launchTask > call. This doesn't always happen, though, so the test was sometimes failing. > > This patch adds in expectations on the executor for the registered and > launchTask calls, though since we still don't care whether or not they > actually happen they are both cardinality AtMost(1). > > > This addresses bug MESOS-378. > https://issues.apache.org/jira/browse/MESOS-378 > > > Diffs > ----- > > src/tests/allocator_tests.cpp 4ae8bfd > > Diff: https://reviews.apache.org/r/9996/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Thomas Marshall > >
