From 7ac010cea48b470e4c2d0ada3de428cf36bf34d4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Patrik=20H=C3=B6glund?= Date: Thu, 26 Mar 2020 14:24:47 +0100 Subject: [PATCH] Fix low bw test. The problem turned out to be that it passes . as the path, and that does not work in the PYTHONPATH. Also remove debug logging. Bug: chromium:1029452 Change-Id: Ied5211f6c039b41da9d77638801e67b7ea8f192f Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/171806 Reviewed-by: Mirko Bonadei Cr-Commit-Position: refs/heads/master@{#30903} --- audio/test/low_bandwidth_audio_test.py | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/audio/test/low_bandwidth_audio_test.py b/audio/test/low_bandwidth_audio_test.py index c048098f73..cf9047640a 100755 --- a/audio/test/low_bandwidth_audio_test.py +++ b/audio/test/low_bandwidth_audio_test.py @@ -230,17 +230,9 @@ def _ConfigurePythonPath(args): # python, so put it in the path for this script before we attempt to import # it. histogram_proto_path = os.path.join( - args.build_dir, 'pyproto', 'tracing', 'tracing', 'proto') + os.path.abspath(args.build_dir), 'pyproto', 'tracing', 'tracing', 'proto') sys.path.insert(0, histogram_proto_path) - logging.info('Contents of build dir:') - logging.info(str(os.listdir(args.build_dir))) - for root, dirs, files in os.walk(os.path.join(args.build_dir, 'pyproto')): - path = root.split(os.sep) - logging.info((len(path) - 1) * '---' + os.path.basename(root)) - for file in files: - logging.info(len(path) * '---' + file) - # Fail early in case the proto hasn't been built. from tracing.proto import histogram_proto if not histogram_proto.HAS_PROTO: @@ -253,7 +245,6 @@ def _ConfigurePythonPath(args): def main(): # pylint: disable=W0101 logging.basicConfig(level=logging.INFO) - logging.info('Invoked with %s', str(sys.argv)) args = _ParseArgs()