Commit f0d41ff4 authored by melandory's avatar melandory Committed by Commit bot

[Password manager tests automation] Fix automated tests runner.

Fix error introduced while adressing comments in
https://codereview.chromium.org/1089383002/

BUG=369521
R=vabr@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#330501}
parent 0ea1107b
...@@ -129,8 +129,10 @@ def RunTestCaseOnWebsite((website, test_case, config)): ...@@ -129,8 +129,10 @@ def RunTestCaseOnWebsite((website, test_case, config)):
result = tests.RunTest(config.chrome_path, config.chromedriver_path, result = tests.RunTest(config.chrome_path, config.chromedriver_path,
profile_path, config.passwords_path, profile_path, config.passwords_path,
website, test_case)[0] website, test_case)[0]
if timeout != timeout.EXECUTED: if timeout.state != timeout.EXECUTED:
result = (website, test_case, False, "Timeout") result = (website, test_case, False,
"Got %d as timeout state (see stopit.ThreadingTimeout for"
"the meaning of the number)" % timeout.state)
_, _, success, _ = result _, _, success, _ = result
if success: if success:
return result return result
......
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