Skip to content

Commit

Permalink
fix test after changes to folly::io::Cursor
Browse files Browse the repository at this point in the history
Summary: {D62460851} changes the return type of `folly::io::Cursor::peek()` and should have changed all uses in the same commit, but this use slipped through.

Reviewed By: DenisYaroshevskiy

Differential Revision: D62641264

fbshipit-source-id: 1ea01d2f7a15fbf23627594e710087825bbc2ee6
  • Loading branch information
yfeldblum authored and facebook-github-bot committed Sep 13, 2024
1 parent e901a4d commit efdffec
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions quic/server/test/QuicServerTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1917,9 +1917,9 @@ void QuicServerWorkerTakeoverTest::testPacketForwarding(
EXPECT_EQ(protocotVersion, 0x0000001);
uint16_t addrLen = (cursor.readBE<uint16_t>());
struct sockaddr* sockaddr = nullptr;
std::pair<const uint8_t*, size_t> addrData = cursor.peek();
EXPECT_GE(addrData.second, addrLen);
sockaddr = (struct sockaddr*)addrData.first;
auto addrData = cursor.peek();
EXPECT_GE(addrData.size(), addrLen);
sockaddr = (struct sockaddr*)addrData.data();
cursor.skip(addrLen);

folly::SocketAddress actualClient;
Expand Down

0 comments on commit efdffec

Please sign in to comment.