Add src-side launcher for telemetry_unittests

testing/scripts/get_compile_targets.py

+#!/usr/bin/env python
+# Copyright 2014 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+import argparse
+import json
+import os
+import sys
+
+import common
+
+
+def main(argv):
+  parser = argparse.ArgumentParser()
+  parser.add_argument('--output', required=True)
+  parser.add_argument('args', nargs=argparse.REMAINDER)
+
+  args = parser.parse_args(argv)
+
+  passthrough_args = args.args
+  if passthrough_args[0] == '--':
+    passthrough_args = passthrough_args[1:]
+
+  results = {}
+
+  for filename in os.listdir(common.SCRIPT_DIR):
+    if not filename.endswith('.py'):
+      continue
+    if filename in ('common.py', 'get_compile_targets.py'):
+      continue
+
+    with common.temporary_file() as tempfile_path:
+      rc = common.run_command(
+          [os.path.join(common.SCRIPT_DIR, filename)] +
+          passthrough_args +

[iannucci, 2014/10/22 07:23:15]

why are passthrough_args before the mode parameter? that would bury the mode
parameter pretty far down the command line (trying to avoid another runtest.py
CLI fiasco :D).

[Paweł Hajdan Jr., 2014/10/22 09:43:27]

Suggestions are welcome - I think the reason is these arguments are defined in
the main parser as opposed to each of the subparsers, since they're common to
all subparsers.

+          [
+              'compile_targets',
+              '--output', tempfile_path
+          ]
+      )
+      if rc != 0:
+        return rc
+
+      with open(tempfile_path) as f:
+        results[filename] = json.load(f)
+
+  with open(args.output, 'w') as f:
+    json.dump(results, f)
+
+  return 0
+
+
+if __name__ == '__main__':
+  sys.exit(main(sys.argv[1:]))