[tools] Pass output to GetOutcome function
R=majeski@google.com TBR=machenbach@chromium.org This is breaking some test runs, e.g. https://chromium-swarm.appspot.com/task?id=3a69c31ccb964d10&refresh=10&show_raw=1. Bug: v8:6917 Change-Id: I61c12eafdf12a1bee2bf08de75f26b0d44fd0056 Reviewed-on: https://chromium-review.googlesource.com/825122 Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org> Cr-Commit-Position: refs/heads/master@{#50089}
This commit is contained in:
parent
9034d8c8a5
commit
599324d03e
@ -357,7 +357,7 @@ class JsonTestProgressIndicator(ProgressIndicator):
|
||||
"stdout": output.stdout,
|
||||
"stderr": output.stderr,
|
||||
"exit_code": output.exit_code,
|
||||
"result": test.suite.GetOutcome(test),
|
||||
"result": test.suite.GetOutcome(test, output),
|
||||
"expected": test.suite.GetExpectedOutcomes(test),
|
||||
"duration": test.duration,
|
||||
|
||||
@ -396,7 +396,7 @@ class FlakinessTestProgressIndicator(ProgressIndicator):
|
||||
|
||||
def HasRun(self, test, output, has_unexpected_output):
|
||||
key = test.get_id()
|
||||
outcome = test.suite.GetOutcome(test)
|
||||
outcome = test.suite.GetOutcome(test, output)
|
||||
assert outcome in ["PASS", "FAIL", "CRASH", "TIMEOUT"]
|
||||
if test.run == 1:
|
||||
# First run of this test.
|
||||
|
Loading…
Reference in New Issue
Block a user