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

Unified Diff: media/audio/cras/cras_unified_unittest.cc

Issue 11959018: Add a unified audio I/O backend for ChromeOS. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Save params, other cleanups suggested by Dale Created 7 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/audio/cras/cras_unified.cc ('k') | media/media.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/cras/cras_unified_unittest.cc
diff --git a/media/audio/cras/cras_unified_unittest.cc b/media/audio/cras/cras_unified_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..73f718481373ca15fc463b768fcb5e4ee31d5e40
--- /dev/null
+++ b/media/audio/cras/cras_unified_unittest.cc
@@ -0,0 +1,157 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include <string>
+
+#include "base/synchronization/waitable_event.h"
+#include "base/test/test_timeouts.h"
+#include "base/time.h"
+#include "media/audio/cras/audio_manager_cras.h"
+#include "media/audio/cras/cras_unified.h"
+#include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+using testing::_;
+using testing::DoAll;
+using testing::InvokeWithoutArgs;
+using testing::Return;
+using testing::SetArgumentPointee;
+using testing::StrictMock;
+
+namespace media {
+
+class MockAudioSourceCallback : public AudioOutputStream::AudioSourceCallback {
+ public:
+ MOCK_METHOD2(OnMoreData, int(AudioBus* audio_bus,
+ AudioBuffersState buffers_state));
+ MOCK_METHOD3(OnMoreIOData, int(AudioBus* source,
+ AudioBus* dest,
+ AudioBuffersState buffers_state));
+ MOCK_METHOD1(OnError, void(AudioOutputStream* stream));
+};
+
+class MockAudioManagerCras : public AudioManagerCras {
+ public:
+ MOCK_METHOD0(Init, void());
+ MOCK_METHOD0(HasAudioOutputDevices, bool());
+ MOCK_METHOD0(HasAudioInputDevices, bool());
+ MOCK_METHOD1(MakeLinearOutputStream, AudioOutputStream*(
+ const AudioParameters& params));
+ MOCK_METHOD1(MakeLowLatencyOutputStream, AudioOutputStream*(
+ const AudioParameters& params));
+ MOCK_METHOD2(MakeLinearOutputStream, AudioInputStream*(
+ const AudioParameters& params, const std::string& device_id));
+ MOCK_METHOD2(MakeLowLatencyInputStream, AudioInputStream*(
+ const AudioParameters& params, const std::string& device_id));
+
+ // We need to override this function in order to skip the checking the number
+ // of active output streams. It is because the number of active streams
+ // is managed inside MakeAudioOutputStream, and we don't use
+ // MakeAudioOutputStream to create the stream in the tests.
+ virtual void ReleaseOutputStream(AudioOutputStream* stream) OVERRIDE {
+ DCHECK(stream);
+ delete stream;
+ }
+};
+
+class CrasUnifiedStreamTest : public testing::Test {
+ protected:
+ CrasUnifiedStreamTest() {
+ mock_manager_.reset(new StrictMock<MockAudioManagerCras>());
+ }
+
+ virtual ~CrasUnifiedStreamTest() {
+ }
+
+ CrasUnifiedStream* CreateStream(ChannelLayout layout) {
+ return CreateStream(layout, kTestFramesPerPacket);
+ }
+
+ CrasUnifiedStream* CreateStream(ChannelLayout layout,
+ int32 samples_per_packet) {
+ AudioParameters params(kTestFormat, layout, kTestSampleRate,
+ kTestBitsPerSample, samples_per_packet);
+ return new CrasUnifiedStream(params, mock_manager_.get());
+ }
+
+ MockAudioManagerCras& mock_manager() {
+ return *(mock_manager_.get());
+ }
+
+ static const ChannelLayout kTestChannelLayout;
+ static const int kTestSampleRate;
+ static const int kTestBitsPerSample;
+ static const AudioParameters::Format kTestFormat;
+ static const uint32 kTestFramesPerPacket;
+
+ scoped_ptr<StrictMock<MockAudioManagerCras> > mock_manager_;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(CrasUnifiedStreamTest);
+};
+
+const ChannelLayout CrasUnifiedStreamTest::kTestChannelLayout =
+ CHANNEL_LAYOUT_STEREO;
+const int CrasUnifiedStreamTest::kTestSampleRate =
+ AudioParameters::kAudioCDSampleRate;
+const int CrasUnifiedStreamTest::kTestBitsPerSample = 16;
+const AudioParameters::Format CrasUnifiedStreamTest::kTestFormat =
+ AudioParameters::AUDIO_PCM_LINEAR;
+const uint32 CrasUnifiedStreamTest::kTestFramesPerPacket = 1000;
+
+TEST_F(CrasUnifiedStreamTest, ConstructedState) {
+ // Should support mono.
+ CrasUnifiedStream* test_stream = CreateStream(CHANNEL_LAYOUT_MONO);
+ test_stream->Close();
+
+ // Should support stereo.
+ test_stream = CreateStream(CHANNEL_LAYOUT_SURROUND);
+ EXPECT_TRUE(test_stream->Open());
+ test_stream->Close();
+
+ // Bad bits per sample.
+ AudioParameters bad_bps_params(kTestFormat, kTestChannelLayout,
+ kTestSampleRate, kTestBitsPerSample - 1,
+ kTestFramesPerPacket);
+ test_stream = new CrasUnifiedStream(bad_bps_params, mock_manager_.get());
+ EXPECT_FALSE(test_stream->Open());
+ test_stream->Close();
+
+ // Bad sample rate.
+ AudioParameters bad_rate_params(kTestFormat, kTestChannelLayout,
+ 0, kTestBitsPerSample, kTestFramesPerPacket);
+ test_stream = new CrasUnifiedStream(bad_rate_params, mock_manager_.get());
+ EXPECT_FALSE(test_stream->Open());
+ test_stream->Close();
+
+ // Check that Mono works too.
+ test_stream = CreateStream(CHANNEL_LAYOUT_MONO);
+ ASSERT_TRUE(test_stream->Open());
+ test_stream->Close();
+}
+
+TEST_F(CrasUnifiedStreamTest, RenderFrames) {
+ CrasUnifiedStream* test_stream = CreateStream(CHANNEL_LAYOUT_MONO);
+ MockAudioSourceCallback mock_callback;
+
+ ASSERT_TRUE(test_stream->Open());
+
+ base::WaitableEvent event(false, false);
+
+ EXPECT_CALL(mock_callback, OnMoreData(_, _))
+ .WillRepeatedly(DoAll(
+ InvokeWithoutArgs(&event, &base::WaitableEvent::Signal),
+ Return(kTestFramesPerPacket)));
+
+ test_stream->Start(&mock_callback);
+
+ // Wait for samples to be captured.
+ EXPECT_TRUE(event.TimedWait(TestTimeouts::action_timeout()));
+
+ test_stream->Stop();
+
+ test_stream->Close();
+}
+
+} // namespace media
« no previous file with comments | « media/audio/cras/cras_unified.cc ('k') | media/media.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698