> On Nov. 15, 2012, 11:10 p.m., Ben Mahler wrote: > > src/tests/allocator_tests.cpp, line 85 > > <https://reviews.apache.org/r/8055/diff/1/?file=189896#file189896line85> > > > > Let's still call this AllocatorTest? > > Looking at the primer that seems to be the convention: > > http://code.google.com/p/googletest/wiki/Primer
changed this to DRFAllocatorTest > On Nov. 15, 2012, 11:10 p.m., Ben Mahler wrote: > > src/tests/master_tests.cpp, line 1012 > > <https://reviews.apache.org/r/8055/diff/1/?file=189901#file189901line1012> > > > > ditto fixture name: > > > > s/WhitelistFixture/MasterWhitelistTest? Changed it to WhiteListTest > On Nov. 15, 2012, 11:10 p.m., Ben Mahler wrote: > > src/tests/resource_offers_tests.cpp, line 519 > > <https://reviews.apache.org/r/8055/diff/1/?file=189902#file189902line519> > > > > s/ResourceOffersTestFixture/ResourceOffersTest calling it MultipleExecutorsTest. > On Nov. 15, 2012, 11:10 p.m., Ben Mahler wrote: > > src/tests/utils.hpp, line 103 > > <https://reviews.apache.org/r/8055/diff/1/?file=189903#file189903line103> > > > > Too bad my CHECK_SOME is not submitted ;) some day. - Vinod ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8055/#review13438 ----------------------------------------------------------- On Nov. 14, 2012, 7:20 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8055/ > ----------------------------------------------------------- > > (Updated Nov. 14, 2012, 7:20 p.m.) > > > Review request for mesos, Benjamin Hindman and Ben Mahler. > > > Description > ------- > > I envisage this new test fixture to do a lot more than creating/deleting > working directories. But thats probably for another review. > > > Diffs > ----- > > src/tests/allocator_tests.cpp 44d72ad2b746b1b7032f8243817a6002100a7503 > src/tests/allocator_zookeeper_tests.cpp > 5f83dd71aa3cac1dd3727b65af087f651f961392 > src/tests/configurator_tests.cpp 91c01ad19c23a3f7b1e0e0c369e8d6d9fa534dc5 > src/tests/fault_tolerance_tests.cpp > 7c3d155479a4f7fda5c14f62909eea5f5a1a92df > src/tests/master_detector_tests.cpp > 5d09babc728d7922e47204290e86943179a34226 > src/tests/master_tests.cpp 948ab5dff34eeba1f3ce593a864ddf282c8b19ed > src/tests/resource_offers_tests.cpp > 5981191597e673c47be331420b71be66456694c6 > src/tests/utils.hpp cc1a81d07c6f23e3e2590c2df485f18d114cc6a6 > src/tests/zookeeper_test.hpp 2aa105fed170b2824040a79bd450129f06c4bc13 > src/tests/zookeeper_test.cpp 4bd6a293c177864b8767c876ccd91e7cbb0f84f0 > > Diff: https://reviews.apache.org/r/8055/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
