commit 9aef115356ce48848790dd64a0f2d2ed9f55149b
author QUICHE team <quiche-dev@google.com> Thu Jan 14 15:56:32 2021 -0800
committer Copybara-Service <copybara-worker@google.com> Thu Jan 14 15:57:14 2021 -0800
tree 9d326d6d360ff1e88bb830a487475a28304b6ca7
parent 1c18357e3cff0f0de8e6c218a0a901a7c2a9da9f

Fixes test flakiness by forcing the test client to reconnect after fuzzed requests.

$ blaze test //gfe/gfe2/e2e:fuzz_internal_urls_test_autobahn_tcp_with_autobahn_tcp_backends --runs_per_test=1000 --test_arg=--gfe2_reloadable_flag_autobahn_boost_tokens_sooner=true

//gfe/gfe2/e2e:fuzz_internal_urls_test_autobahn_tcp_with_autobahn_tcp_backends PASSED in 101.0s
  Stats over 1000 runs: max = 101.0s, min = 26.4s, avg = 46.0s, dev = 11.4s

http://sponge2/45f3aa80-7887-4343-8b71-982b4daebac9

PiperOrigin-RevId: 351894605
Change-Id: I4538f1a7031bab7ace1db5cc77301c69fdf0d6b3

quic/test_tools/quic_test_client.cc

diff --git a/quic/test_tools/quic_test_client.cc b/quic/test_tools/quic_test_client.cc
index eaec7eb..9dcfb42 100644
--- a/quic/test_tools/quic_test_client.cc
+++ b/quic/test_tools/quic_test_client.cc
@@ -658,7 +658,9 @@
 
 void QuicTestClient::Disconnect() {
   ClearPerConnectionState();
-  client_->Disconnect();
+  if (client_->initialized()) {
+    client_->Disconnect();
+  }
   connect_attempted_ = false;
 }