Unit test for AudioFrame output from AcmReceiver::GetAudio

webrtc/modules/audio_coding/acm2/acm_receiver_unittest_oldapi.cc

 /*
  *  Copyright (c) 2013 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/modules/audio_coding/acm2/acm_receiver.h"
 
 #include <algorithm>  // std::min
 #include <memory>
 
 #include "testing/gtest/include/gtest/gtest.h"
+#include "webrtc/base/checks.h"
+#include "webrtc/base/safe_conversions.h"
 #include "webrtc/modules/audio_coding/include/audio_coding_module.h"
 #include "webrtc/modules/audio_coding/acm2/audio_coding_module_impl.h"
 #include "webrtc/modules/audio_coding/neteq/tools/rtp_generator.h"
 #include "webrtc/system_wrappers/include/clock.h"
 #include "webrtc/test/test_suite.h"
 #include "webrtc/test/testsupport/fileutils.h"
 
 namespace webrtc {
 
 namespace acm2 {
@@ skipping 255 matching lines @@
     const CodecIdInst codec(codec_id);
     const int num_10ms_frames = codec.inst.pacsize / (codec.inst.plfreq / 100);
     InsertOnePacketOfSilence(codec.id);
     for (int k = 0; k < num_10ms_frames; ++k) {
       EXPECT_EQ(0, receiver_->GetAudio(kOutSampleRateHz, &frame));
     }
     EXPECT_EQ(codec.inst.plfreq, receiver_->last_output_sample_rate_hz());
   }
 }
 
+class AcmReceiverTestFaxModeOldApi : public AcmReceiverTestOldApi {
+ protected:
+  AcmReceiverTestFaxModeOldApi() {
+    config_.neteq_config.playout_mode = kPlayoutFax;
+  }
+};
+
+#if defined(WEBRTC_ANDROID)
+#define MAYBE_VerifyAudioFrame DISABLED_VerifyAudioFrame
+#else
+#define MAYBE_VerifyAudioFrame VerifyAudioFrame
+#endif
+TEST_F(AcmReceiverTestFaxModeOldApi, MAYBE_VerifyAudioFrame) {
+  // Make sure "fax mode" is enabled. This will avoid delay changes unless the
+  // packet-loss concealment is made. We do this in order to make the timestamp
+  // increments predictable; in normal mode, NetEq may decide to do accelerate
+  // or pre-emptive expand operations after some time, offsetting the timestamp.
+  EXPECT_EQ(kPlayoutFax, config_.neteq_config.playout_mode);
+
+  const RentACodec::CodecId codec_id = RentACodec::CodecId::kOpus;
+  const RentACodec::CodecId kCodecId[] = {codec_id};
+  AddSetOfCodecs(kCodecId);
+
+  const CodecIdInst codec(codec_id);
+  const int output_sample_rate_hz = codec.inst.plfreq;
+  const size_t samples_per_ms = rtc::checked_cast<size_t>(
+      rtc::CheckedDivExact(output_sample_rate_hz, 1000));
+  const int num_10ms_frames = codec.inst.pacsize / (10 * samples_per_ms);

[minyue-webrtc, 2016/04/05 13:00:22]

no CheckedDivExact on this?

[hlundin-webrtc, 2016/04/05 13:24:14]

Done.

+
+  // Expect the first output timestamp to be 30 samples before the first
+  // inserted timestamp (because of NetEq's look-ahead).
+  uint32_t expected_output_ts = last_packet_send_timestamp_ - 30u;

[minyue-webrtc, 2016/04/05 13:00:22]

what would it be if |last_packet_send_timestamp_| is smaller than 30u?

[hlundin-webrtc, 2016/04/05 13:24:14]

In fact it is; last_packet_send_timestamp_ = 0 from the test setup. The
subtraction wraps to the high end of uint32_t as expected.

[minyue-webrtc, 2016/04/05 13:30:04]

Acknowledged.

+
+  AudioFrame frame;
+  for (int i = 0; i < 5; ++i) {
+    InsertOnePacketOfSilence(codec.id);
+    for (int k = 0; k < num_10ms_frames; ++k) {
+      EXPECT_EQ(0, receiver_->GetAudio(output_sample_rate_hz, &frame));
+      EXPECT_EQ(expected_output_ts, frame.timestamp_);
+      expected_output_ts += 10 * samples_per_ms;
+      EXPECT_EQ(10 * samples_per_ms, frame.samples_per_channel_);
+      EXPECT_EQ(output_sample_rate_hz, frame.sample_rate_hz_);
+      EXPECT_EQ(2u, frame.num_channels_);
+      EXPECT_EQ(AudioFrame::kNormalSpeech, frame.speech_type_);
+      EXPECT_EQ(AudioFrame::kVadActive, frame.vad_activity_);

[minyue-webrtc, 2016/04/05 13:00:22]

why is VadActive when packet is silent?

[hlundin-webrtc, 2016/04/05 13:24:14]

There is a restriction on the post-decode VAD in NetEq that makes it always say
"active speech" for sample rates > 16000 Hz. Legacy...

[minyue-webrtc, 2016/04/05 13:30:04]

so if codec_id equals to a 16kHz codec, this line will fail?

Would you add a comment? or even drop this line.

+    }
+  }
+}
+
 #if defined(WEBRTC_ANDROID)
 #define MAYBE_PostdecodingVad DISABLED_PostdecodingVad
 #else
 #define MAYBE_PostdecodingVad PostdecodingVad
 #endif
 TEST_F(AcmReceiverTestOldApi, MAYBE_PostdecodingVad) {
   EXPECT_TRUE(config_.neteq_config.enable_post_decode_vad);
   const CodecIdInst codec(RentACodec::CodecId::kPCM16Bwb);
   ASSERT_EQ(
       0, receiver_->AddCodec(codec.id, codec.inst.pltype, codec.inst.channels,
@@ skipping 105 matching lines @@
               receiver_->last_packet_sample_rate_hz());
     EXPECT_EQ(0, receiver_->LastAudioCodec(&codec));
     EXPECT_TRUE(CodecsEqual(c.inst, codec));
   }
 }
 #endif
 
 }  // namespace acm2
 
 }  // namespace webrtc