Index: components/test/data/password_manager/automated_tests/run_tests.py |
diff --git a/components/test/data/password_manager/automated_tests/run_tests.py b/components/test/data/password_manager/automated_tests/run_tests.py |
index f64a26179317915403655cb32a9c19120adb8cb7..95cb4631bb004eab4e7d088a1de9ecc863588dcb 100644 |
--- a/components/test/data/password_manager/automated_tests/run_tests.py |
+++ b/components/test/data/password_manager/automated_tests/run_tests.py |
@@ -91,10 +91,9 @@ def LogResultsOfTestRun(config, results): |
for result in results: |
website, test_case, success, reason = result |
if not (config.save_only_fails and success): |
- logger.debug("Test case %s has %s on Website %s", test_case, |
- website, {True: "passed", False: "failed"}[success]) |
if not success: |
- logger.debug("Reason of failure: %s", reason) |
+ logger.debug("%s.%s failed with reason: %s", |
+ website, test_case, reason) |
if not success: |
failed_tests.append("%s.%s" % (website, test_case)) |
@@ -120,7 +119,7 @@ def RunTestCaseOnWebsite((website, test_case, config)): |
logger.log(SCRIPT_DEBUG, "Run of test case %s of website %s started", |
test_case, website) |
try: |
- with stopit.ThreadingTimeout(100) as timeout: |
+ with stopit.ThreadingTimeout(seconds=100) as timeout: |
logger.log(SCRIPT_DEBUG, |
"Run test with parameters: %s %s %s %s %s %s", |
config.chrome_path, config.chromedriver_path, |
@@ -132,7 +131,7 @@ def RunTestCaseOnWebsite((website, test_case, config)): |
if timeout.state != timeout.EXECUTED: |
result = (website, test_case, False, |
"Got %d as timeout state (see stopit.ThreadingTimeout for" |
- "the meaning of the number)" % timeout.state) |
+ " the meaning of the number)" % timeout.state) |
_, _, success, _ = result |
if success: |
return result |