From 9d1e07063e38255d5e48c15612376e2986ba1e27 Mon Sep 17 00:00:00 2001 From: Harald Alvestrand Date: Tue, 16 Mar 2021 06:15:01 +0000 Subject: [PATCH] Increase wait-for-lost-packet from 10 to 100 msec in MTU test 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 Commit-Queue: Harald Alvestrand Cr-Commit-Position: refs/heads/master@{#33473} --- pc/data_channel_integrationtest.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pc/data_channel_integrationtest.cc b/pc/data_channel_integrationtest.cc index d4a259b608..1a93492819 100644 --- a/pc/data_channel_integrationtest.cc +++ b/pc/data_channel_integrationtest.cc @@ -502,8 +502,10 @@ TEST_P(DataChannelIntegrationTest, EndToEndCallWithSctpDataChannelHarmfulMtu) { const std::string data(message_size, 'a'); caller()->data_channel()->Send(DataBuffer(data)); // 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, - 10); + 100); if (callee()->data_observer()->received_message_count() == message_count) { ASSERT_EQ(kMessageSizeThatIsNotDelivered, message_size); failure_seen = true;