----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50737/#review151905 -----------------------------------------------------------
3rdparty/libprocess/src/tests/http_tests.cpp (line 257) <https://reviews.apache.org/r/50737/#comment220558> This one needs a `GetParam()`. - Joseph Wu On Oct. 7, 2016, 11:37 a.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50737/ > ----------------------------------------------------------- > > (Updated Oct. 7, 2016, 11:37 a.m.) > > > Review request for mesos, Anand Mazumdar, Joseph Wu, and Vinod Kone. > > > Bugs: MESOS-5966 > https://issues.apache.org/jira/browse/MESOS-5966 > > > Repository: mesos > > > Description > ------- > > This patch parametrizes the HTTP tests in libprocess so > that they run with SSL both enabled and disabled when > the library was compiled with SSL support. > > > Diffs > ----- > > 3rdparty/libprocess/src/tests/http_tests.cpp > 8d6c8c4507b93dd11f927bad2a8f0faeb957e388 > > Diff: https://reviews.apache.org/r/50737/diff/ > > > Testing > ------- > > Tested on OSX with `GTEST_REPEAT=100 GTEST_BREAK_ON_FAILURE=1 > GTEST_FILTER="*HTTPTest*" 3rdparty/libprocess/libprocess-tests` > > > Thanks, > > Greg Mann > >