Rename test: TestDeserializeIgnoresInvalidPayloadTypesInRid.
The test was recently updated not to remove the RID, let's update the
name to reflect this too.
Bug: webrtc:362277533
Change-Id: I4eebd6953ee98e6f4406bb961a88ea89746594a3
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/378880
Commit-Queue: Henrik Boström <hbos@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#43994}
diff --git a/pc/webrtc_sdp_unittest.cc b/pc/webrtc_sdp_unittest.cc
index 8978c7d..822d909 100644
--- a/pc/webrtc_sdp_unittest.cc
+++ b/pc/webrtc_sdp_unittest.cc
@@ -4585,11 +4585,11 @@
CompareRidDescriptionIds(rids, {"5"});
}
-// Removes RIDs that specify a different format than the m= section.
-TEST_F(WebRtcSdpTest, TestDeserializeRemovesRidsWithInvalidCodec) {
+// Ignores codecs from RIDs where the PTs are missing from the m= section.
+TEST_F(WebRtcSdpTest, TestDeserializeIgnoresInvalidPayloadTypesInRid) {
std::string sdp = kUnifiedPlanSdpFullStringNoSsrc;
- sdp += "a=rid:1 send pt=121,120\r\n"; // Should remove 121 and keep RID.
- sdp += "a=rid:2 send pt=121\r\n"; // Should remove 121 and keep RID.
+ sdp += "a=rid:1 send pt=121,120\r\n"; // Should remove 121 and keep 120.
+ sdp += "a=rid:2 send pt=121\r\n"; // Should remove 121.
sdp += "a=simulcast:send 1;2\r\n";
JsepSessionDescription output(kDummyType);
SdpParseError error;