From 056287eee09b82181f3270c38f72f833fd3d7acc Mon Sep 17 00:00:00 2001 From: "tina.legrand@webrtc.org" Date: Wed, 19 Feb 2014 13:45:54 +0000 Subject: [PATCH] This CL separate all ACM tests with new and old implementation of ACM and NetEq. The reason is to debug an issue with failure on Android try bots. We need to see if the error only occurs with the new ACM/NetEq, or if it is a flakiness that affects both. BUG=issue2874 R=minyue@webrtc.org Review URL: https://webrtc-codereview.appspot.com/8829004 git-svn-id: http://webrtc.googlecode.com/svn/trunk@5576 4adac7df-926f-26a2-2b94-8c16560cd09d --- .../modules/audio_coding/main/test/Tester.cc | 76 ++++++++++++++++--- 1 file changed, 66 insertions(+), 10 deletions(-) diff --git a/webrtc/modules/audio_coding/main/test/Tester.cc b/webrtc/modules/audio_coding/main/test/Tester.cc index 31f7317fc7..0307235b10 100644 --- a/webrtc/modules/audio_coding/main/test/Tester.cc +++ b/webrtc/modules/audio_coding/main/test/Tester.cc @@ -43,10 +43,17 @@ TEST(AudioCodingModuleTest, TestAllCodecs) { UseLegacyAcm(&config); webrtc::TestAllCodecs(ACM_TEST_MODE, config).Perform(); + Trace::ReturnTrace(); +} + +TEST(AudioCodingModuleTest, TestAllCodecsNewACM) { + Trace::CreateTrace(); + Trace::SetTraceFile((webrtc::test::OutputPath() + + "acm_allcodecs_trace_new.txt").c_str()); + webrtc::Config config; UseNewAcm(&config); webrtc::TestAllCodecs(ACM_TEST_MODE, config).Perform(); - Trace::ReturnTrace(); } @@ -58,10 +65,17 @@ TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TestEncodeDecode)) { UseLegacyAcm(&config); webrtc::EncodeDecodeTest(ACM_TEST_MODE, config).Perform(); + Trace::ReturnTrace(); +} + +TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TestEncodeDecodeNewACM)) { + Trace::CreateTrace(); + Trace::SetTraceFile((webrtc::test::OutputPath() + + "acm_encodedecode_trace_new.txt").c_str()); + webrtc::Config config; UseNewAcm(&config); webrtc::EncodeDecodeTest(ACM_TEST_MODE, config).Perform(); - Trace::ReturnTrace(); } @@ -73,10 +87,17 @@ TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TestFEC)) { UseLegacyAcm(&config); webrtc::TestFEC(config).Perform(); + Trace::ReturnTrace(); +} + +TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TestFECNewACM)) { + Trace::CreateTrace(); + Trace::SetTraceFile((webrtc::test::OutputPath() + + "acm_fec_trace_new.txt").c_str()); + webrtc::Config config; UseNewAcm(&config); webrtc::TestFEC(config).Perform(); - Trace::ReturnTrace(); } @@ -88,10 +109,17 @@ TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TestIsac)) { UseLegacyAcm(&config); webrtc::ISACTest(ACM_TEST_MODE, config).Perform(); + Trace::ReturnTrace(); +} + +TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TestIsacNewACM)) { + Trace::CreateTrace(); + Trace::SetTraceFile((webrtc::test::OutputPath() + + "acm_isac_trace_new.txt").c_str()); + webrtc::Config config; UseNewAcm(&config); webrtc::ISACTest(ACM_TEST_MODE, config).Perform(); - Trace::ReturnTrace(); } @@ -103,10 +131,17 @@ TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TwoWayCommunication)) { UseLegacyAcm(&config); webrtc::TwoWayCommunication(ACM_TEST_MODE, config).Perform(); + Trace::ReturnTrace(); +} + +TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TwoWayCommunicationNewACM)) { + Trace::CreateTrace(); + Trace::SetTraceFile((webrtc::test::OutputPath() + + "acm_twowaycom_trace_new.txt").c_str()); + webrtc::Config config; UseNewAcm(&config); webrtc::TwoWayCommunication(ACM_TEST_MODE, config).Perform(); - Trace::ReturnTrace(); } @@ -114,13 +149,20 @@ TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TestStereo)) { Trace::CreateTrace(); Trace::SetTraceFile((webrtc::test::OutputPath() + "acm_stereo_trace.txt").c_str()); - webrtc::Config config; + UseLegacyAcm(&config); - webrtc::TestStereo(ACM_TEST_MODE, config).Perform(); - UseNewAcm(&config); + Trace::ReturnTrace(); +} +TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TestStereoNewACM)) { + Trace::CreateTrace(); + Trace::SetTraceFile((webrtc::test::OutputPath() + + "acm_stereo_trace_new.txt").c_str()); + webrtc::Config config; + + UseNewAcm(&config); webrtc::TestStereo(ACM_TEST_MODE, config).Perform(); Trace::ReturnTrace(); } @@ -133,10 +175,17 @@ TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TestVADDTX)) { UseLegacyAcm(&config); webrtc::TestVADDTX(config).Perform(); + Trace::ReturnTrace(); +} + +TEST(AudioCodingModuleTest, DISABLED_ON_ANDROID(TestVADDTXNewACM)) { + Trace::CreateTrace(); + Trace::SetTraceFile((webrtc::test::OutputPath() + + "acm_vaddtx_trace_new.txt").c_str()); + webrtc::Config config; UseNewAcm(&config); webrtc::TestVADDTX(config).Perform(); - Trace::ReturnTrace(); } @@ -148,10 +197,17 @@ TEST(AudioCodingModuleTest, TestOpus) { UseLegacyAcm(&config); webrtc::OpusTest(config).Perform(); + Trace::ReturnTrace(); +} + +TEST(AudioCodingModuleTest, TestOpusNewACM) { + Trace::CreateTrace(); + Trace::SetTraceFile((webrtc::test::OutputPath() + + "acm_opus_trace_new.txt").c_str()); + webrtc::Config config; UseNewAcm(&config); webrtc::OpusTest(config).Perform(); - Trace::ReturnTrace(); }