> On May 3, 2016, 10:32 p.m., Michael Park wrote: > > src/tests/persistent_volume_tests.cpp, line 68 > > <https://reviews.apache.org/r/41613/diff/2/?file=1180931#file1180931line68> > > > > Perhaps we should consider pulling this out to `src/tests/mesos.hpp`.
Good call! I'll move this constant into mesos.hpp. - Greg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41613/#review131557 ----------------------------------------------------------- On Jan. 5, 2016, 2:49 a.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41613/ > ----------------------------------------------------------- > > (Updated Jan. 5, 2016, 2:49 a.m.) > > > Review request for mesos, Alexander Rukletsov, Jie Yu, and Michael Park. > > > Repository: mesos > > > Description > ------- > > Added `DEFAULT_ROLE` constant to persistent volume tests. > > > Diffs > ----- > > src/tests/persistent_volume_tests.cpp > 2fb57814b2805bc76981d1877603a1a033f29289 > > Diff: https://reviews.apache.org/r/41613/diff/ > > > Testing > ------- > > `GTEST_FILTER="PersistentVolumeTest*" bin/mesos-tests.sh` > > > Thanks, > > Greg Mann > >
