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

Unified Diff: webrtc/voice_engine/test/auto_test/extended/agc_config_test.cc

Issue 2738543002: Remove VoEAudioProcessing interface. (Closed)
Patch Set: rebase Created 3 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
Index: webrtc/voice_engine/test/auto_test/extended/agc_config_test.cc
diff --git a/webrtc/voice_engine/test/auto_test/extended/agc_config_test.cc b/webrtc/voice_engine/test/auto_test/extended/agc_config_test.cc
deleted file mode 100644
index 0fc0d39c73a8f882649b99242ca9f8cf1477c291..0000000000000000000000000000000000000000
--- a/webrtc/voice_engine/test/auto_test/extended/agc_config_test.cc
+++ /dev/null
@@ -1,69 +0,0 @@
-/*
- * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
- *
- * Use of this source code is governed by a BSD-style license
- * that can be found in the LICENSE file in the root of the source
- * tree. An additional intellectual property rights grant can be found
- * in the file PATENTS. All contributing project authors may
- * be found in the AUTHORS file in the root of the source tree.
- */
-
-#include "webrtc/voice_engine/test/auto_test/fixtures/after_streaming_fixture.h"
-
-class AgcConfigTest : public AfterStreamingFixture {
- protected:
- void SetUp() {
- // These should be defaults for the AGC config.
- default_agc_config_.digitalCompressionGaindB = 9;
- default_agc_config_.limiterEnable = true;
- default_agc_config_.targetLeveldBOv = 3;
- }
-
- webrtc::AgcConfig default_agc_config_;
-};
-
-// Duplicated in apm_helpers_unittest.cc.
-TEST_F(AgcConfigTest, HasCorrectDefaultConfiguration) {
- webrtc::AgcConfig agc_config;
-
- EXPECT_EQ(0, voe_apm_->GetAgcConfig(agc_config));
-
- EXPECT_EQ(default_agc_config_.targetLeveldBOv, agc_config.targetLeveldBOv);
- EXPECT_EQ(default_agc_config_.digitalCompressionGaindB,
- agc_config.digitalCompressionGaindB);
- EXPECT_EQ(default_agc_config_.limiterEnable, agc_config.limiterEnable);
-}
-
-// Not needed anymore - we're not returning errors anymore, just logging.
-TEST_F(AgcConfigTest, DealsWithInvalidParameters) {
- webrtc::AgcConfig agc_config = default_agc_config_;
- agc_config.digitalCompressionGaindB = 91;
- EXPECT_EQ(-1, voe_apm_->SetAgcConfig(agc_config)) << "Should not be able "
- "to set gain to more than 90 dB.";
- EXPECT_EQ(VE_APM_ERROR, voe_base_->LastError());
-
- agc_config = default_agc_config_;
- agc_config.targetLeveldBOv = 32;
- EXPECT_EQ(-1, voe_apm_->SetAgcConfig(agc_config)) << "Should not be able "
- "to set target level to more than 31.";
- EXPECT_EQ(VE_APM_ERROR, voe_base_->LastError());
-}
-
-// Duplicated in apm_helpers_unittest.cc.
-TEST_F(AgcConfigTest, CanGetAndSetAgcStatus) {
- webrtc::AgcConfig agc_config;
- agc_config.digitalCompressionGaindB = 17;
- agc_config.targetLeveldBOv = 11;
- agc_config.limiterEnable = false;
-
- webrtc::AgcConfig actual_config;
- EXPECT_EQ(0, voe_apm_->SetAgcConfig(agc_config));
- EXPECT_EQ(0, voe_apm_->GetAgcConfig(actual_config));
-
- EXPECT_EQ(agc_config.digitalCompressionGaindB,
- actual_config.digitalCompressionGaindB);
- EXPECT_EQ(agc_config.limiterEnable,
- actual_config.limiterEnable);
- EXPECT_EQ(agc_config.targetLeveldBOv,
- actual_config.targetLeveldBOv);
-}
« no previous file with comments | « webrtc/voice_engine/include/voe_audio_processing.h ('k') | webrtc/voice_engine/test/auto_test/extended/ec_metrics_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698