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

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: workaround for android build issue 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
« no previous file with comments | « webrtc/audio/audio_send_stream.cc ('k') | webrtc/audio/conversion.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..227ec8379971f3e6c2ad73b52acf75872e27a08a 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;
@@ -23,12 +26,51 @@ TEST(AudioSendStreamTest, ConfigToString) {
config.voe_channel_id = 1;
config.cng_payload_type = 42;
config.red_payload_type = 17;
- EXPECT_GT(config.ToString().size(), 0u);
+ EXPECT_EQ("{rtp: {ssrc: 1234, extensions: [{name: "
+ "http://www.webrtc.org/experiments/rtp-hdrext/abs-send-time, id: 3}]}, "
+ "voe_channel_id: 1, cng_payload_type: 42, red_payload_type: 17}",
+ config.ToString());
}
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_FALSE(stats.typing_noise_detected);
+}
+} // namespace test
} // namespace webrtc
« no previous file with comments | « webrtc/audio/audio_send_stream.cc ('k') | webrtc/audio/conversion.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698