Adding basic implementation of AudioNetworkAdaptor.

webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl_unittest.cc

Index: webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl_unittest.cc
diff --git a/webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl_unittest.cc b/webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..e172fa1a6565c821b827bd837cd8e991d857cca6
--- /dev/null
+++ b/webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl_unittest.cc
@@ -0,0 +1,112 @@
+/*
+ *  Copyright (c) 2016 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 <utility>
+#include <vector>
+
+#include "testing/gtest/include/gtest/gtest.h"
+#include "webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h"
+#include "webrtc/modules/audio_coding/audio_network_adaptor/mock/mock_controller.h"
+#include "webrtc/modules/audio_coding/audio_network_adaptor/mock/mock_controller_manager.h"
+
+namespace webrtc {
+
+using ::testing::_;
+using ::testing::NiceMock;
+using ::testing::Return;
+
+namespace {
+
+constexpr size_t kNumControllers = 2;
+
+MATCHER_P(NetworkMetricsIs, metric, "") {
+  return arg.uplink_bandwidth_bps == metric.uplink_bandwidth_bps &&
+         arg.uplink_packet_loss_fraction == metric.uplink_packet_loss_fraction;
+}
+
+MATCHER_P(ConstraintsReceiverFrameLengthRangeIs, frame_length_range, "") {
+  return (*arg.receiver_frame_length_range).min_frame_length_ms ==

[hlundin-webrtc, 2016/09/07 14:20:31]

*arg. is the same as arg->

[minyue-webrtc, 2016/09/08 11:06:14]

Not true, |receiver_frame_length_range| is Optional, and * is to get its value.

[kwiberg-webrtc, 2016/09/08 11:25:34]

Still, (*foo).bar is the same as foo->bar even when foo is an Optional, so you
should be able to write

  arg.receiver_frame_length_range->min_frame_length_ms

+             frame_length_range.min_frame_length_ms &&
+         (*arg.receiver_frame_length_range).max_frame_length_ms ==
+             frame_length_range.max_frame_length_ms;
+}
+
+}  // namespace
+
+class AudioNetworkAdaptorImplTest : public ::testing::Test {
+ protected:
+  AudioNetworkAdaptorImplTest()
+      : audio_network_adaptor_(nullptr), mock_controller_manager_(nullptr) {}
+
+  void SetUp() override {
+    std::vector<Controller*> controllers;
+    for (size_t i = 0; i < kNumControllers; ++i) {
+      auto controller =
+          std::unique_ptr<MockController>(new NiceMock<MockController>());
+      EXPECT_CALL(*controller, Die());
+      controllers.push_back(controller.get());
+      mock_controllers_.push_back(std::move(controller));
+    }
+
+    auto controller_manager = std::unique_ptr<MockControllerManager>(
+        new NiceMock<MockControllerManager>());
+
+    mock_controller_manager_ = controller_manager.get();
+    EXPECT_CALL(*mock_controller_manager_, Die());
+    EXPECT_CALL(*mock_controller_manager_, GetControllers())
+        .WillRepeatedly(Return(controllers));

[hlundin-webrtc, 2016/09/07 14:20:31]

Won't controllers have been gone out of scope when the expected calls happen?

[minyue-webrtc, 2016/09/08 11:06:14]

This is what I asked myself too. I think gmock automatically makes a
ReturnRefOfCopyAction, and hence handle it properly.

https://cs.chromium.org/chromium/src/testing/gmock/include/gmock/gmock-action...

[hlundin-webrtc, 2016/09/08 13:52:05]

Acknowledged.

+    EXPECT_CALL(*mock_controller_manager_, GetSortedControllers(_))
+        .WillRepeatedly(Return(controllers));
+
+    // AudioNetworkAdaptorImpl governs the lifetime of controller manager.
+    audio_network_adaptor_.reset(new AudioNetworkAdaptorImpl(
+        AudioNetworkAdaptorImpl::Config(), std::move(controller_manager)));
+  }
+
+  std::unique_ptr<AudioNetworkAdaptorImpl> audio_network_adaptor_;
+
+  MockControllerManager* mock_controller_manager_;
+
+  std::vector<std::unique_ptr<MockController>> mock_controllers_;
+};
+
+TEST_F(AudioNetworkAdaptorImplTest,
+       MakeDecisionIsCalledOnGetEncoderRuntimeConfig) {
+  constexpr int kBandwidth = 16000;
+  constexpr float kPacketLoss = 0.7f;
+
+  Controller::NetworkMetrics check;
+  check.uplink_bandwidth_bps = rtc::Optional<int>(kBandwidth);
+
+  for (auto& mock_controller : mock_controllers_) {
+    EXPECT_CALL(*mock_controller, MakeDecision(NetworkMetricsIs(check), _));
+  }
+  audio_network_adaptor_->SetUplinkBandwidth(kBandwidth);
+  audio_network_adaptor_->GetEncoderRuntimeConfig();
+
+  check.uplink_packet_loss_fraction = rtc::Optional<float>(kPacketLoss);
+  for (auto& mock_controller : mock_controllers_) {
+    EXPECT_CALL(*mock_controller, MakeDecision(NetworkMetricsIs(check), _));
+  }
+  audio_network_adaptor_->SetUplinkPacketLossFraction(kPacketLoss);
+  audio_network_adaptor_->GetEncoderRuntimeConfig();
+}
+
+TEST_F(AudioNetworkAdaptorImplTest,
+       SetConstraintsIsCalledOnSetFrameLengthRange) {
+  for (auto& mock_controller : mock_controllers_) {
+    EXPECT_CALL(*mock_controller,
+                SetConstraints(ConstraintsReceiverFrameLengthRangeIs(
+                    Controller::Constraints::FrameLengthRange(20, 120))));
+  }
+  audio_network_adaptor_->SetReceiverFrameLengthRange(20, 120);
+}
+
+}  // namespace webrtc