[crosperf] Make generate_report report actual failures.
We would never increment `num_success`, so we'd always return failure.
This passed tests, since there were no tests that checked for success.
Also, testRunActionsRunsAllActionsRegardlessOfExceptions wasn't
really doing what it was supposed to; exceptions wouldn't get raised
(since the mocked WriteFile turns into a nop), so all of the WriteFile
actions were "passing". Because of the aforementioned bug, this test
seemed to work, as well.
BUG=None
TEST=generate_report_unittest.py passes; cros lint is happy.
Change-Id: If4d71af3415d97be8d5ac3bec796fe4589d50aaa
Reviewed-on: https://chrome-internal-review.googlesource.com/289802
Commit-Ready: George Burgess <gbiv@google.com>
Tested-by: George Burgess <gbiv@google.com>
Reviewed-by: Caroline Tice <cmtice@google.com>
2 files changed