Commit c6232a86 authored by vabr's avatar vabr Committed by Commit bot

[Password manager Python tests] Squash INFO logs into one per test

This is a rather cosmetic change: INFO log level is used to print test summaries. This CL makes the script log one summary instead of 2 consecutive ones. These logs end up in a spreadsheet, and having one line (as opposed to 2) per test is easier to process for statistics.

R=engedy@chromium.org
BUG=468245

Review URL: https://codereview.chromium.org/1023673005

Cr-Commit-Position: refs/heads/master@{#321448}
parent 5581a3b6
......@@ -219,8 +219,9 @@ def run_tests(config_path):
runners.append(TestRunner(specific_test_cmd, test_name))
time.sleep(1)
failed_tests = [(name, log) for (name, passed, log) in results if not passed]
logger.info("%d failed tests out of %d", len(failed_tests), len(results))
logger.info("Failing tests: %s", [name for (name, _) in failed_tests])
logger.info("%d failed tests out of %d, failing tests: %s",
len(failed_tests), len(results),
[name for (name, _) in failed_tests])
logger.debug("Logs of failing tests: %s", failed_tests)
......
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