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

Unified Diff: webrtc/audio/audio_send_stream_unittest.cc

Issue 1414743004: Implement AudioSendStream::GetStats(). (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: comments Created 5 years, 2 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/audio/audio_send_stream_unittest.cc
diff --git a/webrtc/audio/audio_send_stream_unittest.cc b/webrtc/audio/audio_send_stream_unittest.cc
index e5d73ff0f29185dd81500fe833a07588334d4e97..97aab330eff4d0f332b808ead65d6aa4cc59e167 100644
--- a/webrtc/audio/audio_send_stream_unittest.cc
+++ b/webrtc/audio/audio_send_stream_unittest.cc
@@ -11,8 +11,11 @@
#include "testing/gtest/include/gtest/gtest.h"
#include "webrtc/audio/audio_send_stream.h"
+#include "webrtc/audio/conversion.h"
+#include "webrtc/test/fake_voice_engine.h"
namespace webrtc {
+namespace test {
TEST(AudioSendStreamTest, ConfigToString) {
const int kAbsSendTimeId = 3;
@@ -27,8 +30,44 @@ TEST(AudioSendStreamTest, ConfigToString) {
}
TEST(AudioSendStreamTest, ConstructDestruct) {
+ FakeVoiceEngine voice_engine;
AudioSendStream::Config config(nullptr);
config.voe_channel_id = 1;
- internal::AudioSendStream send_stream(config);
+ internal::AudioSendStream send_stream(config, &voice_engine);
}
+
+TEST(AudioSendStreamTest, GetStats) {
+ FakeVoiceEngine voice_engine;
+ AudioSendStream::Config config(nullptr);
+ config.rtp.ssrc = FakeVoiceEngine::kSendSsrc;
+ config.voe_channel_id = FakeVoiceEngine::kSendChannelId;
+ internal::AudioSendStream send_stream(config, &voice_engine);
+
+ AudioSendStream::Stats stats = send_stream.GetStats();
+ const CallStatistics& call_stats = FakeVoiceEngine::kSendCallStats;
+ const CodecInst& codec_inst = FakeVoiceEngine::kSendCodecInst;
+ const ReportBlock& report_block = FakeVoiceEngine::kSendReportBlock;
+ EXPECT_EQ(FakeVoiceEngine::kSendSsrc, stats.local_ssrc);
+ EXPECT_EQ(static_cast<int64_t>(call_stats.bytesSent), stats.bytes_sent);
+ EXPECT_EQ(call_stats.packetsSent, stats.packets_sent);
+ EXPECT_EQ(static_cast<int32_t>(report_block.cumulative_num_packets_lost),
+ stats.packets_lost);
+ EXPECT_EQ(Q8ToFloat(report_block.fraction_lost), stats.fraction_lost);
+ EXPECT_EQ(std::string(codec_inst.plname), stats.codec_name);
+ EXPECT_EQ(static_cast<int32_t>(report_block.extended_highest_sequence_number),
+ stats.ext_seqnum);
+ EXPECT_EQ(static_cast<int32_t>(report_block.interarrival_jitter /
+ (codec_inst.plfreq / 1000)), stats.jitter_ms);
+ EXPECT_EQ(call_stats.rttMs, stats.rtt_ms);
+ EXPECT_EQ(static_cast<int32_t>(FakeVoiceEngine::kSendSpeechInputLevel),
+ stats.audio_level);
+ EXPECT_EQ(-1, stats.aec_quality_min);
+ EXPECT_EQ(FakeVoiceEngine::kSendEchoDelayMedian, stats.echo_delay_median_ms);
+ EXPECT_EQ(FakeVoiceEngine::kSendEchoDelayStdDev, stats.echo_delay_std_ms);
+ EXPECT_EQ(FakeVoiceEngine::kSendEchoReturnLoss, stats.echo_return_loss);
+ EXPECT_EQ(FakeVoiceEngine::kSendEchoReturnLossEnhancement,
+ stats.echo_return_loss_enhancement);
+ EXPECT_EQ(false, stats.typing_noise_detected);
+}
+} // namespace test
} // namespace webrtc

Powered by Google App Engine
This is Rietveld 408576698