----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44084/#review127502 -----------------------------------------------------------
3rdparty/libprocess/3rdparty/stout/tests/os/filesystem_tests.cpp (lines 211 - 256) <https://reviews.apache.org/r/44084/#comment190836> Same as https://reviews.apache.org/r/45015/, I thought we concluded that actions such as `os::mkdir` and `os::touch` should be `ASSERT_*`, and the things that are being actually being tested here, i.e. `os::exists` should be `EXPECT_*`. - Michael Park On March 24, 2016, 2:59 p.m., Alex Clemmer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44084/ > ----------------------------------------------------------- > > (Updated March 24, 2016, 2:59 p.m.) > > > Review request for mesos, Alex Naparu, Daniel Pravat, Artem Harutyunyan, > Joris Van Remoortere, Michael Park, M Lawindi, and Yi Sun. > > > Repository: mesos > > > Description > ------- > > Windows: Forked os::rename(). > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/Makefile.am > 7bd4bfbc2ec5922879dcefddc12137336b11be52 > 3rdparty/libprocess/3rdparty/stout/include/stout/os.hpp > 79e30ca04c6d23f92e3a2f80fbe38ae63fde3520 > 3rdparty/libprocess/3rdparty/stout/include/stout/os/posix/rename.hpp > PRE-CREATION > 3rdparty/libprocess/3rdparty/stout/include/stout/os/rename.hpp > 7b2215208bcaa5613426d8ede9cab21d1a084631 > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/rename.hpp > PRE-CREATION > 3rdparty/libprocess/3rdparty/stout/tests/os/filesystem_tests.cpp > 4c30189bb8261ccfc699da0f31b8b1fd3e9b3c83 > > Diff: https://reviews.apache.org/r/44084/diff/ > > > Testing > ------- > > > Thanks, > > Alex Clemmer > >