Fix some test expectation regarding SendControlFrame().
Before IETF QUIC, only 1 control frame will be sent for a stream reset. But IETF requires both a RST and STOP_SENDING. The tests that explicitly expect SendControlFrame() to be called once is making the wrong assumption.
The reason they still pass is the first write was buffered so the second write won't trigger.
Remove the explicit Times(1) to avoid confusion.
gfe-relnote: n/a. test only.
PiperOrigin-RevId: 276736424
Change-Id: Iffa8de79c51e28355ef4aa8381ae049948e40b67
diff --git a/quic/core/quic_stream_test.cc b/quic/core/quic_stream_test.cc
index 7aa69e8..d209454 100644
--- a/quic/core/quic_stream_test.cc
+++ b/quic/core/quic_stream_test.cc
@@ -910,7 +910,9 @@
EXPECT_TRUE(session_->HasUnackedStreamData());
EXPECT_CALL(*connection_,
OnStreamReset(stream_->id(), QUIC_STREAM_CANCELLED));
- EXPECT_CALL(*connection_, SendControlFrame(_)).Times(1);
+ EXPECT_CALL(*connection_, SendControlFrame(_))
+ .Times(AtLeast(1))
+ .WillRepeatedly(Invoke(&ClearControlFrame));
EXPECT_CALL(*session_, SendRstStream(stream_->id(), QUIC_STREAM_CANCELLED, 9))
.WillOnce(InvokeWithoutArgs([this]() {
return QuicSessionPeer::SendRstStreamInner(
