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

Disables AudioDeviceTest.RunPlayoutAndRecordingInFullDuplex on Android

Browse Source 
TBR=henrik.lundin

Bug: webrtc:7744
Change-Id: I0ebfd3016d9d6b815d5b1801e8481363da11af54
Reviewed-on: https://webrtc-review.googlesource.com/7200
Reviewed-by: Henrik Andreassson <henrika@webrtc.org>
Commit-Queue: Henrik Andreassson <henrika@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20179}
This commit is contained in:
henrika
2017-10-06 11:23:30 +02:00
committed by Commit Bot
parent bf35eb29b5
commit 3def74b1b8
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
modules/audio_device/android/audio_device_unittest.cc
View File

@ -960,7 +960,13 @@ TEST_F(AudioDeviceTest, RunPlayoutWithFileAsSource) {
// recording side and decreased by the playout side.
// TODO(henrika): tune the final test parameters after running tests on several
// different devices.
TEST_F(AudioDeviceTest, RunPlayoutAndRecordingInFullDuplex) {
// Disabling this test on bots since it is difficult to come up with a robust
// test condition that all worked as intended. The main issue is that, when
// swarming is used, an initial latency can be built up when the both sides
// starts at different times. Hence, the test can fail even if audio works
// as intended. Keeping the test so it can be enabled manually.
// http://bugs.webrtc.org/7744
TEST_F(AudioDeviceTest, DISABLED_RunPlayoutAndRecordingInFullDuplex) {
EXPECT_EQ(record_channels(), playout_channels());
EXPECT_EQ(record_sample_rate(), playout_sample_rate());
NiceMock<MockAudioTransportAndroid> mock(kPlayout | kRecording);