From 57d13105e5cf77bf0289edab1757fcfa9c63ab87 Mon Sep 17 00:00:00 2001 From: Minyue Li Date: Thu, 19 Aug 2021 14:30:45 +0200 Subject: [PATCH] A test to verify counter on discarded packet upon reordering. Bug: webrtc:8199 Change-Id: Ia8c545e4ae0ff98c22031b633d1bca4ba58e9e7b Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/229380 Commit-Queue: Minyue Li Reviewed-by: Jakob Ivarsson Cr-Commit-Position: refs/heads/master@{#34804} --- modules/audio_coding/neteq/neteq_impl_unittest.cc | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/modules/audio_coding/neteq/neteq_impl_unittest.cc b/modules/audio_coding/neteq/neteq_impl_unittest.cc index b0fee47ade..154a48ec4c 100644 --- a/modules/audio_coding/neteq/neteq_impl_unittest.cc +++ b/modules/audio_coding/neteq/neteq_impl_unittest.cc @@ -550,6 +550,7 @@ TEST_F(NetEqImplTest, VerifyTimestampPropagation) { TEST_F(NetEqImplTest, ReorderedPacket) { UseNoMocks(); + // Create a mock decoder object. MockAudioDecoder mock_decoder; @@ -648,6 +649,9 @@ TEST_F(NetEqImplTest, ReorderedPacket) { // out-of-order packet should have been discarded. EXPECT_TRUE(packet_buffer_->Empty()); + // NetEq `discarded_primary_packets` should capture this packet discard. + EXPECT_EQ(1u, neteq_->GetOperationsAndState().discarded_primary_packets); + // Verify `output.packet_infos_`. Expect to only see the second packet. ASSERT_THAT(output.packet_infos_, SizeIs(1)); {