> On Jan. 31, 2017, 11:38 p.m., Vinod Kone wrote: > > The description says "agent validation tests" but this file is master > > validation tests. Are you going to fix agent validation tests as well?
Ah sorry, that was just some crossed wires in my brain. There aren't any such occurrences in the agent validation tests; I checked :) I can do a sweep of the rest of the tests though; it'll be simple enough to update everything. - Greg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56139/#review163739 ----------------------------------------------------------- On Feb. 1, 2017, midnight, Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56139/ > ----------------------------------------------------------- > > (Updated Feb. 1, 2017, midnight) > > > Review request for mesos and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > This patch removes calls to `.Times(1)` from `EXPECT_CALL` > invocations in the master validation tests because it is > redundant. > > > Diffs > ----- > > src/tests/master_validation_tests.cpp > edb57407e08cdbd8fbf10a9e1493cab3b4979bb8 > > Diff: https://reviews.apache.org/r/56139/diff/ > > > Testing > ------- > > `bin/mesos-tests.sh --gtest_filter="*Validation*"` > > > Thanks, > > Greg Mann > >
