admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Disable a couple flaky RampUpTest tests on Mac.

Browse Source 
Specifically, UpDownUpAudioVideoTransportSequenceNumberRtx
and AudioTransportSequenceNumber.

TBR=stefan@webrtc.org

Bug: webrtc:8878
Change-Id: I24ad65dc56068bb01c53d97511faadc22bd13c31
Reviewed-on: https://webrtc-review.googlesource.com/52161
Reviewed-by: Taylor Brandstetter <deadbeef@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#21992}
This commit is contained in:
Taylor Brandstetter 2018-02-12 15:16:02 -08:00
parent ef76e5a062
commit ab86e7ffe2
1 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
call/rampup_tests.cc
View File

@ -598,7 +598,15 @@ TEST_F(RampUpTest, DISABLED_UpDownUpTransportSequenceNumberPacketLoss) {
RunBaseTest(&test);
}
TEST_F(RampUpTest, UpDownUpAudioVideoTransportSequenceNumberRtx) {
// TODO(bugs.webrtc.org/8878)
#if defined(WEBRTC_MAC)
#define MAYBE_UpDownUpAudioVideoTransportSequenceNumberRtx \
DISABLED_UpDownUpAudioVideoTransportSequenceNumberRtx
#else
#define MAYBE_UpDownUpAudioVideoTransportSequenceNumberRtx \
UpDownUpAudioVideoTransportSequenceNumberRtx
#endif
TEST_F(RampUpTest, MAYBE_UpDownUpAudioVideoTransportSequenceNumberRtx) {
std::vector<int> loss_rates = {0, 0, 0, 0};
RampUpDownUpTester test(3, 1, 0, kStartBitrateBps,
RtpExtension::kTransportSequenceNumberUri, true,
@ -650,7 +658,13 @@ TEST_F(RampUpTest, TransportSequenceNumberSimulcastRedRtx) {
RunBaseTest(&test);
}
TEST_F(RampUpTest, AudioTransportSequenceNumber) {
// TODO(bugs.webrtc.org/8878)
#if defined(WEBRTC_MAC)
#define MAYBE_AudioTransportSequenceNumber DISABLED_AudioTransportSequenceNumber
#else
#define MAYBE_AudioTransportSequenceNumber AudioTransportSequenceNumber
#endif
TEST_F(RampUpTest, MAYBE_AudioTransportSequenceNumber) {
RampUpTester test(0, 1, 0, 300000, 10000,
RtpExtension::kTransportSequenceNumberUri, false, false,
false);