diff options
| author | George Burgess IV <gbiv@google.com> | 2016-08-25 11:57:01 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2016-08-26 17:09:03 -0700 |
| commit | afb8cc77e82c35faedfe541d097fc01fd1d7ca3d (patch) | |
| tree | cd2473b3f517ca25afe48ff7ef8d9b741d68045e /crosperf/results_organizer_unittest.py | |
| parent | 2f4a226d58efdf4eabe560a64e711c16599db08d (diff) | |
| download | toolchain-utils-afb8cc77e82c35faedfe541d097fc01fd1d7ca3d.tar.gz | |
crosperf: refactor results_organizer; NFC.
As part of playing with results_report (so I could make it less
ChromeOS-dependent), I started refactoring results_organizer, which is
only used by results_report.
This was the result. The goal was to make it easier to understand what,
exactly, results_organizer is trying to do.
BUG=chromium:641098
TEST=./test_all.sh passes.
Change-Id: I8e53175fbc1ded6f23631550223905b3321376ca
Reviewed-on: https://chrome-internal-review.googlesource.com/280815
Commit-Ready: George Burgess <gbiv@google.com>
Tested-by: George Burgess <gbiv@google.com>
Reviewed-by: Caroline Tice <cmtice@google.com>
Diffstat (limited to 'crosperf/results_organizer_unittest.py')
| -rwxr-xr-x | crosperf/results_organizer_unittest.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/crosperf/results_organizer_unittest.py b/crosperf/results_organizer_unittest.py index 74d5c23a..0d214124 100755 --- a/crosperf/results_organizer_unittest.py +++ b/crosperf/results_organizer_unittest.py @@ -16,7 +16,7 @@ import unittest from benchmark_run import BenchmarkRun from results_cache import Result -from results_organizer import ResultOrganizer +from results_organizer import OrganizeResults import mock_instance @@ -101,8 +101,8 @@ class ResultOrganizerTest(unittest.TestCase): b.result.keyvals = mock_instance.keyval[i] i += 1 - ro = ResultOrganizer(benchmark_runs, labels, benchmarks) - self.assertEqual(ro.result, result) + organized = OrganizeResults(benchmark_runs, labels, benchmarks) + self.assertEqual(organized, result) if __name__ == '__main__': |