Merge test results presentation of multiple shards.

build/android/pylib/results/presentation/test_results_presentation.py

-#! /usr/bin/env python
+#!/usr/bin/env python
 #
 # Copyright 2017 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 collections
 import json
 import tempfile
 import os
 import sys
 import urllib
 
+
 CURRENT_DIR = os.path.dirname(os.path.abspath(__file__))
 BASE_DIR = os.path.abspath(os.path.join(
     CURRENT_DIR, '..', '..', '..', '..', '..'))
 
 sys.path.append(os.path.join(BASE_DIR, 'build', 'android'))
 from pylib.utils import google_storage_helper  # pylint: disable=import-error
+from pylib.results.presentation import standard_gtest_merge

[jbudorick, 2017/06/21 21:05:06]

nit: this should precede the google_storage_helper import

[BigBossZhiling, 2017/06/21 22:10:24]

Done.

 
 sys.path.append(os.path.join(BASE_DIR, 'third_party'))
 import jinja2  # pylint: disable=import-error
 JINJA_ENVIRONMENT = jinja2.Environment(
     loader=jinja2.FileSystemLoader(os.path.dirname(__file__)),
     autoescape=True)
 
 
 def cell(data, html_class='center'):
   """Formats table cell data for processing in jinja template."""
@@ skipping 362 matching lines @@
     if ((not 'buildnumber' in build_properties) or
         (not 'buildername' in build_properties)):
       raise parser.error('Build number/builder name not specified.')
     build_number = build_properties['buildnumber']
     builder_name = build_properties['buildername']
   elif args.build_number and args.builder_name:
     build_number = args.build_number
     builder_name = args.builder_name
 
   if args.positional:
     if not len(args.positional) == 1:

[jbudorick, 2017/06/21 21:05:05]

nit: flip this conditional around to

  if len(args.positional) == 1:
    json_file = args.json_file
  else:
    ...

[BigBossZhiling, 2017/06/21 22:10:25]

Done.

-      raise parser.error('More than 1 json file specified.')
-    json_file = args.positional[0]
+      if args.output_json and args.summary_json:
+        standard_gtest_merge.standard_gtest_merge(
+            args.output_json, args.summary_json, args.positional)
+        json_file = args.output_json
+      else:
+        raise Exception('Arguments (output_json/summary_json) of'

[jbudorick, 2017/06/21 21:05:05]

This was not what I meant.

If args.output_json is missing, raise an exception saying that. If
args.summary_json is missing, raise an exception saying that. The two do not
have to be covered by the same exception.

[BigBossZhiling, 2017/06/21 22:10:25]

Done.

+                        ' merge API are missing. ')
+    else:
+      json_file = args.positional[0]
   elif args.json_file:
     json_file = args.json_file
 
   if not os.path.exists(json_file):
     raise IOError('--json-file %s not found.' % json_file)
 
   # Link to result details presentation page is a part of the page.
   result_html_string, dest, result_details_link = result_details(
       json_file, args.cs_base_url, args.bucket,
       args.test_name, builder_name, build_number)
@@ skipping 10 matching lines @@
     with open(json_file) as original_json_file:
       json_object = json.load(original_json_file)
       json_object['links'] = {'result_details': result_details_link}
       with open(args.output_json, 'w') as f:
         json.dump(json_object, f)
   else:
     print result_details_link
 
 if __name__ == '__main__':
   sys.exit(main())