Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(96)

Side by Side Diff: webrtc/modules/audio_device/ios/audio_device_unittest_ios.cc

Issue 2652423002: Disabled two iOS tests due to bot breakage. Affected tests are (Closed)
Patch Set: Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 580 matching lines...) Expand 10 before | Expand all | Expand 10 after
591 // Failing when running on real iOS devices: bugs.webrtc.org/6889. 591 // Failing when running on real iOS devices: bugs.webrtc.org/6889.
592 TEST_F(AudioDeviceTest, DISABLED_StartStopPlayout) { 592 TEST_F(AudioDeviceTest, DISABLED_StartStopPlayout) {
593 StartPlayout(); 593 StartPlayout();
594 StopPlayout(); 594 StopPlayout();
595 StartPlayout(); 595 StartPlayout();
596 StopPlayout(); 596 StopPlayout();
597 } 597 }
598 598
599 // Tests that recording can be initiated, started and stopped. No audio callback 599 // Tests that recording can be initiated, started and stopped. No audio callback
600 // is registered in this test. 600 // is registered in this test.
601 TEST_F(AudioDeviceTest, StartStopRecording) { 601 // Disabled due to failure: bugs.webrtc.org/7056
602 TEST_F(AudioDeviceTest, DISABLED_StartStopRecording) {
602 StartRecording(); 603 StartRecording();
603 StopRecording(); 604 StopRecording();
604 StartRecording(); 605 StartRecording();
605 StopRecording(); 606 StopRecording();
606 } 607 }
607 608
608 // Verify that calling StopPlayout() will leave us in an uninitialized state 609 // Verify that calling StopPlayout() will leave us in an uninitialized state
609 // which will require a new call to InitPlayout(). This test does not call 610 // which will require a new call to InitPlayout(). This test does not call
610 // StartPlayout() while being uninitialized since doing so will hit a 611 // StartPlayout() while being uninitialized since doing so will hit a
611 // RTC_DCHECK. 612 // RTC_DCHECK.
(...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
726 StartPlayout(); 727 StartPlayout();
727 StartRecording(); 728 StartRecording();
728 test_is_done_->Wait(kTestTimeOutInMilliseconds); 729 test_is_done_->Wait(kTestTimeOutInMilliseconds);
729 StopRecording(); 730 StopRecording();
730 StopPlayout(); 731 StopPlayout();
731 } 732 }
732 733
733 // Start playout and read audio from an external PCM file when the audio layer 734 // Start playout and read audio from an external PCM file when the audio layer
734 // asks for data to play out. Real audio is played out in this test but it does 735 // asks for data to play out. Real audio is played out in this test but it does
735 // not contain any explicit verification that the audio quality is perfect. 736 // not contain any explicit verification that the audio quality is perfect.
736 TEST_F(AudioDeviceTest, RunPlayoutWithFileAsSource) { 737 // Disabled due to failure: bugs.webrtc.org/7056
738 TEST_F(AudioDeviceTest, DISABLED_RunPlayoutWithFileAsSource) {
737 // TODO(henrika): extend test when mono output is supported. 739 // TODO(henrika): extend test when mono output is supported.
738 EXPECT_EQ(1, playout_channels()); 740 EXPECT_EQ(1, playout_channels());
739 NiceMock<MockAudioTransportIOS> mock(kPlayout); 741 NiceMock<MockAudioTransportIOS> mock(kPlayout);
740 const int num_callbacks = kFilePlayTimeInSec * kNumCallbacksPerSecond; 742 const int num_callbacks = kFilePlayTimeInSec * kNumCallbacksPerSecond;
741 std::string file_name = GetFileName(playout_sample_rate()); 743 std::string file_name = GetFileName(playout_sample_rate());
742 std::unique_ptr<FileAudioStream> file_audio_stream( 744 std::unique_ptr<FileAudioStream> file_audio_stream(
743 new FileAudioStream(num_callbacks, file_name, playout_sample_rate())); 745 new FileAudioStream(num_callbacks, file_name, playout_sample_rate()));
744 mock.HandleCallbacks(test_is_done_.get(), file_audio_stream.get(), 746 mock.HandleCallbacks(test_is_done_.get(), file_audio_stream.get(),
745 num_callbacks); 747 num_callbacks);
746 // SetMaxPlayoutVolume(); 748 // SetMaxPlayoutVolume();
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
816 StopPlayout(); 818 StopPlayout();
817 StopRecording(); 819 StopRecording();
818 // Verify that the correct number of transmitted impulses are detected. 820 // Verify that the correct number of transmitted impulses are detected.
819 EXPECT_EQ(latency_audio_stream->num_latency_values(), 821 EXPECT_EQ(latency_audio_stream->num_latency_values(),
820 static_cast<size_t>( 822 static_cast<size_t>(
821 kImpulseFrequencyInHz * kMeasureLatencyTimeInSec - 1)); 823 kImpulseFrequencyInHz * kMeasureLatencyTimeInSec - 1));
822 latency_audio_stream->PrintResults(); 824 latency_audio_stream->PrintResults();
823 } 825 }
824 826
825 } // namespace webrtc 827 } // namespace webrtc
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698