Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(586)

Unified Diff: scripts/slave/recipe_modules/test_results/resources/upload_test_results.py

Issue 2475973002: [recipe_modules/test_result] Support uploading full json results format for upload_test_results.py (Closed)
Patch Set: Address review comments Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/test_results/resources/upload_test_results_unittest.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipe_modules/test_results/resources/upload_test_results.py
diff --git a/scripts/slave/recipe_modules/test_results/resources/upload_test_results.py b/scripts/slave/recipe_modules/test_results/resources/upload_test_results.py
index 7a9d3d55576fb23039a2121a59ff4a6510c0be89..2f3963c11698788a183ebdbc6ef5b35977d4ec27 100755
--- a/scripts/slave/recipe_modules/test_results/resources/upload_test_results.py
+++ b/scripts/slave/recipe_modules/test_results/resources/upload_test_results.py
@@ -135,12 +135,19 @@ def main(args):
with file(options.input_json) as json_file:
results_json = json_file.read()
- files = generate_json_results_file(
- results_json, builder_name=options.builder_name,
- build_number=options.build_number,
- results_directory=options.results_directory,
- chrome_revision=options.chrome_revision,
- master_name=options.master_name)
+ content = json.loads(results_json)
+ if content.get('version', 0) >= 3:
+ print 'Input JSON file probably has full json results format'
+ files = [(os.path.basename(options.input_json), options.input_json)]
+ else:
+ print ('Input JSON file probably has gtest format. Converting to full json'
+ ' results format')
+ files = generate_json_results_file(
+ results_json, builder_name=options.builder_name,
+ build_number=options.build_number,
+ results_directory=options.results_directory,
+ chrome_revision=options.chrome_revision,
+ master_name=options.master_name)
# Upload to a test results server if specified.
if options.test_results_server and options.master_name:
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/test_results/resources/upload_test_results_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698