Conversational Speech tool, MultiEndCall::CheckTiming() and tests

webrtc/modules/audio_processing/test/conversational_speech/mock_wavreader_factory.cc

 /*
  *  Copyright (c) 2017 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_processing/test/conversational_speech/mock_wavreader_factory.h"
 
+#include "webrtc/base/logging.h"
+#include "webrtc/base/pathutils.h"
+#include "webrtc/modules/audio_processing/test/conversational_speech/mock_wavreader.h"
+#include "webrtc/test/gmock.h"
+
 namespace webrtc {
 namespace test {
 namespace conversational_speech {
 
-MockWavReaderFactory::MockWavReaderFactory() = default;
+using testing::_;
+using testing::Invoke;
+
+MockWavReaderFactory::MockWavReaderFactory(const Params& default_params)
+    : default_params_(default_params) {
+  ON_CALL(*this, Create(_)).WillByDefault(Invoke(
+      this, &MockWavReaderFactory::CreateMock));
+}
+
+MockWavReaderFactory::MockWavReaderFactory(
+    const Params& default_params,
+    const std::map<std::string, const Params>& params)
+        : MockWavReaderFactory(default_params) {
+  audiotrack_names_params_.insert(params.begin(), params.end());

[hlundin-webrtc, 2017/04/06 08:10:04]

You should be able to use the copy constructor, and put in the initializer list:
audiotrack_names_params_(params)
Then you can also make audiotrack_names_params_ const.

[AleBzk, 2017/04/06 16:42:42]

If I'm correct, we have two options:
1) instead of calling the MockWavReaderFactory(const Params& default_params)
ctor in the initializer list of the second ctor, we duplicate the code of the
former in the latter and then yes, we can do as you suggest.
2) we leave everything as it is to avoid code duplication

Personally, I prefer the second.
But there might be other ways of using the copy ctor while still calling
MockWavReaderFactory(const Params& default_params).

[hlundin-webrtc, 2017/04/07 10:24:09]

I wonder if we are talking about different things. I mean you could write this
ctor as:

MockWavReaderFactory::MockWavReaderFactory(
    const Params& default_params,
    const std::map<std::string, const Params>& params)
        : MockWavReaderFactory(default_params)
          audiotrack_names_params_(params) {}

That is, use the copy ctor of audiotrack_names_params_ instead of calling the
insert() method. Unless I'm mistaken, you should then also be able to make
audiotrack_names_params_ const declared, since you are only setting it in an
initializer list.

[AleBzk, 2017/04/07 11:37:06]

Yes, then I understood correctly your point.
I already tried that and it fails with " error: an initializer for a delegating
constructor must appear alone".

However, I fixed by removing the ctor with a single arg and adding a default
value.
Now audiotrack_names_params_is const ref.

[hlundin-webrtc, 2017/04/07 11:54:18]

Do like this:
Let the member be a mutable variable (not a ref).
Use copy ctor here.

+}
 
 MockWavReaderFactory::~MockWavReaderFactory() = default;
 
+std::unique_ptr<WavReaderInterface> MockWavReaderFactory::CreateMock(
+    const std::string& filepath) {
+  // Search the parameters corresponding to filepath.
+  const rtc::Pathname audiotrack_file_path(filepath);
+  auto it = audiotrack_names_params_.find(audiotrack_file_path.filename());

[hlundin-webrtc, 2017/04/06 08:10:04]

const auto it?

[AleBzk, 2017/04/06 16:42:42]

Done.

+
+  // If not found, use default parameters.
+  if (it == audiotrack_names_params_.end()) {
+    LOG(LS_VERBOSE) << "using default parameters for " << filepath;
+    return std::unique_ptr<WavReaderInterface>(
+        new MockWavReader(default_params_.sample_rate,
+                          default_params_.num_channels,
+                          default_params_.num_samples));
+  }
+
+  // Found, use the audiotrack-specific parameters.
+  LOG(LS_VERBOSE) << "using ad-hoc parameters for " << filepath;
+  LOG(LS_VERBOSE) << "sample_rate " << it->second.sample_rate;
+  LOG(LS_VERBOSE) << "num_channels " << it->second.num_channels;
+  LOG(LS_VERBOSE) << "num_samples " << it->second.num_samples;
+  return std::unique_ptr<WavReaderInterface>(
+      new MockWavReader(it->second.sample_rate,
+                        it->second.num_channels,
+                        it->second.num_samples));
+}
+
 }  // namespace conversational_speech
 }  // namespace test
 }  // namespace webrtc