From 240e783d7f3ce2bf4a39d4ae44f56dc73c77ea23 Mon Sep 17 00:00:00 2001 From: Philipp Hancke Date: Tue, 8 Aug 2023 15:17:23 +0200 Subject: [PATCH] Stop using invalid payload type 200 in audio/red unit test and fix the follow-up mistake in the test BUG=None Change-Id: Id7a20769cc1d03dd8154564f948e8138ff8c4e74 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/315220 Commit-Queue: Henrik Lundin Reviewed-by: Henrik Lundin Cr-Commit-Position: refs/heads/main@{#40528} --- .../codecs/red/audio_encoder_copy_red_unittest.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/audio_coding/codecs/red/audio_encoder_copy_red_unittest.cc b/modules/audio_coding/codecs/red/audio_encoder_copy_red_unittest.cc index 20d85a1d15..e9b1b079ca 100644 --- a/modules/audio_coding/codecs/red/audio_encoder_copy_red_unittest.cc +++ b/modules/audio_coding/codecs/red/audio_encoder_copy_red_unittest.cc @@ -46,7 +46,7 @@ class AudioEncoderCopyRedTest : public ::testing::Test { timestamp_(4711), sample_rate_hz_(16000), num_audio_samples_10ms(sample_rate_hz_ / 100), - red_payload_type_(200) { + red_payload_type_(63) { AudioEncoderCopyRed::Config config; config.payload_type = red_payload_type_; config.speech_encoder = std::unique_ptr(mock_encoder_); @@ -475,7 +475,7 @@ TEST_F(AudioEncoderCopyRedTest, CheckRFC2198Header) { EXPECT_EQ(encoded_[2] & 0x3u, encoded_info_.redundant[1].encoded_bytes >> 8); EXPECT_EQ(encoded_[3], encoded_info_.redundant[1].encoded_bytes & 0xff); - EXPECT_EQ(encoded_[4], primary_payload_type | 0x80); + EXPECT_EQ(encoded_[4], primary_payload_type); timestamp_delta = encoded_info_.encoded_timestamp - encoded_info_.redundant[1].encoded_timestamp; }