Skip to content

A
angle
Commit 1429130b by Yuly Novikov Committed by Angle LUCI CQ
Options

Gold Tests: Fix batch result reporting

Previously artifacts were not reset for a new trace in a batch Bug: angleproject:5966 Change-Id: Ia6ed8c070eaaeca7e6cea887419666f630648df8 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2925218Reviewed-by: 's avatarYuly Novikov <ynovikov@chromium.org> Commit-Queue: Yuly Novikov <ynovikov@chromium.org>
parent 45e1596c
Showing with 2 additions and 2 deletions
src/tests/restricted_traces/restricted_trace_gold_tests.py
...@@ -380,9 +380,9 @@ def _run_tests(args, tests, extra_flags, env, screenshot_dir, results, test_resu ...@@ -380,9 +380,9 @@ def _run_tests(args, tests, extra_flags, env, screenshot_dir, results, test_resu
batch_result = PASS if run_wrapper(args, cmd, env, batch_result = PASS if run_wrapper(args, cmd, env,
tempfile_path) == 0 else FAIL tempfile_path) == 0 else FAIL
artifacts = {}
for trace in batch: for trace in batch:
artifacts = {}
if batch_result == PASS: if batch_result == PASS:
result = upload_test_result_to_skia_gold(args, gold_session_manager, result = upload_test_result_to_skia_gold(args, gold_session_manager,
gold_session, gold_properties, gold_session, gold_properties,
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment