message: Add unit test for sending NewConnectionIdFrame
diff: @@ -1184,6 +1184,34 @@ TEST_F(QuicTransportTest, ResendPathResponseOnLoss) {
  [&](auto&) { return false; }));
  }
  +TEST_F(QuicTransportTest, SendNewConnectionIdFrame) {
  + EXPECT_CALL(*socket_, write(_, _)).WillOnce(Invoke(bufLength));
  + auto& conn = transport_->getConnectionState();
  + NewConnectionIdFrame newConnId(
  + 1, ConnectionId({2, 4, 2, 3}), StatelessResetToken());
  + conn.pendingEvents.frames.push_back(newConnId);
  + transport_->updateWriteLooper(true);
  + loopForWrites();
  +
  + EXPECT_TRUE(conn.pendingEvents.frames.empty());
  + EXPECT_EQ(1, transport_->getConnectionState().outstandingPackets.size());
  + auto packet =
  + getLastOutstandingPacket(
  + transport_->getConnectionState(), PacketNumberSpace::AppData)
  + ->packet;
  + bool foundNewConnectionId = false;
  + for (auto& simpleFrame : all_frames<QuicSimpleFrame>(packet.frames)) {
  + folly::variant_match(
  + simpleFrame,
  + [&](const NewConnectionIdFrame& frame) {
  + EXPECT_EQ(frame, newConnId);
  + foundNewConnectionId = true;
  + },
  + [&](auto&) {});
  + }
  + EXPECT_TRUE(foundNewConnectionId);
  +}
  +
  TEST_F(QuicTransportTest, NonWritableStreamAPI) {
  auto streamId = transport_->createBidirectionalStream().value();
  auto buf = buildRandomInputData(20);
  