zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add expected default values to video configuration tests.

Browse Source 
Bug: webrtc:9860
Change-Id: I772edf3c143eafaa97a37e596f3ea5a41f00b800
Reviewed-on: https://webrtc-review.googlesource.com/c/107676
Reviewed-by: Åsa Persson <asapersson@webrtc.org>
Commit-Queue: Benjamin Wright <benwright@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25344}
This commit is contained in:
Benjamin Wright
2018-10-23 23:32:54 -07:00
committed by Commit Bot
parent 44a262a6aa
commit 4cb4786d78
1 changed files with 11 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
video/end_to_end_tests/config_tests.cc
View File

@ -49,6 +49,12 @@ TEST_F(ConfigEndToEndTest, VerifyDefaultSendConfigParameters) {
<< "Enabling RTX requires rtpmap: rtx negotiation.";
EXPECT_TRUE(default_send_config.rtp.extensions.empty())
<< "Enabling RTP extensions require negotiation.";
EXPECT_EQ(nullptr, default_send_config.frame_encryptor)
<< "Enabling Frame Encryption requires a frame encryptor to be attached";
EXPECT_FALSE(
default_send_config.crypto_options.sframe.require_frame_encryption)
<< "Enabling Require Frame Encryption means an encryptor must be "
"attached";
VerifyEmptyNackConfig(default_send_config.rtp.nack);
VerifyEmptyUlpfecConfig(default_send_config.rtp.ulpfec);
@ -70,12 +76,16 @@ TEST_F(ConfigEndToEndTest, VerifyDefaultVideoReceiveConfigParameters) {
<< "Enabling RTX requires rtpmap: rtx negotiation.";
EXPECT_TRUE(default_receive_config.rtp.extensions.empty())
<< "Enabling RTP extensions require negotiation.";
VerifyEmptyNackConfig(default_receive_config.rtp.nack);
EXPECT_EQ(-1, default_receive_config.rtp.ulpfec_payload_type)
<< "Enabling ULPFEC requires rtpmap: ulpfec negotiation.";
EXPECT_EQ(-1, default_receive_config.rtp.red_payload_type)
<< "Enabling ULPFEC requires rtpmap: red negotiation.";
EXPECT_EQ(nullptr, default_receive_config.frame_decryptor)
<< "Enabling Frame Decryption requires a frame decryptor to be attached";
EXPECT_FALSE(
default_receive_config.crypto_options.sframe.require_frame_encryption)
<< "Enabling Require Frame Encryption means a decryptor must be attached";
}
TEST_F(ConfigEndToEndTest, VerifyDefaultFlexfecReceiveConfigParameters) {