----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52284/#review160145 -----------------------------------------------------------
Fix it, then Ship it! src/tests/master_quota_tests.cpp (lines 420 - 422) <https://reviews.apache.org/r/52284/#comment231183> Let's move this up right after `SetForNonExistentRole`. src/tests/master_quota_tests.cpp (lines 447 - 448) <https://reviews.apache.org/r/52284/#comment231182> Let's rename the test not here but while updating it in https://reviews.apache.org/r/52103/ - Alexander Rukletsov On Nov. 11, 2016, 8:55 p.m., Zhitao Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52284/ > ----------------------------------------------------------- > > (Updated Nov. 11, 2016, 8:55 p.m.) > > > Review request for mesos, Alexander Rukletsov and Xiaojian Huang. > > > Bugs: MESOS-4292 > https://issues.apache.org/jira/browse/MESOS-4292 > > > Repository: mesos > > > Description > ------- > > Implement more quota validation tests. > > > Diffs > ----- > > src/tests/master_quota_tests.cpp 48be7406181646c8cc1d169b82a4a4ca71cdf03b > > Diff: https://reviews.apache.org/r/52284/diff/ > > > Testing > ------- > > > Thanks, > > Zhitao Li > >
