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

Increase wait-for-lost-packet from 10 to 100 msec in MTU test

Browse Source 
This increases the running time of the test, but seems to be needed
to avoid flakiness on Windows.

Bug: webrtc:12587
Change-Id: Id8c49910e276b2754244d977d66241e6e211c720
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/212023
Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#33473}
This commit is contained in:
Harald Alvestrand
2021-03-16 06:15:01 +00:00
committed by Commit Bot
parent 596ba4ccd5
commit 9d1e07063e
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
pc/data_channel_integrationtest.cc
View File

@ -502,8 +502,10 @@ TEST_P(DataChannelIntegrationTest, EndToEndCallWithSctpDataChannelHarmfulMtu) {
const std::string data(message_size, 'a'); const std::string data(message_size, 'a');
caller()->data_channel()->Send(DataBuffer(data)); caller()->data_channel()->Send(DataBuffer(data));
// Wait a very short time for the message to be delivered. // Wait a very short time for the message to be delivered.
// Note: Waiting only 10 ms is too short for Windows bots; they will
// flakily fail at a random frame.
WAIT(callee()->data_observer()->received_message_count() > message_count, WAIT(callee()->data_observer()->received_message_count() > message_count,
10); 100);
if (callee()->data_observer()->received_message_count() == message_count) { if (callee()->data_observer()->received_message_count() == message_count) {
ASSERT_EQ(kMessageSizeThatIsNotDelivered, message_size); ASSERT_EQ(kMessageSizeThatIsNotDelivered, message_size);
failure_seen = true; failure_seen = true;