Remove test AudioCodingModuleTest.TestAPI
Since it isn't being run by the bots, it has bit rotted; when I try to run it manually, it fails with a long list of error messages: Error Calling API in file ../../modules/audio_coding/test/APITest.cc at line 995 Error Calling API in file ../../modules/audio_coding/test/APITest.cc at line 996 >>> Error Enabling VAD <<< Error Calling API in file ../../modules/audio_coding/test/APITest.cc at line 995 Error Calling API in file ../../modules/audio_coding/test/APITest.cc at line 996 >>> Error Enabling DTX <<< >>> Error Enabling VAD <<< Error Calling API in file ../../modules/audio_coding/test/APITest.cc at line 995 Error Calling API in file ../../modules/audio_coding/test/APITest.cc at line 996 >>> Error Enabling VAD <<< Error Calling API in file ../../modules/audio_coding/test/APITest.cc at line 995 Error Calling API in file ../../modules/audio_coding/test/APITest.cc at line 996 Error Calling API in file ../../modules/audio_coding/test/APITest.cc at line 985 ...and so on. Bug: webrtc:8396 Change-Id: Id8f1e01a751b4bb3527702b7b7a4986ce0abb378 Reviewed-on: https://webrtc-review.googlesource.com/81745 Reviewed-by: Henrik Lundin <henrik.lundin@webrtc.org> Commit-Queue: Karl Wiberg <kwiberg@webrtc.org> Cr-Commit-Position: refs/heads/master@{#23542}
This commit is contained in:
Karl Wiberg
File diff suppressed because it is too large
Load Diff
@ -1,162 +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.
|
||||
*/
|
||||
|
||||
#ifndef MODULES_AUDIO_CODING_TEST_APITEST_H_
|
||||
#define MODULES_AUDIO_CODING_TEST_APITEST_H_
|
||||
|
||||
#include <memory>
|
||||
|
||||
#include "modules/audio_coding/include/audio_coding_module.h"
|
||||
#include "modules/audio_coding/test/ACMTest.h"
|
||||
#include "modules/audio_coding/test/Channel.h"
|
||||
#include "modules/audio_coding/test/PCMFile.h"
|
||||
#include "modules/audio_coding/test/utility.h"
|
||||
#include "rtc_base/synchronization/rw_lock_wrapper.h"
|
||||
#include "system_wrappers/include/event_wrapper.h"
|
||||
|
||||
namespace webrtc {
|
||||
|
||||
enum APITESTAction {
|
||||
TEST_CHANGE_CODEC_ONLY = 0,
|
||||
DTX_TEST = 1
|
||||
};
|
||||
|
||||
class APITest : public ACMTest {
|
||||
public:
|
||||
APITest();
|
||||
~APITest();
|
||||
|
||||
void Perform();
|
||||
private:
|
||||
int16_t SetUp();
|
||||
|
||||
static bool PushAudioThreadA(void* obj);
|
||||
static bool PullAudioThreadA(void* obj);
|
||||
static bool ProcessThreadA(void* obj);
|
||||
static bool APIThreadA(void* obj);
|
||||
|
||||
static bool PushAudioThreadB(void* obj);
|
||||
static bool PullAudioThreadB(void* obj);
|
||||
static bool ProcessThreadB(void* obj);
|
||||
static bool APIThreadB(void* obj);
|
||||
|
||||
void CheckVADStatus(char side);
|
||||
|
||||
// Set Min delay, get delay, playout timestamp
|
||||
void TestDelay(char side);
|
||||
|
||||
// Unregister a codec & register again.
|
||||
void TestRegisteration(char side);
|
||||
|
||||
// Playout Mode, background noise mode.
|
||||
// Receiver Frequency, playout frequency.
|
||||
void TestPlayout(char receiveSide);
|
||||
|
||||
//
|
||||
void TestSendVAD(char side);
|
||||
|
||||
void CurrentCodec(char side);
|
||||
|
||||
void ChangeCodec(char side);
|
||||
|
||||
void Wait(uint32_t waitLengthMs);
|
||||
|
||||
void RunTest(char thread);
|
||||
|
||||
bool PushAudioRunA();
|
||||
bool PullAudioRunA();
|
||||
bool ProcessRunA();
|
||||
bool APIRunA();
|
||||
|
||||
bool PullAudioRunB();
|
||||
bool PushAudioRunB();
|
||||
bool ProcessRunB();
|
||||
bool APIRunB();
|
||||
|
||||
//--- ACMs
|
||||
std::unique_ptr<AudioCodingModule> _acmA;
|
||||
std::unique_ptr<AudioCodingModule> _acmB;
|
||||
|
||||
//--- Channels
|
||||
Channel* _channel_A2B;
|
||||
Channel* _channel_B2A;
|
||||
|
||||
//--- I/O files
|
||||
// A
|
||||
PCMFile _inFileA;
|
||||
PCMFile _outFileA;
|
||||
// B
|
||||
PCMFile _outFileB;
|
||||
PCMFile _inFileB;
|
||||
|
||||
//--- I/O params
|
||||
// A
|
||||
int32_t _outFreqHzA;
|
||||
// B
|
||||
int32_t _outFreqHzB;
|
||||
|
||||
// Should we write to file.
|
||||
// we might skip writing to file if we
|
||||
// run the test for a long time.
|
||||
bool _writeToFile;
|
||||
//--- Events
|
||||
// A
|
||||
EventTimerWrapper* _pullEventA; // pulling data from ACM
|
||||
EventTimerWrapper* _pushEventA; // pushing data to ACM
|
||||
EventTimerWrapper* _processEventA; // process
|
||||
EventWrapper* _apiEventA; // API calls
|
||||
// B
|
||||
EventTimerWrapper* _pullEventB; // pulling data from ACM
|
||||
EventTimerWrapper* _pushEventB; // pushing data to ACM
|
||||
EventTimerWrapper* _processEventB; // process
|
||||
EventWrapper* _apiEventB; // API calls
|
||||
|
||||
// keep track of the codec in either side.
|
||||
uint8_t _codecCntrA;
|
||||
uint8_t _codecCntrB;
|
||||
|
||||
// Is set to true if there is no encoder in either side
|
||||
bool _thereIsEncoderA;
|
||||
bool _thereIsEncoderB;
|
||||
bool _thereIsDecoderA;
|
||||
bool _thereIsDecoderB;
|
||||
|
||||
bool _sendVADA;
|
||||
bool _sendDTXA;
|
||||
ACMVADMode _sendVADModeA;
|
||||
|
||||
bool _sendVADB;
|
||||
bool _sendDTXB;
|
||||
ACMVADMode _sendVADModeB;
|
||||
|
||||
int32_t _minDelayA;
|
||||
int32_t _minDelayB;
|
||||
bool _payloadUsed[32];
|
||||
|
||||
bool _verbose;
|
||||
|
||||
int _dotPositionA;
|
||||
int _dotMoveDirectionA;
|
||||
int _dotPositionB;
|
||||
int _dotMoveDirectionB;
|
||||
|
||||
char _movingDot[41];
|
||||
|
||||
VADCallback* _vadCallbackA;
|
||||
VADCallback* _vadCallbackB;
|
||||
RWLockWrapper& _apiTestRWLock;
|
||||
bool _randomTest;
|
||||
int _testNumA;
|
||||
int _testNumB;
|
||||
};
|
||||
|
||||
} // namespace webrtc
|
||||
|
||||
#endif // MODULES_AUDIO_CODING_TEST_APITEST_H_
|
||||
@ -13,7 +13,6 @@
|
||||
#include <vector>
|
||||
|
||||
#include "modules/audio_coding/include/audio_coding_module.h"
|
||||
#include "modules/audio_coding/test/APITest.h"
|
||||
#include "modules/audio_coding/test/EncodeDecodeTest.h"
|
||||
#include "modules/audio_coding/test/PacketLossTest.h"
|
||||
#include "modules/audio_coding/test/TestAllCodecs.h"
|
||||
@ -129,11 +128,3 @@ TEST(AudioCodingModuleTest, TestPacketLossStereoBurst) {
|
||||
#endif
|
||||
webrtc::PacketLossTest(2, 10, 10, 2).Perform();
|
||||
}
|
||||
|
||||
// The full API test is too long to run automatically on bots, but can be used
|
||||
// for offline testing. User interaction is needed.
|
||||
#ifdef ACM_TEST_FULL_API
|
||||
TEST(AudioCodingModuleTest, TestAPI) {
|
||||
webrtc::APITest().Perform();
|
||||
}
|
||||
#endif
|
||||
|
||||
Reference in New Issue
Block a user