From baac146246ad8fb099a10f23f5cc66976c2d5a91 Mon Sep 17 00:00:00 2001 From: Yang Chi Date: Fri, 4 Jun 2021 13:47:03 -0700 Subject: [PATCH] Fix WriteFlowControl test Summary: The assignment is wrong Reviewed By: mjoras Differential Revision: D28847255 fbshipit-source-id: 9133d88c953f74c8caac303f20c74bc089574d11 --- quic/api/test/QuicTransportTest.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/quic/api/test/QuicTransportTest.cpp b/quic/api/test/QuicTransportTest.cpp index 1275d8eee..dc0092628 100644 --- a/quic/api/test/QuicTransportTest.cpp +++ b/quic/api/test/QuicTransportTest.cpp @@ -597,7 +597,7 @@ TEST_F(QuicTransportTest, WriteFlowControl) { loopForWrites(); EXPECT_EQ(conn.outstandings.packets.size(), 1); - auto& packet = + const auto& packet = getFirstOutstandingPacket(conn, PacketNumberSpace::AppData)->packet; bool blockedFound = false; bool dataBlockedFound = false; @@ -654,9 +654,10 @@ TEST_F(QuicTransportTest, WriteFlowControl) { // Verify that there is one Data Blocked frame emitted. EXPECT_EQ(conn.outstandings.packets.size(), num_outstandings + 2); - packet = getLastOutstandingPacket(conn, PacketNumberSpace::AppData)->packet; + const auto& packet2 = + getLastOutstandingPacket(conn, PacketNumberSpace::AppData)->packet; dataBlockedFound = false; - for (auto& frame : packet.frames) { + for (auto& frame : packet2.frames) { auto dataBlocked = frame.asDataBlockedFrame(); if (!dataBlocked) { continue;