Chromium Code Reviews| Index: util/test/multiprocess_exec_test.cc |
| diff --git a/util/test/multiprocess_exec_test.cc b/util/test/multiprocess_exec_test.cc |
| index c30843ed51dda7c7dec49d4ceeaa985837eaad66..855f84729232e9a92abd94563b4fed4185ae0ed7 100644 |
| --- a/util/test/multiprocess_exec_test.cc |
| +++ b/util/test/multiprocess_exec_test.cc |
| @@ -36,7 +36,10 @@ class TestMultiprocessExec final : public MultiprocessExec { |
| char c = 'z'; |
| CheckedWriteFD(WritePipeFD(), &c, 1); |
|
Robert Sesek
2014/12/15 19:41:18
Change this to a LoggingX?
|
| - CheckedReadFD(ReadPipeFD(), &c, 1); |
| + // Use LoggingReadFD() instead of CheckedReadFD() for the first read so that |
| + // the test can fail gracefully with a gtest assertion if the child does not |
| + // execute properly. |
| + ASSERT_TRUE(LoggingReadFD(ReadPipeFD(), &c, 1)); |
| EXPECT_EQ('Z', c); |
| } |