diff options
| author |  Craig Tiller <ctiller@google.com> | 2015-06-22 13:50:59 -0700 |
|---|---|---|
| committer | Craig Tiller <ctiller@google.com> | 2015-06-22 13:50:59 -0700 |
| commit | 8cca4694ee03f7b5df191211a488c6a7438e64e3 (patch) | |
| tree | 2d6eb5fbfffe119f121e165dbb0577eda725b71f /test/cpp/end2end/client_crash_test.cc | |
| parent | 8222b19eb14273df8124cb5f46f629836e72f6ad (diff) | |
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.
Diffstat (limited to 'test/cpp/end2end/client_crash_test.cc')
| -rw-r--r-- | test/cpp/end2end/client_crash_test.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/test/cpp/end2end/client_crash_test.cc b/test/cpp/end2end/client_crash_test.cc index 7876e8dee3..645226f375 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()); } |