Преглед на файлове

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.
Craig Tiller преди 10 години
родител
ревизия
8cca4694ee
променени са 1 файла, в които са добавени 2 реда и са изтрити 1 реда
  1. 2 1
      test/cpp/end2end/client_crash_test.cc

+ 2 - 1
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());
 }