Low-bandwidth audio testing

webrtc/audio/BUILD.gn

 # Copyright (c) 2015 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.
 
 import("../webrtc.gni")
 if (is_android) {
@@ skipping 60 matching lines @@
       "../modules/audio_device:mock_audio_device",
       "../modules/audio_mixer:audio_mixer_impl",
       "../modules/congestion_controller:congestion_controller",
       "../modules/pacing:pacing",
       "../test:test_common",
       "../test:test_support",
       "utility:utility_tests",
       "//testing/gmock",
       "//testing/gtest",
     ]
+    if (rtc_enable_protobuf) {
+      deps += [ ":low_bandwidth_audio_test" ]
+    }
+
     if (!build_with_chromium && is_clang) {
       # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
       suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
     }
   }
 
   if (rtc_enable_protobuf) {
     rtc_executable("low_bandwidth_audio_test") {
       testonly = true
 
       sources = [
         "test/low_bandwidth_audio_test.cc",
+        "test/low_bandwidth_audio_test.h",
       ]
 
-      deps = []
+      deps = [
+        "../common_audio",
+        "../system_wrappers",
+        "../test:fake_audio_device",
+        "../test:run_test",
+        "../test:test_common",
+        "../test:test_main",
+      ]
+      if (is_android) {
+        deps += [ "//testing/android/native_test:native_test_native_code" ]
+      } else {
+        deps += [ "../modules/video_capture:video_capture_internal_impl" ]

[stefan-webrtc, 2017/03/20 14:40:10]

Why do we need a video capturer for the audio tests?

[oprypin_webrtc, 2017/03/20 15:09:16]

I suspect that call_test pulls that in, since it also supports video. I get an
error without it, see previous comments on this line:
https://codereview.webrtc.org/2694203002/diff/160001/webrtc/audio/BUILD.gn#ne...

[stefan-webrtc, 2017/03/20 15:42:32]

Shouldn't it be added as a dependency on the target containing call_test then?

[oprypin_webrtc, 2017/03/20 16:02:35]

Done. Thanks.

+      }
 
-      if (is_android) {
-        deps += [ "//testing/android/native_test:native_test_support" ]
+      data = [
+        "//resources/voice_engine/audio_tiny16.wav",
+      ]
+
+      if (!build_with_chromium && is_clang) {
+        # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163)
+        suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
       }
     }
   }
 }