From 8cca4694ee03f7b5df191211a488c6a7438e64e3 Mon Sep 17 00:00:00 2001 From: Craig Tiller Date: Mon, 22 Jun 2015 13:50:59 -0700 Subject: [PATCH] Fix flaky test Assertation could be false if the server responds quickly enough (it's unlikely to). Removing the assert does not change the nature of the test. --- test/cpp/end2end/client_crash_test.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/cpp/end2end/client_crash_test.cc b/test/cpp/end2end/client_crash_test.cc index 7876e8dee3a..645226f3754 100644 --- a/test/cpp/end2end/client_crash_test.cc +++ b/test/cpp/end2end/client_crash_test.cc @@ -140,7 +140,8 @@ TEST_F(CrashTest, KillAfterWrite) { KillServer(); - EXPECT_FALSE(stream->Read(&response)); + // This may succeed or fail depending on how quick the server was + stream->Read(&response); EXPECT_FALSE(stream->Finish().ok()); }