Index: third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_from_try_jobs.py |
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_from_try_jobs.py b/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_from_try_jobs.py |
index d8617bb75d0a1d182c0f4749e4ec234b664e69d2..28f21214c2f737037769ea7e286e4f425bd86602 100644 |
--- a/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_from_try_jobs.py |
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_from_try_jobs.py |
@@ -76,27 +76,26 @@ class RebaselineFromTryJobs(AbstractParallelRebaselineCommand): |
def _test_prefix_list(self, issue_number): |
"""Returns a collection of test, builder and file extensions to get new baselines for.""" |
- builders_to_tests = self._builders_to_tests(issue_number) |
+ builds_to_tests = self._builds_to_tests(issue_number) |
result = {} |
- for builder, tests in builders_to_tests.iteritems(): |
+ for build, tests in builds_to_tests.iteritems(): |
for test in tests: |
if test not in result: |
result[test] = {} |
- # TODO(qyearsley): Consider using TestExpectations.suffixes_for_test_result. |
- result[test][builder] = BASELINE_SUFFIX_LIST |
+ result[test][build] = BASELINE_SUFFIX_LIST |
return result |
- def _builders_to_tests(self, issue_number): |
+ def _builds_to_tests(self, issue_number): |
"""Fetches a list of try bots, and for each, fetches tests with new baselines.""" |
_log.debug('Getting results for Rietveld issue %d.' % issue_number) |
try_jobs = latest_try_jobs(issue_number, self._try_bots(), self.web) |
if not try_jobs: |
_log.debug('No try job results for builders in: %r.' % (self._try_bots(),)) |
- builders_to_tests = {} |
+ builds_to_tests = {} |
for job in try_jobs: |
test_results = self._unexpected_mismatch_results(job) |
- builders_to_tests[job.builder_name] = sorted(r.test_name() for r in test_results) |
- return builders_to_tests |
+ builds_to_tests[(job.builder_name, job.build_number)] = sorted(r.test_name() for r in test_results) |
+ return builds_to_tests |
def _try_bots(self): |
"""Retuns a collection of try bot builders to fetch results for.""" |
@@ -119,5 +118,6 @@ class RebaselineFromTryJobs(AbstractParallelRebaselineCommand): |
_log.info('No tests to rebaseline.') |
return |
_log.info('Tests to rebaseline:') |
- for test, builders in test_prefix_list.iteritems(): |
- _log.info(' %s: %s', test, ', '.join(sorted(builders))) |
+ for test, builds in test_prefix_list.iteritems(): |
+ builds_str = ', '.join(sorted('%s (%s)' % b for b in builds)) |
+ _log.info(' %s: %s', test, builds_str) |