commit 23b307ec8a004c98b981a5caec64d025fe156b28
author bnc <bnc@google.com> Fri Jul 19 05:15:07 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Fri Jul 19 05:15:40 2019 -0700
tree 56677a3fa0dcb67d6cf9d4e3405b67c9949f9e93
parent 2d0148292d4fffb2e267bee5698c8657ef0b106f

Set maximum allowed push id to large value in QuicSpdyClientSessionTest.TooManyPushPromises.

This test fails in Chromium with "excessive output", blocking QUICHE merge.
Without a large maximum allowed push id, a large number of gMock warnings are
printed about uninteresting CloseConnection() calls.  The test passes because
MockQuicSpdySession::CloseConnection() is mocked out, but Chromium does not
stomach the output volume.

gfe-relnote: n/a, test-only change.
PiperOrigin-RevId: 258948786
Change-Id: I8c06baa8314bc1153fa5d4168fa99e181163df2a

quic/core/http/quic_spdy_client_session_test.cc

diff --git a/quic/core/http/quic_spdy_client_session_test.cc b/quic/core/http/quic_spdy_client_session_test.cc
index dfa3031..070acdf 100644
--- a/quic/core/http/quic_spdy_client_session_test.cc
+++ b/quic/core/http/quic_spdy_client_session_test.cc
@@ -926,6 +926,8 @@
       session_.get(), QuicMakeUnique<QuicSpdyClientStream>(
                           stream_id, session_.get(), BIDIRECTIONAL));
 
+  session_->set_max_allowed_push_id(kMaxQuicStreamId);
+
   EXPECT_CALL(*connection_, OnStreamReset(_, QUIC_REFUSED_STREAM));
 
   for (size_t promise_count = 0; promise_count <= session_->get_max_promises();