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.
kinsonyang · Jun 22, 2015 · 8cca469 · 8cca469
1 parent 8222b19
commit 8cca469
Showing 1 changed file with 2 additions and 1 deletion.
diff --git 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());
 }